[m-rev.] [for review] Fix some styling of the menu.

Paul Bone paul at bone.id.au
Thu Apr 18 23:23:44 AEST 2013


For review by Peter.

Fix some styling of the menu.

I've had some feedback that the sub menu options don't look/feel like menu
options, the user said that he didn't realize they were clickable.  To try
to fix this I've painted them the same colour as the menu options (this
provide better contrast for my testes too) and made the font size the same.
I've also added 5px of padding on all sides of each submenu's text and
enabled the same hover behaviour as the main menu options.  The latter two
changes I think help the most to show that they're clickable.

My personal testes are that grey on white is horrible.  It provides very
little contrast and I wish that companies such as Google and Apple wouldn't
do it.  And I'd rather we didn't either.

css/common.css:
    As above.
---
 css/common.css | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/css/common.css b/css/common.css
index dbf709d..2d59e66 100644
--- a/css/common.css
+++ b/css/common.css
@@ -127,19 +127,19 @@ margin-bottom: 20px;
 }
 
 ul.submenu li{
-font-size: 15px;
+font-size: 16px;
 background: none;
 border: none;
 font-family: georgia;
-padding: 0px;
+padding: 5px;
 }
 
 ul.submenu li a:link, ul.submenu li a:visited{
-color: #808080;
+color: #006fbc;
 } 
 
 ul.submenu li:hover{
-background: none;
+background-color: #e6e6e6;
 }
 
 ul.submenu a:hover{
-- 
1.8.1.3




More information about the reviews mailing list