Closed Bug 866081 Opened 7 years ago Closed 7 years ago

about:plugins changing header page (enhancement)

Categories

(Core :: Plug-ins, enhancement, P3)

enhancement

Tracking

()

VERIFIED FIXED
mozilla23

People

(Reporter: tiziana, Assigned: mga)

Details

(Whiteboard: [mentor=bsmedberg][good first bug])

Attachments

(1 file, 1 obsolete file)

1. Open Firefox
2. Go to about:plugins page
3. The header says "Enabled plugins"

Expected Result: 
Given that in that page are listed plugins with state enabled/disabled couldn't be better the header "Installed plugins" or a separation between enabled and disabled plugins?.
Evilpie, wasn't this part of my review comments at https://bugzilla.mozilla.org/show_bug.cgi?id=661961#c8 ?
Priority: -- → P3
Whiteboard: [mentor=bsmedberg][good first bug]
Hi, I would be interested in realizing this enhancement, is the page written in HTML5 + JavaScript? What IRC channel do I need to join in order to talk further about what needs to be done?
I can start tomorrow :)
Assignee: nobody → alastra.mariagrazia
Comment on attachment 742727 [details] [diff] [review]
Change two labels in plugins.properties, replacing "enabled" with "installed"

Somewhere in our localization guidelines it says that when you change the meaning of a string (such as enabledplugins_label in this case) you should change the string name also. I think you should change the name to installedplugins_label (ditto nopluginsareinstalled_label).
Attachment #742727 - Flags: review?(benjamin) → review-
Now I changed also the label names.
Attachment #742727 - Attachment is obsolete: true
Attachment #743704 - Flags: review?(benjamin)
Attachment #743704 - Flags: review?(benjamin) → review+
Thanks for the patch!
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/52669c3bc4a0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
I confirm this is fixed on Latest Nightly on Windows 7 x64, Mac OS 10.8.3 and Ubuntu 12.10:

Build ID:20130501085824
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.