Bug 6174 - Default skin: Alphapagebar in-page links position
: Default skin: Alphapagebar in-page links position
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 7.0
: PC Windows XP
: P2 minor (vote)
: ---
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-11-16 12:36 UTC by Jim McAtee
Modified: 2007-11-19 07:31 UTC (History)
0 users

See Also:
Category: ---


Attachments
force to top (939 bytes, patch)
2007-11-17 21:45 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2007-11-16 12:36:30 UTC
The alphapagebar isn't working quite right for in-page links.  For instance, when browsing artists if I go to the letter 'V', which is on the page containing T-U-V-W, it brings the first item beginning with 'V' into the very bottom of the frame, while the Classic skin correctly brings it to the top.
Comment 1 Michael Herger 2007-11-17 03:21:04 UTC
I know that's not perfect. But the difference between the two is that in Classic the whole page is scrolling. In Default it's only a DIV. And within DIVs link targets don't work. I therefore had to come up with some JS. Not sure I can make it much better than this.
Comment 2 Jim McAtee 2007-11-17 07:46:11 UTC
That's the trouble with creating a browser-based interface, isn't it?  Users expect some standard behaviors that they've experienced on countless other web pages.
Comment 3 KDF 2007-11-17 21:45:05 UTC
Created attachment 2422 [details]
force to top

This reverses the scrollIntoView by first setting the scroll to the bottom then moving the anchor into view.  The number used for scroll might need to be set with a higher value to deal with really long lists (does anyone set the itemsPerPage that high?).  I've tested with Firefox and Safari.
Comment 4 Michael Herger 2007-11-19 00:00:45 UTC
Change 14827 - ok with Opera as well, will test with IE later. Thanks again!
Comment 5 Michael Herger 2007-11-19 07:31:44 UTC
confirmed with IE