A couple of NNTP bug fixes

Make action buttons listen to the enter key.

Hide menus when the window is blurred or the user presses Alt or Meta.

Only show tooltip for the title of the thumbnail.

Allow mouse over on the window menu.

Made the windows opaque but with a hsla background color to make it more readable.

Quote all background image URLs so that we can display icons for URLs with spaces in them.

Call preventDefault when using the keyboard to navigate the options menu so that the page does not scroll.

BUG=15268, 15503, 15715, 15769, 15411, 15458
TEST=See bug descriptions


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19884

[Glen Murphy, 2009-07-03 01:51:01]

LG, nits.

http://codereview.chromium.org/149163/diff/1/3
File chrome/browser/resources/new_new_tab.html (right):

http://codereview.chromium.org/149163/diff/1/3#newcode233
Line 233: .style.backgroundImage:'url("chrome://favicon/' + url +
80 col (only if you care about it)

http://codereview.chromium.org/149163/diff/1/3#newcode304
Line 304: jsvalues=".style.backgroundImage:'url("chrome://favicon/' + url +
'")';
80 col (only if you care about it)

http://codereview.chromium.org/149163/diff/1/4
File chrome/browser/resources/new_new_tab.js (right):

http://codereview.chromium.org/149163/diff/1/4#newcode959
Line 959: var windowMenu = {
Add short description

http://codereview.chromium.org/149163/diff/1/4#newcode979
Line 979: //window.addEventListener('blur', this.boundHide_);
commented out code necessary?

http://codereview.chromium.org/149163/diff/1/4#newcode988
Line 988: //window.removeEventListener('blur', self.boundHide_);
commented out code necessary?

[arv (Not doing code reviews), 2009-07-03 02:04:16]

http://codereview.chromium.org/149163/diff/1/3
File chrome/browser/resources/new_new_tab.html (right):

http://codereview.chromium.org/149163/diff/1/3#newcode233
Line 233: .style.backgroundImage:'url("chrome://favicon/' + url +
On 2009/07/03 01:51:01, Glen Murphy wrote:
> 80 col (only if you care about it)

Done.

http://codereview.chromium.org/149163/diff/1/3#newcode304
Line 304: jsvalues=".style.backgroundImage:'url("chrome://favicon/' + url +
'")';
On 2009/07/03 01:51:01, Glen Murphy wrote:
> 80 col (only if you care about it)

Done.

http://codereview.chromium.org/149163/diff/1/4
File chrome/browser/resources/new_new_tab.js (right):

http://codereview.chromium.org/149163/diff/1/4#newcode959
Line 959: var windowMenu = {
On 2009/07/03 01:51:01, Glen Murphy wrote:
> Add short description

Done.

Changed to use a constructor and moved the init code into it.

http://codereview.chromium.org/149163/diff/1/4#newcode979
Line 979: //window.addEventListener('blur', this.boundHide_);
On 2009/07/03 01:51:01, Glen Murphy wrote:
> commented out code necessary?

Done.

http://codereview.chromium.org/149163/diff/1/4#newcode988
Line 988: //window.removeEventListener('blur', self.boundHide_);
On 2009/07/03 01:51:01, Glen Murphy wrote:
> commented out code necessary?

Done.