Merge lp:~jstpierre/loggerhead/css-changes into lp:loggerhead
Proposed by
Jasper St. Pierre
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | John A Meinel | ||||
Approved revision: | 450 | ||||
Merged at revision: | 448 | ||||
Proposed branch: | lp:~jstpierre/loggerhead/css-changes | ||||
Merge into: | lp:loggerhead | ||||
Diff against target: |
260 lines (+80/-65) 6 files modified
loggerhead/controllers/annotate_ui.py (+29/-13) loggerhead/controllers/view_ui.py (+4/-4) loggerhead/static/css/global.css (+1/-1) loggerhead/static/css/view.css (+24/-20) loggerhead/templates/inventory.pt (+6/-10) loggerhead/templates/view.pt (+16/-17) |
||||
To merge this branch: | bzr merge lp:~jstpierre/loggerhead/css-changes | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John A Meinel | Approve | ||
Review via email: mp+58855@code.launchpad.net |
To post a comment you must log in.
I like the idea of the changes. But I did a quick test and the annotation column on the left 'blew up' and consumed pretty much the whole screen.
I don't think it is really usable in that state. If we can find a way to prevent the annotation information from consuming so much width I think this is ready to land.
Another thing that would be nice, but certainly doesn't block this, is if the columns were split, such that selecting text would not give you the annotations. Certainly doesn't have to be done to get this landed.
But I think it would be a usability regression if we landed this as is, because many pages will push the content you want to see off to the right.