Closed
Bug 380381
Opened 17 years ago
Closed 17 years ago
Winstripe background tabs polish
Categories
(Firefox :: Tabbed Browser, enhancement)
Tracking
()
RESOLVED
FIXED
Firefox 3 alpha8
People
(Reporter: dao, Assigned: dao)
Details
(Keywords: polish)
Attachments
(5 files, 12 obsolete files)
44.78 KB,
image/png
|
Details | |
45.98 KB,
image/png
|
beltzner
:
ui-review+
|
Details |
8.88 KB,
application/zip
|
Details | |
6.65 KB,
patch
|
Details | Diff | Splinter Review | |
42.34 KB,
image/gif
|
Details |
(In reply to bug 350690 comment 85) > (In reply to bug 350690 comment 80) > > (In reply to bug 350690 comment 77) > > > > > I like https://bugzilla.mozilla.org/attachment.cgi?id=237225 more than > > > https://bugzilla.mozilla.org/attachment.cgi?id=237224, as I like having just > > > the selected tab be lighter. > > > > I agree that having the selected tab lighter is good, but not at the expense of > > having the background tabs disappear into the tabstrip. One of the goals of the > > redesign of the tabstrip was to have the background tabs (on both platforms) be > > visually distinct from the tabstrip - I feel they're too transparent in the > > first one. > > From the screenshots I had agreed with Seth, but after testing on Linux I think > I agree with Jay. I actually can't easily tell where my background tabs are or > how many I have with the current patch on this. I then tried testing with the > dialog color behind everything, but I think I screwed up the patch, because it > didn't do what I wanted :). So I'd like to see a patch that puts the dialog > color behind everything; I think that would perhaps be better. I think this needs re-evaluation, as the tabstrip's background images was changed afterwards (bug 350689).
Assignee | ||
Comment 1•17 years ago
|
||
Assignee | ||
Comment 2•17 years ago
|
||
Apparently removing the background color makes background tabs *more* distinct from the tabstrip.
Assignee | ||
Comment 3•17 years ago
|
||
Attachment #264459 -
Flags: review?(mconnor)
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•17 years ago
|
Attachment #264459 -
Attachment is obsolete: true
Attachment #264459 -
Flags: review?(mconnor)
Assignee | ||
Comment 4•17 years ago
|
||
Attachment #264460 -
Flags: review?(mconnor)
Comment 5•17 years ago
|
||
The "screenshots with tabstrip" shot makes me a little anxious, as this change doesn't seem to play quite as nicely there as in your first pair of shots, but that first pair is a definite win. As long as this improves things on all background colors (make sure to test on black/dark backgrounds), I'd say it should go in. Then again, my opinion doesn't count for anything on this one, as I'm not even really contributing anymore, let alone a UI reviewer!
Assignee | ||
Comment 6•17 years ago
|
||
The darker the background the lesser should this change make a difference. (black means no difference at all.)
Assignee | ||
Comment 7•17 years ago
|
||
The original goal was to make background tabs more distinct from the active tab. To this, I add to bring Winstripe a bit closer to what mconnor plans for Pinstripe: http://steelgryphon.com/blog/?p=105
Attachment #264455 -
Attachment is obsolete: true
Attachment #264456 -
Attachment is obsolete: true
Attachment #264458 -
Attachment is obsolete: true
Attachment #264460 -
Attachment is obsolete: true
Attachment #264460 -
Flags: review?(mconnor)
Assignee | ||
Comment 8•17 years ago
|
||
Assignee | ||
Comment 9•17 years ago
|
||
Assignee | ||
Comment 10•17 years ago
|
||
Assignee | ||
Updated•17 years ago
|
Attachment #269591 -
Flags: ui-review?(beltzner)
Assignee | ||
Updated•17 years ago
|
Summary: Remove background color for background tabs → Winstripe background tabs polish
Assignee | ||
Comment 11•17 years ago
|
||
tiny fix for some of the images. not updating the screenshot, as I don't think anybody would see the difference ;)
Attachment #269588 -
Attachment is obsolete: true
Comment 12•17 years ago
|
||
Comment on attachment 269591 [details]
screenshot: after
This is mostly there, but the removal of the outline on background tabs now results in there being too much empty space between background tabs.
I like the darkening and removal of the white outline and the reduction in tab height, but keep the width the same.
Attachment #269591 -
Flags: ui-review?(beltzner) → ui-review-
Assignee | ||
Comment 13•17 years ago
|
||
Attachment #269591 -
Attachment is obsolete: true
Attachment #269612 -
Attachment is obsolete: true
Assignee | ||
Comment 14•17 years ago
|
||
Attachment #276951 -
Flags: ui-review?(beltzner)
Assignee | ||
Comment 15•17 years ago
|
||
Attachment #276952 -
Flags: review?(enndeakin)
Assignee | ||
Updated•17 years ago
|
Attachment #269587 -
Attachment is obsolete: true
Comment 16•17 years ago
|
||
Comment on attachment 276951 [details]
screenshot: after
Nice. Very nice.
Attachment #276951 -
Flags: ui-review?(beltzner) → ui-review+
Updated•17 years ago
|
Attachment #276952 -
Flags: review?(enndeakin) → review+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 17•17 years ago
|
||
would somebody please check this in?
Attachment #276952 -
Attachment is obsolete: true
Assignee | ||
Comment 18•17 years ago
|
||
Attachment #276950 -
Attachment is obsolete: true
Comment 19•17 years ago
|
||
This patch no longer applies cleanly, can you post an updated version please.
Assignee | ||
Comment 20•17 years ago
|
||
Attachment #277871 -
Attachment is obsolete: true
Comment 21•17 years ago
|
||
Checking in browser/themes/winstripe/browser/browser.css; /cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v <-- browser.css new revision: 1.79; previous revision: 1.78 done Checking in browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-end-bkgnd-hover.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-end-bkgnd-hover.png,v <-- alltabs-box-overflow-end-bkgnd-hover.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-end-bkgnd.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-end-bkgnd.png,v <-- alltabs-box-overflow-end-bkgnd.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-start-bkgnd-hover.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-start-bkgnd-hover.png,v <-- alltabs-box-overflow-start-bkgnd-hover.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-start-bkgnd.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/alltabs-box-overflow-start-bkgnd.png,v <-- alltabs-box-overflow-start-bkgnd.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-disabled.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-disabled.png,v <-- tab-arrow-end-bkgnd-disabled.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-enabled.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-enabled.png,v <-- tab-arrow-end-bkgnd-enabled.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-hover.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-end-bkgnd-hover.png,v <-- tab-arrow-end-bkgnd-hover.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-disabled.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-disabled.png,v <-- tab-arrow-start-bkgnd-disabled.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-enabled.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-enabled.png,v <-- tab-arrow-start-bkgnd-enabled.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-hover.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-arrow-start-bkgnd-hover.png,v <-- tab-arrow-start-bkgnd-hover.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-left-bkgnd.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-left-bkgnd.png,v <-- tab-left-bkgnd.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-middle-bkgnd.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-middle-bkgnd.png,v <-- tab-middle-bkgnd.png new revision: 1.3; previous revision: 1.2 done Checking in browser/themes/winstripe/browser/tabbrowser/tab-right-bkgnd.png; /cvsroot/mozilla/browser/themes/winstripe/browser/tabbrowser/tab-right-bkgnd.png,v <-- tab-right-bkgnd.png new revision: 1.3; previous revision: 1.2 done
Assignee | ||
Updated•17 years ago
|
Target Milestone: --- → Firefox 3 M8
Comment 22•17 years ago
|
||
The darker frame is after the polish.
Assignee | ||
Comment 23•17 years ago
|
||
maybe also interesting for others: <stevee> for me it has made it harder to distinguish between inactive tabs and the tabbar <stevee> and also the tab dropdown is harder to notice? <dao> well, we have only one color to play with (-moz-dialog) <dao> if you extend the contrast between the active tab and background tabs, that lowers the contrast between background tabs and the tab strip <dao> that's not ideal, but not particularly a usability problem IMHO <stevee> hmm.. just feels like it's been dulled more than 'polished' for me on win2k <dao> it's not literally polished, just revised
You need to log in
before you can comment on or make changes to this bug.
Description
•