342695 - Improve SeaMonkey theme appearance on OS/2

Status: RESOLVED FIXED

(SeaMonkey :: General, defect)

Description

[Peter Weilbacher]

Similar to the work in bug 314843 and bug 336997 for toolkit apps I would like to improve the appearance of the Classic theme for SeaMonkey on OS/2, so that it looks more "native", especially regarding to menus and scrollbars.

My guess is that this is really only relevant for the 1.8 branch but the patches should also go onto trunk.

Comment 1

[Peter Weilbacher]

Attachment: A first draft (against trunk)

Not ready for review yet, I first have to find out if a few weirdnesses were caused by this.

Comment 2

[Peter Weilbacher]

Attachment: Screenshot of the new menu, toolbar, and scrollbar appearance

Comment 3

[Peter Weilbacher]

Attachment: Bring SeaMonkey OS/2 theme to save level as Firefox

OK, this basically achieves what the respective Toolkit patch did after bug 336997. The GIF images are the same as in that bug, so I won't attach them again.  I resorted a few lines about Windows in the jar.mn so that I could do it with two #ifdefs.

Comment 4

[[e-mail]]

Comment on attachment 230305 [details] [diff] [review]
Bring SeaMonkey OS/2 theme to save level as Firefox

I suppose it will do given that nobody's written nsNativeThemeOS2.cpp yet ;-)

Comment 5

[Mike Kaply [:mkaply]]

Comment on attachment 230305 [details] [diff] [review]
Bring SeaMonkey OS/2 theme to save level as Firefox

We can't really do an nsNativeThemeOS2 :)

Comment 6

[Peter Weilbacher]

Thanks for reviewing, patch now checked in on trunk.

Comment 7

[[e-mail]]

Comment on attachment 230305 [details] [diff] [review]
Bring SeaMonkey OS/2 theme to save level as Firefox

>+#ifdef XP_OS2
>+* skin/classic/global/menu.css                                (global/os2/menu.css)
>+  skin/classic/global/menu/menu-check.gif                     (global/os2/menu/menu-check.gif)
>+  skin/classic/global/menu/menu-arrow.gif                     (global/os2/menu/menu-arrow.gif)
>+  skin/classic/global/toolbar.css                             (global/os2/toolbar.css)
>+#else
> * skin/classic/global/menu.css                                (global/win/menu.css)
I didn't spot this until it merge conflicted with another patch I have in my tree, but I don't think the os2 menu.css strictly needs that leading * as it isn't preprocessed.

Comment 8

[Peter Weilbacher]

Right, thanks. Removed the "*" in front of os2/menu.css in CVS.

Comment 9

[Peter Weilbacher]

Users in the newsgroups found some minor issues that I think we should correct before putting this into branch:
- if menu colors are different from grey the colored regions is only behind
  the words on the main menubar with grey gaps in between
- when clicking the arrow on the right of the URL field the background of the
  dropdown is colored in the menu color
- the same is true for the dropdown in the "Print" panel

Comment 10

[Peter Weilbacher]

Attachment: minor corrections

This fixes the problems for me and also corrects the double autocomplete section in menu.css.

Comment 11

[Mike Kaply [:mkaply]]

Comment on attachment 232281 [details] [diff] [review]
minor corrections

r=mkaply

Comment 12

[Peter Weilbacher]

Additional patch checked in.

Comment 13

[Peter Weilbacher]

OK, checked the patches from attachment 230305 [details] [diff] [review] and attachment 232281 [details] [diff] [review] into 1.8 branch, including the typo mentioned in comment 7 and all new GIFs.