Richard Zach Reporter
	Description • 25 years ago

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).

	leger
	Comment 1 • 25 years ago

Bulk move of all Pref UI component bugs to new Preferences component.  Pref UI 
component will be deleted.

	Richard Zach Reporter
	Comment 2 • 25 years ago

*** Bug 28681 has been marked as a duplicate of this bug. ***

	Richard Zach Reporter
	Comment 3 • 25 years ago

Can't use font prefs without this.

	Richard Zach Reporter
	Comment 4 • 25 years ago

*** Bug 29480 has been marked as a duplicate of this bug. ***

	Raefer Gabriel
	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).

	sairuh (rarely reading bugmail)
	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.

	Richard Zach 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.

	matt
	Comment 8 • 25 years ago

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.
 

	rickg
	Comment 9 • 25 years ago

marking PDT+, on the proviso that it's fixed by 3/3.

	rods (gone)
	Comment 10 • 25 years ago

waqar, isn't this the same bug as last week?

	waqar
	Comment 11 • 25 years ago

*** Bug 29223 has been marked as a duplicate of this bug. ***

	waqar
	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.

	Jim Roskind
	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

	Christopher Blizzard (:blizzard) 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.

	Phil Peterson
	Comment 15 • 25 years ago

PDT-

	Richard Zach Reporter
	Comment 16 • 25 years ago

*** Bug 29998 has been marked as a duplicate of this bug. ***

	waqar
	Comment 17 • 25 years ago

*** Bug 29978 has been marked as a duplicate of this bug. ***

	Richard Zach Reporter
	Comment 18 • 25 years ago

*** Bug 30538 has been marked as a duplicate of this bug. ***

	Richard Zach Reporter
	Comment 19 • 25 years ago

*** Bug 30665 has been marked as a duplicate of this bug. ***

	Christopher Blizzard (: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.

	rods (gone)
	Comment 21 • 25 years ago

This should be a PDT+ bug....

	Christopher Blizzard (:blizzard) Assignee
	Comment 23 • 25 years ago

Patch attached.  Waiting for a review.

	Richard Zach Reporter
	Comment 24 • 25 years ago

*** Bug 32173 has been marked as a duplicate of this bug. ***

	Christopher Blizzard (:blizzard) Assignee
	Comment 25 • 25 years ago

*** Bug 29142 has been marked as a duplicate of this bug. ***

	Christopher Blizzard (:blizzard) Assignee
	Comment 26 • 25 years ago

*** Bug 29142 has been marked as a duplicate of this bug. ***

	Christopher Blizzard (:blizzard) 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.

	Richard Zach Reporter
	Comment 28 • 25 years ago

*** Bug 32330 has been marked as a duplicate of this bug. ***

	R.K.Aa.
	Comment 29 • 25 years ago

Is this in the nightly builds yet? Still doesn't work. Tested with 2000031806
(nb1b) and 2000031808

	Brian Hunt
	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.

	Christopher Blizzard (:blizzard) Assignee
	Comment 31 • 25 years ago

Maybe I don't have enough fonts installed.  I don't have a scrollbar in my font
list.

	Christopher Blizzard (:blizzard) 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.

	Christopher Blizzard (:blizzard) 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.

	Christopher Blizzard (:blizzard) Assignee
	Comment 35 • 25 years ago

pavlov has reviewed.

	Brian Hunt
	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.

	Christopher Blizzard (:blizzard) Assignee
	Comment 37 • 25 years ago

Checked in.

	John Morrison
	Comment 38 • 25 years ago

qa -> jrgm

	Christopher Blizzard (:blizzard) Assignee
	Comment 39 • 25 years ago

*** Bug 32827 has been marked as a duplicate of this bug. ***

	Jeffrey Baker
	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.

	Christopher Blizzard (:blizzard) 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?

	John Morrison
	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.

	Christopher Blizzard (:blizzard) Assignee
	Comment 43 • 25 years ago

Thanks.  When you open that bug, assign it to me.

	Jeffrey Baker
	Comment 44 • 25 years ago

Yep definitely fixed on the trunk.  Sorry for the confusion, I don't usually use
the nb1b builds.

	R.K.Aa.
	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.

	Christopher Blizzard (:blizzard) Assignee
	Comment 46 • 25 years ago

Please ignore the spam.  Changing address.

	John Morrison
	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).

	John Morrison
	Comment 48 • 25 years ago

vrfy

	waqar
	Comment 49 • 25 years ago

*** Bug 32177 has been marked as a duplicate of this bug. ***

	R.K.Aa.
	Comment 50 • 25 years ago

filed bug 36430