188927 - [modern, mac] redundant resize widget in status bar

Status: VERIFIED FIXED

Description

[sairuh (rarely reading bugmail)]

using the modern theme in OS X (either CFM or mach-o) there are two resize
widgets in the status bar: one underneath the vertical scrollbar, and another to
the left of it. there should be only one.

not a problem with the aqua/classic theme. not a problem on win2k or linux.

screenshot coming up.

Comment 1

[sairuh (rarely reading bugmail)]

Created attachment 111411 [details]
screenshot of redundant window resize widgets

this is only a problem on mac with the modern theme.

Comment 2

[Patty Mac]

I also saw that as well on Mac os x (netscape trunk build: 2002-01-10-08-TRUNK),
modern theme.

Comment 3

[Boris Zbarsky [:bzbarsky, bz on IRC]]

See bug 184169

Comment 4

[Simon Fraser]

Yes, this makes us look stupid. Another issue is that we now have a special
resize cursor that only shows up for the left-most resize widget. No other Mac
OS X app uses a resize cursor.

Comment 5

[Boris Zbarsky [:bzbarsky, bz on IRC]]

So..  Are there mac-specific theme files in Modern?  Could we put a rule in
those setting the resizer back to "-moz-appearance: resizer", which is stubbed
out on the mac?

More to the point, perhaps the entire widget should not be there on the mac....

Comment 6

[Simon Fraser]

Can we fix this? It really makes Mac builds look stupid, and I think needs to be
fixed for 1.3.

Comment 7

[Simon Fraser]

Why this is evil:
1. Two resize widgets show up in the corner of the window (see screenshot)
2. For the "XP" resize widget, we show a special resize cursor.
3. (Most evil) Because of other bugs, you can move the mouse outside of the
   window, but the resize cursor sticks, thus giving incorrect visual feedback.
   This makes it easy to click, and switch to another app rather than resizing
   the window. This has me cursing several times per day.

Comment 8

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (away December 22 - January 13) (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Created attachment 113494 [details] [diff] [review]
patch

I tested this in unix/platformXUL.css on my Linux box, and it got rid of the
resizer widget.  Could someone test it on Mac?

Comment 9

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (away December 22 - January 13) (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Taking bug.

Comment 10

[Simon Fraser]

Comment on attachment 113494 [details] [diff] [review]
patch

Works for me; tested Modern and Classic. Thanks dbaron!

Comment 11

[jag (Peter Annema)]

Comment on attachment 113494 [details] [diff] [review]
patch

r=jag

Comment 12

[Asa Dotzler [:asa]]

Comment on attachment 113494 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.3beta.

Comment 13

[David Baron :dbaron: 🏴󠁵󠁳󠁣󠁡󠁿 ⌚UTC-8 (away December 22 - January 13) (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)]

Fix checked in to trunk, 2003-02-04 19:00 PST.

Comment 14

[Grace Bush]

verified on Mac 2003040703