The default bug view has changed. See this FAQ.

[Mac default] adopt editBMPanel to new light coloring

RESOLVED FIXED in seamonkey2.15

Status

SeaMonkey
Themes
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Trunk
seamonkey2.15
x86
Mac OS X

SeaMonkey Tracking Flags

(seamonkey2.14 fixed)

Details

Attachments

(2 attachments, 4 obsolete attachments)

(Assignee)

Description

5 years ago
Created attachment 653110 [details] [diff] [review]
WIP patch

Bug 771284 will change the way arrow panels look (dark --> light). We need to adjust the css for our editBMPanel in order to not make it look broken.

Attaching a WIP patch.
(Assignee)

Comment 1

5 years ago
Note to self:
* Fix colors (fff --> FFFFFF etc)
* Double-check focus border
(Assignee)

Comment 2

5 years ago
Created attachment 657011 [details] [diff] [review]
Fixed colors
Attachment #653110 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 3

5 years ago
Created attachment 657698 [details] [diff] [review]
Almost there...

panel hack will be removed in final patch...
Attachment #657011 - Attachment is obsolete: true
(Assignee)

Comment 4

5 years ago
2.14 will need this, since the panel change was made for mozilla 17.
status-seamonkey2.14: --- → affected
(Assignee)

Updated

5 years ago
Target Milestone: --- → seamonkey2.15
(Assignee)

Comment 5

5 years ago
Created attachment 660885 [details] [diff] [review]
Final version

Some polish and clean-up, compared to the previous patch.
Attachment #657698 - Attachment is obsolete: true
Attachment #660885 - Flags: review?(mnyromyr)

Comment 6

5 years ago
Comment on attachment 660885 [details] [diff] [review]
Final version

The light colour scheme is definitely an improvement — I never liked the dark stuff anyway. ;-)

>+#editBMPanel_rows > row > textbox[focused="true"],
>+#editBMPanel_rows > row > hbox > textbox[focused="true"] {
>+  border-color: -moz-mac-focusring !important;
>+  /*box-shadow: 0 0 1px -moz-mac-focusring inset,
>+              0 0 4px 1px -moz-mac-focusring,
>+              0 0 1.5px 1px -moz-mac-focusring; */
>+}

The focus rect around the "name" and "tags" fields does not match the one around all other elements, though, which looks very odd. I assume you just forgot to remove the /* */ above and it wasn't on purpose … ;-)

Plus, I wonder why you made the three buttons at the bottom that oblong? They look different now than almost everywhere else, not like "real" buttons …
Attachment #660885 - Flags: review?(mnyromyr) → review-
(Assignee)

Comment 7

5 years ago
(In reply to Karsten Düsterloh from comment #6)
> Comment on attachment 660885 [details] [diff] [review]
> Final version
> 
> The light colour scheme is definitely an improvement — I never liked the
> dark stuff anyway. ;-)
> 
> >+#editBMPanel_rows > row > textbox[focused="true"],
> >+#editBMPanel_rows > row > hbox > textbox[focused="true"] {
> >+  border-color: -moz-mac-focusring !important;
> >+  /*box-shadow: 0 0 1px -moz-mac-focusring inset,
> >+              0 0 4px 1px -moz-mac-focusring,
> >+              0 0 1.5px 1px -moz-mac-focusring; */
> >+}
> 
> The focus rect around the "name" and "tags" fields does not match the one
> around all other elements, though, which looks very odd. I assume you just
> forgot to remove the /* */ above and it wasn't on purpose … ;-)
>

Ouch, sorry - I commented that out when testing if the !important was needed (yes, it was), then I must have forgotten it.

> Plus, I wonder why you made the three buttons at the bottom that oblong?
> They look different now than almost everywhere else, not like "real" buttons
> …

The min-width of 79px comes from button.css, I guess you could see them as panel "versions" of the normal buttons. Note that they look exactly the same as the ones in the  Firefox version of the panel. I see your point, but I would rather stay in parity with Firefox here (and hasn't Apple's buttons always looked a bit too wide if you don't have enough text in the label?)
(Assignee)

Comment 8

5 years ago
Created attachment 667537 [details] [diff] [review]
New version

See my previous comment.
Attachment #660885 - Attachment is obsolete: true
Attachment #667537 - Flags: review?(mnyromyr)

Comment 9

5 years ago
(In reply to Stefan [:stefanh] from comment #7)
> > Plus, I wonder why you made the three buttons at the bottom that oblong?
> > They look different now than almost everywhere else, not like "real" buttons
> > …
> 
> The min-width of 79px comes from button.css, I guess you could see them as
> panel "versions" of the normal buttons.

I'm not concerned about their length but about their actual _form_. Usually, all(?) our buttons have normal Mac-like form with "half circles" on the left and right, but these don't. 

> Note that they look exactly the same as the ones in the Firefox version 
> of the panel.

Maybe.
I'd bet they didn't care about the form on Mac.

> I see your point

I'm not sure you do. ;-)
(Assignee)

Comment 10

5 years ago
Created attachment 667599 [details]
Save dialog on 10.8

Comment 11

5 years ago
Comment on attachment 667537 [details] [diff] [review]
New version

Ah, well, who am I to blame Apple of inconsistent design …
Attachment #667537 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 12

5 years ago
http://hg.mozilla.org/comm-central/rev/9c30ed8bfa50
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 13

5 years ago
Comment on attachment 667537 [details] [diff] [review]
New version

[Approval Request Comment]
Regression caused by (bug #): Bug 771284
User impact if declined: Odd-looking editBookmarkPanel (mixture of old/new styles)
Testing completed (on m-c, etc.): Nope
Risk to taking this patch (and alternatives if risky): No risk, only theme changes
String changes made by this patch: None
Attachment #667537 - Flags: approval-comm-aurora?

Updated

5 years ago
Attachment #667537 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 14

5 years ago
http://hg.mozilla.org/releases/comm-aurora/rev/ee67eb163cf6
status-seamonkey2.14: affected → fixed
You need to log in before you can comment on or make changes to this bug.