Closed
Bug 78343
Opened 24 years ago
Closed 23 years ago
scrollbar thumb doesn't highlight during drag
Categories
(Core :: XUL, defect, P4)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla0.9.8
People
(Reporter: hwaara, Assigned: hwaara)
References
Details
Attachments
(1 file)
(deleted),
patch
|
andreww
:
review+
hewitt
:
superreview+
|
Details | Diff | Splinter Review |
If I grab a scrollbars "elevator", and drags it through a document or a webpage
then it does not highlight.
It should be highlighted all the time while the mouse is down. I believe this is
a regression.
i see this on linux too: when i click, the elevator hightlights, but as soon as
the drag starts, highlight vanish.
Could be a "modern3" thing, in which case it should be dup'ed against bug 78221
Comment 2•24 years ago
|
||
Well, I only see it in modern, and I don't think it is intended behavior (hence
not a dup of bug 78221) so I'll let the modern folks sort it out. ->hewitt
Assignee: trudelle → hewitt
Assignee | ||
Comment 3•24 years ago
|
||
No, this is not a themes thing. This is XPToolkit.
Testcase:
1. Click on a widget (choose whichever you like the most: button,
toolbar-button, scrollbar, form-button etc.) and then drag the mouse outside of
the widget - and mouse over it again.
It should depress as soon as you enter it again with the cursor, right? It
doesn't! This is throughout the app.
This started happening just before the modern-landing, and it happens in all
themes so it is certainly XPToolkit. Trudelle, get this reassigned ASAP - it's
really major loss of functionality.
Thanks
Assignee | ||
Comment 4•24 years ago
|
||
I would believe this is critical (or even a smoketest blocker for widget tests?)
but I will let the QAs sort that out.
CC'ing a bunch of folks to get attention, sorry if you are not related to this
problem, feel free to un-cc. :)
Summary: scrollbars elevator doesn't highlight during drag → focus on widgets is seriously messed up
Comment 5•24 years ago
|
||
Okay, some one was messing around with mousein/out behavior recently, this might
be related.
Comment 7•24 years ago
|
||
i see this on mac classic in the 5/1 build. the thumb deselects without the
mouse doing anything. i filed a bug which is probably a dupe.
Comment 8•24 years ago
|
||
The testcase above doesn't sound anything like the original description, but it
does sound like a dup of several other bugs. This bug is about the original
described problem, we aren't morphing it into 20 other problems. restoring
original summary.
Summary: focus on widgets is seriously messed up → scrollbars elevator doesn't highlight during drag
Comment 9•24 years ago
|
||
removing myself from the CC.. with a comment that hwaara, you changed have the
summary from a perfectly good, specific bug ("scrollbars elevator doesn't
highlight during drag") into a vague, unmeasurable, unverifiable bug ("focus on
widgets is seriously messed up").. good job!
Summary: scrollbars elevator doesn't highlight during drag → focus on widgets is seriously messed up
Updated•24 years ago
|
Summary: focus on widgets is seriously messed up → scrollbar thumb doesn't highlight during drag
Assignee | ||
Comment 10•24 years ago
|
||
Trudelle, it's the same bug: click on widget, mouse out, mouse in again.
note to alecf: so why did you change it back after trudelle's change?
Comment 11•24 years ago
|
||
Hwaraa, the difference is that, when dragging, you are not mousing out of the
widget. I still see this on Linux today, but no longer on Mac.
Comment 12•24 years ago
|
||
Hmm. delta to drag before hilite goes seems to be about the same as minimal
thumb size. coincidence?
Comment 13•24 years ago
|
||
->evaughan/future. re there any other Linux apps that hilite the thumb like this?
Assignee: trudelle → evaughan
Target Milestone: --- → Future
Comment 14•24 years ago
|
||
I strongly protest futuring this bug. Basic UI fucntionality and a regression no
less.
Comment 15•24 years ago
|
||
NC4.* on IRIX 6.5:
thumb highlights on mouseover - remains on during grab - vanishes when grab is
relased AND mouse is not over thumb. (If cursor is still over thumb when grab is
released, hightlighting is left on untill cursor moves away. This is also how
native IRIX apps behave.
NC4.* on Linux doesn't highlight thumb at any point, I believe.
I agree this shouldn't be futured:
Either the whole feature should be removed - or highlighting should behave in
some logical manner. As it works right now it's clearly bugged and all too
visible. Personally I'd prefer a fix - I liked the highlighting.
Comment 16•24 years ago
|
||
Okay, since this has once again regressed on Mac, where the highlighting is
standard behavior, and also on Win32, I'll pull it back in and hope that we have
time to deal with it. If not, we should consider fixing by removing the
hightlight altogether, which is more correct behavior on Linux & Win9X anyway.
->0.9.2/P4/ALL/ALL
OS: other → All
Priority: -- → P4
Hardware: PC → All
Target Milestone: Future → mozilla0.9.2
Comment 17•24 years ago
|
||
Interesting observation in bug 80027:
When you grab the thumb with MIDDLE mouse button, hightlighting is "correct". I
don't think it should highlight then at all, but is it possible this is all
caused by mouse-buttons getting mixed up in scrollbar code somewhere?
Comment 18•24 years ago
|
||
*** Bug 80027 has been marked as a duplicate of this bug. ***
Comment 19•24 years ago
|
||
It should hightlight also when middle-button grabbing. Forget comment above.
Comment 20•24 years ago
|
||
> Hmm. delta to drag before hilite goes seems to be about the same as minimal
> thumb size. coincidence?
I'd guess so, yes, a coincidence based on how fast you normally drag. On the Mac
at least (2001051109, Mac OS 9.1, Classic theme), delta to drag before highlight
goes isn't measured in pixels -- it's measured in tenths of a second.
FWIW, I vote for retaining the highlight, even though it's buggy. By the time
the highlight disappears, the user will often not be looking at the scrollbar
any more, so they may not even notice.
I filed bug 80462 for the separate issue of widgets not regaining mousedown
appearance. I *think* that other bug is also happening to scrollbar thumbs, but
it's a bit hard to tell while this bug is also present.
Comment 21•24 years ago
|
||
*** This bug has been marked as a duplicate of 80462 ***
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
Comment 22•24 years ago
|
||
I don't think this is a dup. The highlight was lost even if I didn't move the
mouse at all, just held it on the same pixel for 0.5 seconds. But I'll have to
download a new build to be sure ...
Comment 23•24 years ago
|
||
Hmm, okay. I can't get my scrollbar to change shades at all, so it's hard to
test.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 24•24 years ago
|
||
MPT: I was trying to indicate that the highlight disappeared for me based only
on distance moved, not time or speed, and that distance was about the size of
the minimal thumb. I think that the distraction of a disappearing highlight
more than makes up for any purported benefit, but don't really feel strongly
either way. Looks like the highlight has been removed recently anyway.
Comment 25•24 years ago
|
||
on mac, at least, mpt is correct and that the hilight goes away after a set
amount of time, regardless of distance.
it appears that hilight has been removed from modern, but probably not from classic.
Comment 26•24 years ago
|
||
->0.9.3/helpwanted. Possible limbo candidate if an easy fix appears, but I
wouldn't want my enginers spending any signficant time on it in this release
cycle.
Keywords: helpwanted
Target Milestone: mozilla0.9.2 → mozilla0.9.3
Comment 27•23 years ago
|
||
The highglight works perfectly now (build 2001062105, Mac OS 9.1), but only if
you drag the topmost/bottommost (black) pixel of the vertical scrollbar thumb,
or the leftmost/rightmost (black) pixel of the horizontal scrollbar thumb. The
highglight doesn't happen if you drag any other part of the thumb.
So, my guess is that the highlight was turned off incompletely when it wasn't
working properly, but now it's working again, and it needs to be turned back on.
Comment 28•23 years ago
|
||
Doesn't look like this is getting fixed before the freeze tomorrow night.
Pushing out a milestone. Please correct if I'm mistaken.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Comment 29•23 years ago
|
||
I think we can live without this for 0.9.4. let me know if this is not right.
Target Milestone: mozilla0.9.4 → mozilla0.9.5
Updated•23 years ago
|
Status: REOPENED → ASSIGNED
Target Milestone: mozilla0.9.6 → mozilla0.9.7
Updated•23 years ago
|
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Comment 31•23 years ago
|
||
This worksforme in build 2001122008, Mac OS 9.1, Classic theme -- I guess it was
fixed by Hewitt's Classic rewrite. It's still not fixed in Modern.
Assignee | ||
Comment 32•23 years ago
|
||
Ok, so everything works as it should except for Modern. (Mac Classic works,
Windows Classic never did the highlight thing; so it works and Modern is still
not working.)
Assignee | ||
Comment 33•23 years ago
|
||
Comment 34•23 years ago
|
||
Comment on attachment 62572 [details] [diff] [review]
Fix modern
r=andreww
Attachment #62572 -
Flags: review+
Comment 35•23 years ago
|
||
Comment on attachment 62572 [details] [diff] [review]
Fix modern
sr=hewitt
Attachment #62572 -
Flags: superreview+
Assignee | ||
Comment 36•23 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago → 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•