Merge lp:~cruzjbishop/loggerhead/UI-Round-1 into lp:loggerhead

Proposed by Cruz Bishop on 2012-09-21
Status: Merged
Merged at revision: 486
Proposed branch: lp:~cruzjbishop/loggerhead/UI-Round-1
Merge into: lp: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:~cruzjbishop/loggerhead/UI-Round-1
Reviewer Review Type Date Requested Status
Benji York (community) code 2012-09-21 Approve on 2012-09-24
Review via email: mp+125621@code.launchpad.net

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://i.imgur.com/U9mRu.png

2: Make the "file diff" (for lack of a better name) header more tab-like in appearance. http://i.imgur.com/U9mRu.png

3: Give top-level menu entries rounded corners. http://i.imgur.com/YzI8f.png and http://i.imgur.com/8CgbG.png

To post a comment you must log in.
Benji York (benji) wrote :

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.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loggerhead/static/css/diff.css'
2--- loggerhead/static/css/diff.css 2012-03-28 20:45:44 +0000
3+++ loggerhead/static/css/diff.css 2012-09-21 02:32:18 +0000
4@@ -6,7 +6,7 @@
5 color: #666;
6 background: #fcfcfc url(../images/bg_infobox.gif) repeat-y top right;
7 border: 1px solid #b7b7b7;
8- border-radius: 5px;
9+ border-radius: 13px / 7px;
10 }
11 ul#list-files {
12 float: right;
13@@ -39,7 +39,7 @@
14 background: #eef6fa;
15 border: 1px solid #d2d2d2;
16 margin: 10px 0 0 0;
17- border-radius: 5px 5px 5px 0px;
18+ border-radius: 5px 5px 8px 0px / 17px 17px 5px 0px;
19 }
20 .diffBox img {
21 vertical-align: middle;
22
23=== modified file 'loggerhead/static/css/global.css'
24--- loggerhead/static/css/global.css 2011-11-23 17:21:39 +0000
25+++ loggerhead/static/css/global.css 2012-09-21 02:32:18 +0000
26@@ -131,11 +131,13 @@
27 ul#menuTabs li a:hover {
28 background-color: #f3f3f3;
29 text-decoration: none;
30+ border-radius: 5px;
31 }
32 ul#menuTabs li a#on {
33 font-weight: bold;
34 color: #fff;
35 background-color: #747474;
36+ border-radius: 5px;
37 }
38
39 /* Sub-menu Tabs */

Subscribers

People subscribed via source and target branches