Bug 9055 – Clicking Highlighter in SC/web main menu should respect targets in links

Bug 9055 - Clicking Highlighter in SC/web main menu should respect targets in links

Summary:

Clicking Highlighter in SC/web main menu should respect targets in links

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.2
Platform:	PC Windows XP

Importance:	P5 minor (vote)
Target Milestone:	7.x
Assigned To:	Michael Herger

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-08-06 00:23 UTC by Michael Herger
Modified:	2009-07-31 10:26 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
possible fix (957 bytes, patch) 2008-08-06 03:02 UTC, Michael Herger	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Michael Herger 2008-08-06 00:23:13 UTC

When clicking the alarm item it doesn't always open in a new window, but sometimes will display the settings screen in the left frame. This happens if you don't click the link, but only the highlighter.

Comment 1 Michael Herger 2008-08-06 03:02:51 UTC

Created attachment 3749 [details]
possible fix

Problem: this can cause CSRF warning when opening eg. the settings

Comment 2 Michael Herger 2008-08-06 03:23:12 UTC

change 22408 - in cases where the highlighted element would link to a new window (not only a frame), we can't always open a link, as eg. the alarm link in the main menu would trigger a CSRF warning. In this case just swallow the click and don't do anything. The user will have to click a bit more precise to hit the link.

Comment 3 James Richardson 2008-12-15 12:06:00 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Comment 4 Chris Owens 2009-07-31 10:26:39 UTC

Reduce number of active targets for SC