TPAC: saner layout needed

Bug #878264 reported by Lebbeous Fogle-Weekley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Thomas Berezansky

Bug Description

So we used a pretty skin as the foundation for getting the TPAC started, and that's great, but now we have very specific CSS using fixed widths and absolute positioning and other techniques that make adding, removing, and reordering components unduly difficult.

If we want to add something to the left side of an interface, it should be easy. There's no reason that it should push other content off the white-background part of the interface into some wasted grey margin area on the right.

That's the kind of problem we don't need to have, so this bug will serve as a place where we can discuss commits that progress us towards more manageable css and layout.

Also as part of this, we still have tons of inline styles to replace with semantic classes. It's just a lot of grunt work, but we can get it done.

Revision history for this message
Dan Scott (denials) wrote :

Join in the working branch at collab/dbs/tpac-non-fixed-width to help address some of these issues.

Changed in evergreen:
milestone: 2.2.0alpha1 → 2.2.0alpha2
Dan Scott (denials)
tags: added: pullrequest
Revision history for this message
Dan Scott (denials) wrote :

I'd like to see this branch merged, as I have made a number of commits that will likely end up causing conflict with other more functional branches. "git log --first-parent" will show approximately 15 commits to this branch that provide an evolution towards a saner, non-fixed width layout.

There is certainly still plenty to be done, but the potential for duplicated effort & conflicts grows the longer that this stays out of master.

Dan Scott (denials)
Changed in evergreen:
assignee: Lebbeous Fogle-Weekley (lebbeous) → nobody
Changed in evergreen:
assignee: nobody → Thomas Berezansky (tsbere)
Revision history for this message
Thomas Berezansky (tsbere) wrote :

Looks good in a quick battery of testing. Pushed!

Changed in evergreen:
status: New → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.