Closed
Bug 452175
Opened 16 years ago
Closed 16 years ago
Slice new verbatim design
Categories
(Webtools Graveyard :: Verbatim, defect)
Webtools Graveyard
Verbatim
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rdoherty, Assigned: rdoherty)
References
Details
Attachments
(2 files)
1.09 KB,
image/png
|
Details | |
3.16 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
Tracking bug for new verbatim design to be sliced.
Comment 1•16 years ago
|
||
New design is looking great. As requested from IRC, some changes: 1) Need a .widemodule class that inherits everything from .module but is the full browser width (or 980px if you like). 2) We need a .nav-secondary class for the grey box at the top of pages like http://sm-cms01.mozilla.com:8081/en/languages/fr/. Simple is fine - we can spruce it up later. 3) In the latest firefox nightly the nav-main header is on two lines. Did something break in the nightly or is our design broken? I can screenshot if you like but if it's broken for you you'll know it - it's very obvious. Also adding some dependencies for already filed bugs.
Comment 2•16 years ago
|
||
(In reply to comment #1) > 3) In the latest firefox nightly the nav-main header is on two lines. Did > something break in the nightly or is our design broken? I can screenshot if > you like but if it's broken for you you'll know it - it's very obvious. The header is fine normally but breaks after someone logs in. Sounds like this one is just another view of bug 453247.
Assignee | ||
Comment 3•16 years ago
|
||
Can I get a screenshot of the broken nav? Or an account on a dev instance with the new design so I can test. Also, in order for me to create a module with 100% width, I'll need a new psd mockup with wider grey borders. I can't stretch the current ones.
Comment 4•16 years ago
|
||
> Also, in order for me to create a module with 100% width, I'll need a new psd
> mockup with wider grey borders. I can't stretch the current ones.
I'll stretch them
Assignee | ||
Comment 5•16 years ago
|
||
Here's a patch to * add a widemodule (.module-primary) * fix the nav breaking when logged in (add class="logged-in" if logged in and add the classes "logged-in" and "admin" when logged in as admin) * Slightly style the secondary nav so it's not so ugly. This really needs some UED lovin', there's so much crap/options in there it's hard to style. Rename widemod-borders.png to mod-borders-wide.png and put in /img for the wide module to work. I also cleaned up some of the layout to fix some tiny issues with centering.
Attachment #337131 -
Flags: review?(clouserw)
Assignee | ||
Comment 6•16 years ago
|
||
I also noticed all pages have the class "home" on the body tag. That's only for the real homepage, any other page should have no class.
Comment 7•16 years ago
|
||
Thanks, everything up to here is committed. I also did some other tweaks so be sure to svn up.
Updated•16 years ago
|
Attachment #337131 -
Flags: review?(clouserw) → review+
Comment 8•16 years ago
|
||
Thanks so much Ryan! New design has landed
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•