Profile/user menu on NTP should look more clickable?

chrome/browser/resources/ntp4/new_tab.css

 /* Copyright (c) 2011 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 html {
   font-family: segoe ui, arial, helvetica, sans-serif;
   font-size: 14px;
   /* It's necessary to put this here instead of in body in order to get the
      background-size of 100% to work properly */
@@ skipping 166 matching lines @@
   cursor: pointer;
   font-size: 13px;
   /* Leave room for the scrollbar. */
   margin-left: 13px;
   margin-right: 13px;
   margin-top: 5px;
   padding: 0;
   position: fixed;
   right: 0;
   text-align: right;
   text-decoration: none;

[Evan Stade, 2011/12/06 20:21:39]

i dont think you need this any more?

[Tyler Breisacher (Chromium), 2011/12/06 22:49:59]

Good point. In fact, as far as I can tell we never needed it.

   top: 0;
   z-index: 100;
   -webkit-box-shadow: none;
 }
 
+#login-container:hover {
+  text-decoration: underline;

[sail, 2011/12/06 19:02:42]

Currently we use the login-container to show two different things:
  #1 To show the email address of the logged in user
  #2 If no user is logged in then we use it to prompt the user to login:
        Not signed in to Chrome
        (You're missing out -- sign in)

I think we only want to put the underline in case #1.

Doing this should be fairly simple.  Just add a new class to the email address
in this code:
http://codesearch.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/ui/web...

See this for a good example on how to do that:
http://codesearch.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/ui/web...

[Tyler Breisacher (Chromium), 2011/12/06 19:33:22]

for #1, I'm seeing my email address on my installed Chrome (16.0.912.59 beta)
but my name on Chromium built from source (17.0.964.0 (Developer Build
113232-dirty Linux)).
for #2, I'm seeing that on Chrome but not seeing anything on Chromium. Is this
actually a Chrome/Chromium difference, or do I need to change a preference to
make it show up the way you're describing?

[sail, 2011/12/06 19:41:43]

Yea, this is a new change on trunk that uses your full name from your Google
Account. Making that underlined on hover still seems like a good idea (though
definitely run it by Jeff).
We enable/disable this using a server side check. To force it to show just
change this else if statement:
     
http://codesearch.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/ui/web...
to just be true.

+}
+
 html[dir='rtl'] #login-container {
   left: 0;
   right: auto;
 }
 
 .login-status-icon {
   background-position: right center;
   background-repeat: no-repeat;
   min-height: 27px;
   -webkit-padding-end: 37px;
@@ skipping 133 matching lines @@
   padding: 0;
   position: absolute;
   margin: 0;
   max-width: 150px;
   min-width: 90px;
   outline: none;
   top: 0;
   z-index: 5;
   -webkit-transition: width 150ms, right 150ms, background-color 150ms;
 }