Closed Bug 342695 Opened 15 years ago Closed 15 years ago

Improve SeaMonkey theme appearance on OS/2

Categories

(SeaMonkey :: General, defect)

x86
OS/2
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

Details

(Keywords: fixed-seamonkey1.1a)

Attachments

(3 files, 1 obsolete file)

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.
Attached patch A first draft (against trunk) (obsolete) — Splinter Review
Not ready for review yet, I first have to find out if a few weirdnesses were caused by this.
Attachment #227007 - Attachment is obsolete: true
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.
Attachment #230305 - Flags: superreview?
Attachment #230305 - Flags: review?(mozilla)
Attachment #230305 - Flags: superreview? → superreview?(neil)
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 ;-)
Attachment #230305 - Flags: superreview?(neil) → superreview+
Comment on attachment 230305 [details] [diff] [review]
Bring SeaMonkey OS/2 theme to save level as Firefox

We can't really do an nsNativeThemeOS2 :)
Attachment #230305 - Flags: review?(mozilla) → review+
Thanks for reviewing, patch now checked in on trunk.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
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.
Right, thanks. Removed the "*" in front of os2/menu.css in CVS.
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
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
This fixes the problems for me and also corrects the double autocomplete section in menu.css.
Attachment #232281 - Flags: review?(mozilla)
Comment on attachment 232281 [details] [diff] [review]
minor corrections

r=mkaply
Attachment #232281 - Flags: review?(mozilla) → review+
Additional patch checked in.
Status: REOPENED → RESOLVED
Closed: 15 years ago15 years ago
Resolution: --- → FIXED
Attachment #232281 - Flags: approval-seamonkey1.1a?
Attachment #230305 - Flags: approval-seamonkey1.1a+
Attachment #232281 - Flags: approval-seamonkey1.1a? → approval-seamonkey1.1a+
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.
Keywords: fixed1.8.1
You need to log in before you can comment on or make changes to this bug.