Merge lp://staging/~cruzjbishop/loggerhead/UI-Round-1 into lp://staging/loggerhead
Proposed by
Cruz Bishop
| Status: | Merged |
|---|---|
| Merged at revision: | 486 |
| Proposed branch: | lp://staging/~cruzjbishop/loggerhead/UI-Round-1 |
| Merge into: | lp://staging/loggerhead |
| Diff against target: |
39 lines (+4/-2) 2 files modified
loggerhead/static/css/diff.css (+2/-2) loggerhead/static/css/global.css (+2/-0) |
| To merge this branch: | bzr merge lp://staging/~cruzjbishop/loggerhead/UI-Round-1 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Benji York (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
Introduce some more minor UI changes
Description of the change
This merge request pulls in some minor UI changes to do with rounded corners.
The changes are as follows:
1: Introduce more natural looking borders in the info box. http://
2: Make the "file diff" (for lack of a better name) header more tab-like in appearance. http://
3: Give top-level menu entries rounded corners. http://
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I had a hard time deciding whether or not eliding the vendor-prefixed versions of the rounding directives is acceptable here. I just barely came down on the side of leaving them out, but if me mentioning it spurs you to put them in, I won't complain.