Closed
Bug 377439
Opened 18 years ago
Closed 16 years ago
Switching scrollbar styles in Appearance.prefpane doesn't work right.
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.1b2
People
(Reporter: cbarrett, Assigned: mstange)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
STR:
1. Open a window, navigate to a page with a scrollbar on it (Minefield start page works).
2. Open System Prefs -> Appearance
3. Change your setting for "Place scroll arrows:"
4. Switch back to Minefield. Try scrolling the page, moving the mouse over the scrollbar track, doing things to the scrollbar.
Expected results:
Either the scrollbar switches to the new style immediately and completely, or (not as good, but still not horrible) keeps the old style until you create a new one.
Actual results:
The scrollbar adopts the visual style of the new setting, but functionally, still works the same! Awesome...
This happens because the initalize funtion in the XBL binding for scrollbars only gets run when a scrollbar is created.
Three ways to fix this:
1) Get that code to run again when the metric changes (or poll and check the metric against a saved value and or something...)
2) Get scrollbars to expose what metric value they're using, so we can draw with that. New scrollbars would then use the new style. This might be OK. It'd be better than nothing, and better than option 3, imo.
3) Have the theme stuff check the metric value for scroll styles *once* at startup, and a restart to change it. I'm not that thrilled about it, but it'd certainly solve this problem.
Flags: blocking1.9?
Updated•18 years ago
|
Flags: blocking1.9? → blocking1.9-
Reporter | ||
Comment 1•18 years ago
|
||
(In reply to comment #0)
> 3) Have the theme stuff check the metric value for scroll styles *once* at
> startup, and a restart to change it. I'm not that thrilled about it, but it'd
> certainly solve this problem.
This is probably the quickest and easiest thing to do, I'll get a patch for this eventually.
Keywords: regression
Reporter | ||
Comment 2•17 years ago
|
||
This will be fixed by 384612
Reporter | ||
Comment 3•17 years ago
|
||
bug 384612 that is.
With the patch in bug 384612 this should get better. What should happen is that changes to the system prefs will be fully ignored until firefox is restarted.
Turns out my patch there isn't going to help here that much. We still need to make nsILookAndFeel cache these values at startup or the first time it's called.
No longer depends on: 384612
Reporter | ||
Updated•17 years ago
|
Keywords: helpwanted
Reporter | ||
Updated•17 years ago
|
Assignee: mozcbarrett → joshmoz
Assignee | ||
Updated•16 years ago
|
Assignee: joshmoz → mstange
Status: NEW → ASSIGNED
Hardware: PC → All
Assignee | ||
Comment 6•16 years ago
|
||
The "AppleAquaScrollBarVariantChanged" notification name isn't documented, but Webkit uses it, too: http://trac.webkit.org/changeset/36860
I asked Dave Hyatt if it's safe to use and he said "i don't see why not... if they change it, they'd break webkit".
Attachment #342933 -
Flags: review?(roc)
Attachment #342933 -
Flags: review?(roc) → review+
Assignee | ||
Comment 7•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: helpwanted
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b2
You need to log in
before you can comment on or make changes to this bug.
Description
•