Maniac Vlad Florin (:vladmaniac) Reporter
	Description • 14 years ago

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a6pre) Gecko/20100625 Minefield/3.7a6pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a6pre) Gecko/20100625 Minefield/3.7a6pre

Selected tab in "Page Info" Menu should change color (appear as highlighted on MouseOver() event). 

Reproducible: Always

Steps to Reproduce:
1. Open any web page in the URL bar. (e.g. www.yahoo.com)
2. Navigate Tools → Page Info





Actual Results:  
Menu Items are static for Page Info while active for other Menu's (e.g. Bookmark Manager). No feedback provided for performing button events. 

Expected Results:  
Inspect the Main Menu Icons. When performing button actions e.g. mouse click, mouse over, selection, etc. button should react with feedback (be highlighted)

	Cameron McCormack (:heycam)
	Comment 1 • 14 years ago

As well as not showing a highlight colour when mousing over one of the icons, the currently selected one isn't highlighted either.  (In the screenshot, the selected icon is "Media", and my mouse pointer was over the "Permissions" icon.)

	Andrés Delfino
	Comment 2 • 14 years ago

This is a regression from Firefox 3.6.6.

	Andrés Delfino
	Comment 3 • 14 years ago

Could anyone please add "regression" (with no quotes) to keywords? Can't do that myself.

	Tanner M. Young [:tmyoung] Assignee
	Comment 4 • 14 years ago

I don't know how we broke it, but I will see if I can find out.

	Tanner M. Young [:tmyoung] Assignee
	Comment 5 • 14 years ago

We're missing "#viewGroup > radio[selected="true"]" in our css file, which extensions.css had, but it was removed by Bug 560602 in http://hg.mozilla.org/mozilla-central/rev/2dcd870f9e31.

I'll try to fix this as quickly as possible.

	Tanner M. Young [:tmyoung] Assignee
	Comment 6 • 14 years ago

Is this a problem on any other OS than windows?

	Tanner M. Young [:tmyoung] Assignee
	Comment 7 • 14 years ago

This should fix it on the Windows theme, but if it is broken on other OSes, we will need to fix their skins, too.

	Maniac Vlad Florin (:vladmaniac) Reporter
	Comment 9 • 14 years ago

Hi all, I confirm it's a problem only for the windows themes. I've checked also Mac Snow Leopard(32-b) and Ubuntu 10.04(64-b).

	Daniel Brooks [:db48x]
	Comment 10 • 14 years ago

Comment on attachment 455390 [details] [diff] [review]
Patch v. 1.0

Yea, good catch. r=db48x

	Dão Gottwald [:dao]
	Comment 11 • 14 years ago

Comment on attachment 455390 [details] [diff] [review]
Patch v. 1.0

> #viewGroup > radio {
>   list-style-image: url("chrome://browser/skin/pageInfo.png");
>   -moz-box-orient: vertical;
>   -moz-box-align: center;
>   -moz-appearance: none;
>   padding: 5px 3px 1px 3px;
>+  margin: 0px 1px 0px 1px;

margin: 0 1px;

>+  min-width: 4.5em;
>+}
>+
>+#viewGroup > radio:hover {
>+	background-color: #E0E8F6;
>+	color: black;
>+}

fix indentation

	Tanner M. Young [:tmyoung] Assignee
	Comment 12 • 14 years ago

Fixed.

	Dão Gottwald [:dao]
	Comment 13 • 14 years ago

Comment on attachment 455469 [details] [diff] [review]
Patch v. 1.1

I was going to say this doesn't need another review, but here's another nit:

> #viewGroup > radio {
>   list-style-image: url("chrome://browser/skin/pageInfo.png");
>   -moz-box-orient: vertical;
>   -moz-box-align: center;
>   -moz-appearance: none;
>   padding: 5px 3px 1px 3px;
>+  margin: 0px 1px;

s/0px/0/

	Tanner M. Young [:tmyoung] Assignee
	Comment 14 • 14 years ago

Fixed and ready for landing.

	Ayan Shah[:ashah]
	Comment 15 • 14 years ago

I can reproduce this bug on Mac OS.

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:2.0b1) Gecko/20100630 Firefox/4.0b1

	Tanner M. Young [:tmyoung] Assignee
	Comment 16 • 14 years ago

I am filing a separate bug for the Mac issue, since I don't have a Mac development environment; I can generate patches, but I can't test them (that I'm aware).  So I am limiting this bug to Windows Page Info issues regressed by the removal of the old extensions manager.

	Tanner M. Young [:tmyoung] Assignee
	Comment 17 • 14 years ago

I have filed Bug 576586 for Mac css issues.

	Dão Gottwald [:dao]
	Comment 18 • 14 years ago

http://hg.mozilla.org/mozilla-central/rev/49dec7e8ec3f