Closed
Bug 26508
Opened 25 years ago
Closed 25 years ago
select in prefs doesn't scroll
Categories
(Core :: XUL, defect, P3)
Tracking
()
VERIFIED
FIXED
M15
People
(Reporter: rzach, Assigned: blizzard)
References
Details
(Keywords: platform-parity, relnote, Whiteboard: [PDT-])
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
The drop down menus in the font pref panel don't scroll. Toreproduce: 1. Open font prefs panel 2. Click on one of the font face select widgets 3. Click in scrollbar Actual result: nothing happens Expected result: font list should scroll Linux build 2000.02.02.09 First thought this was just a problem with the length of the selection list and thus a widget problem. But I tested it with an HTML select field with about 2000 options, and it worked fine (if slowly).
Bulk move of all Pref UI component bugs to new Preferences component. Pref UI component will be deleted.
Component: Pref UI → Preferences
Comment 5•25 years ago
|
||
Thanks for pointing me to the right bug, Zach (it didn't show up in the bugzilla list of Preferences bugs, and I don't know why). This is a real pain for playing around with prefs, as I'm sure nobody wants font adjustment in the beta release to require manual prefs file editing (as it does currently for me).
Comment 6•25 years ago
|
||
is this still a problem for you? perhaps it because i don't have enough fonts installed on either my linux or winNT box, but i don't even see scrollbars on the font droplists... (i do see the scrollbars on the font lists on my mac, and they do scroll.) tested with opt comm bits, 2000022808.
Keywords: pp
Reporter | ||
Comment 7•25 years ago
|
||
Yes, still broken on 2000.02.27.09. When the select menu drops down, I can only select the fonts displayed. Clicking in the scroll bar doesn't have an effect.
This happens for any select in the preferences. Happens on search also. Looks like the scroll bar is there since it is dynamically built. Otherwise i'm not sure how you can get a scrollbar on a drop down list.
Assignee: matt → rods
Summary: Font face selection in font prefs doesn't scroll → select in prefs doesn't scroll
marking PDT+, on the proviso that it's fixed by 3/3.
Whiteboard: [PDT+]
Comment 12•25 years ago
|
||
After talking to pavlov, we came to the conclusion that to fix this would require lots of rewriting of GTK code, and I would be fixed by rod's gfx scroll for select.
Depends on: 18895
Comment 13•25 years ago
|
||
This bug depends on 18895... but that bug is NOT marked PDT+ The last comment acts like this bug is "waiting" for that checkin :-/. I'm going to remove the PDT+ marking on this, and let the PDT either mark both bugs as PDT+, or get this one off the radar. Please correct me if I misunderstood. Thanks, Jim
Whiteboard: [PDT+]
Assignee | ||
Comment 14•25 years ago
|
||
I'm curious about this bug. I'm going to take a look at it. It sounds like a grab problem to me.
Assignee: waqar → blizzard
Assignee | ||
Comment 20•25 years ago
|
||
I know what's causing this and I've been working on a patch. It's a kinda complicated event problem. However, I don't want to waste my time on this since it's been PDT-'ed.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 22•25 years ago
|
||
Assignee | ||
Comment 27•25 years ago
|
||
Fix checked in. I rewrote the superwin event filter to handle grabs better. That code was a sewer and needed to be rewritten and documented anyway.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 29•25 years ago
|
||
Is this in the nightly builds yet? Still doesn't work. Tested with 2000031806 (nb1b) and 2000031808
Comment 30•25 years ago
|
||
You can now select fonts, but build 2000.03.18.08 still does not scroll the font list. This might not be as resolved as it appears.
Assignee | ||
Comment 31•25 years ago
|
||
Maybe I don't have enough fonts installed. I don't have a scrollbar in my font list.
Assignee | ||
Comment 32•25 years ago
|
||
Ok, I installed enough fonts on my system to get the scrollbar and I see the problem now. Re-opening to fix it.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 33•25 years ago
|
||
Assignee | ||
Comment 34•25 years ago
|
||
Another day, another patch. This should fix the problem where there's a GtkWidget ( i.e. a scrollbar ) on a GdkWindow that has had gdk_pointer_grab() called on it. Looking for a review. Pav? It's easy, it's well commented.
Status: REOPENED → ASSIGNED
Comment 36•25 years ago
|
||
The GtkWidget (ie. scrollbar) should also capture the up/down arrow keys, pgup/pgdn, etc, appropriately, which it does not right now, as pressing those keys goes to the next/previous option-set, and consequently crashes the browser.
Assignee | ||
Comment 37•25 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Comment 38•25 years ago
|
||
qa -> jrgm
Component: Preferences → XP Toolkit/Widgets
QA Contact: sairuh → jrgm
Comment 40•25 years ago
|
||
This is marked fixed, but it is demonstrably broken in Build 2000032305 i686-pc-linux-gnu. Scrollbars on pref combo boxes do not respond to mouse input. The lists themselves will drag-scroll down but will not to the same in the other direction. I recommend reopening this bug, unless it simply hasn't propogated to nightlies yet.
Assignee | ||
Comment 41•25 years ago
|
||
The only place that I've got a scrollbar in the prefs dialog is on the fonts selection and it works fine here. Are you seeing it somewhere else?
Comment 42•25 years ago
|
||
Sorry -- should have commented quickly earlier ... this is in the trunk, but not the beta1 branch. The build that jwbaker is using is a beta1 build. This works correctly for me on linux for the trunk build for today (2000032309) There is a problem with responding to key events (including a crash), but given: 1) the native scrollbar is due to go away 2) the exact response to what the scrollbar should do on certain keystrokes is unclear to me (I was looking at what it does for various situations on motif and gtk and it seems app-dependent -- I need to look at this some more). 3) the crash looks only semi-related to how the scrollbar behaves (it looks like the key event bubbles up, and it's the left-hand panel that gets the event and crashes) then I'll marked this as verified for the original bug report, and open a couple of other bugs -- one for the key event handling, and another for the crash.
Status: RESOLVED → VERIFIED
Comment 44•25 years ago
|
||
Yep definitely fixed on the trunk. Sorry for the confusion, I don't usually use the nb1b builds.
Comment 45•25 years ago
|
||
well...no rest for the vicked. There IS still a problem - somewhere. For one, the scrollbar widgets in prefs respond slow - initially. Like there are double up of them - one set that doesn't work - then one that does, when it gets to think about it. Once it responds, it's quick enough. But after the first checking (gtk widget rewrite) that didn't fix the frozen scrollbar after all, a series of messages started to appear in the first nightly build published. Looks like this whenever a page with forms is loaded: -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #2.2 Bailing on width 1905 <= aCachedAvailableSize.width 1905 -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #2.2 Bailing on width 1905 <= aCachedAvailableSize.width 1905 -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #2.2 Bailing on width 1635 <= aCachedAvailableSize.width 1635 -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #2.2 Bailing on width 1635 <= aCachedAvailableSize.width 1635 -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet -------------- #1 Bailing on aCachedAvailableSize.height -1 != kSizeNotSet and so forth - page up and page down in console. Now: after the second - superwin fix - was checked in - i today had a crash. I had two browser windows open, closed one and clicked a "back" in the other - and voila: Document http://bugzilla.mozilla.org/show_bug.cgi?id=27652 loaded successfully Document: Done (9.091 secs) WEBSHELL- = 6 WEBSHELL- = 5 WEBSHELL- = 4 Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' warning: attempted to CreateNative() without a superwin parent Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' warning: attempted to CreateNative() without a superwin parent Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Damn, brother. That's not a superwin. Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Damn, brother. That's not a superwin. Gdk-CRITICAL **: file gdkgc.c: line 288 (gdk_gc_unref): assertion `gc != NULL' failed. Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Gtk-WARNING **: invalid cast from (NULL) pointer to `GtkObject' Document http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED&rep_platform=PC&email1=&emailtype1=substring&emailassigned_to1=1&email2=&emailtype2=substring&emailreporter2=1&bugidtype=include&bug_id=&changedin=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&short_desc=border&short_desc_type=substring&long_desc=&long_desc_type=substring&bug_file_loc=&bug_file_loc_type=substring&status_whiteboard=&status_whiteboard_type=substring&keywords=&keywords_type=anywords&field0-0-0=noop&type0-0-0=noop&value0-0-0=&cmdtype=doit&newqueryname=&order=Reuse+same+sort+as+last+time##1 loaded successfully Document: Done (9.658 secs) Gdk-CRITICAL **: file gdkwindow.c: line 1707 (gdk_window_get_parent): assertion `window != NULL' failed. Gdk-CRITICAL **: file gdkwindow.c: line 1707 (gdk_window_get_parent): assertion `window != NULL' failed. [dark@somewhere dark]$ These things are related. It's the first time i managed to achieve a crash though so the risk is minimal - but something is rotten in the state of scrollbars. There's a chance the fix needs a fix.
Assignee | ||
Comment 46•25 years ago
|
||
Please ignore the spam. Changing address.
Assignee: blizzard → blizzard
Status: VERIFIED → NEW
Comment 47•25 years ago
|
||
remarking FIXED. dark@c2i.net : if you are still having the other problems you noted with scrollbars, please file a new bug (limit of one defect per bug report :-]) By the way, the "Bailing ..." messages are actually good news (avoiding unnecessary work).
Status: NEW → RESOLVED
Closed: 25 years ago → 25 years ago
You need to log in
before you can comment on or make changes to this bug.
Description
•