| Message ID | 1440440620-25937-10-git-send-email-damien.lespiau@intel.com |
|---|---|
| State | Changes Requested |
| Headers | show |
Maybe you do this in a latter patch, but you could probably drop the 'Project: NAME' part from the breadcrumbs bar, as it's appears to be a duplicate of what's in the title bar. Seeing as this commit is stylistic only though:
Acked-by: Stephen Finucane <stephen.finucane@intel.com>
On Wed, Sep 09, 2015 at 03:33:53PM +0100, Finucane, Stephen wrote: > Maybe you do this in a latter patch, but you could probably drop the 'Project: NAME' part from the breadcrumbs bar, as it's appears to be a duplicate of what's in the title bar. Seeing as this commit is stylistic only though: > > Acked-by: Stephen Finucane <stephen.finucane@intel.com> Funny you say that, removing the whole breadcrumbs bar is actually part of the plan, because that bar duplicates information found elsewhere and/or pages reachable with the back button. That's a larger change though and I wanted to be conservative.
diff --git a/htdocs/css/style.css b/htdocs/css/style.css index 2d8a6b2..3cf4b95 100644 --- a/htdocs/css/style.css +++ b/htdocs/css/style.css @@ -41,14 +41,9 @@ a:hover { #nav { float: left width: 100%; - background: #e8e8e8; - border-bottom: 0.2em solid #786fb4; - font-size: 90%; - padding: 0.2em 0.5em; -} - -#nav a { - text-decoration: underline; + border-bottom: 0.1em solid #ddd; + padding: 0.2em 1em; + color: #a0a0a0; } #navleft {
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> --- htdocs/css/style.css | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-)