Bug 340171 - [NLS] Facet uninstall page should use "plug-in" instead of "plugin" in text
Summary: [NLS] Facet uninstall page should use "plug-in" instead of "plugin" in text
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Libra (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: ---   Edit
Assignee: Kaloyan Raev CLA
QA Contact: Kaloyan Raev CLA
URL:
Whiteboard: PMC
Keywords: PII
Depends on:
Blocks:
 
Reported: 2011-03-16 11:15 EDT by Nitin Dahyabhai CLA
Modified: 2022-02-24 11:24 EST (History)
1 user (show)

See Also:
david_williams: pmc_approved+
kaloyan: pmc_approved? (raghunathan.srinivasan)
kaloyan: pmc_approved? (naci.dai)
kaloyan: pmc_approved? (deboer)
kaloyan: pmc_approved? (neil.hauge)
kaloyan: pmc_approved? (kaloyan)
kaloyan: pmc_approved? (cbridgha)

Attachments
patch (1.12 KB, patch)
2011-04-19 11:37 EDT, Kaloyan Raev CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nitin Dahyabhai CLA 2011-03-16 11:15:55 EDT 
Looking at the osgi_bundle_facet_uninstall_page.png image from the Libra M6 N&N, I see that there's text that has the word "plugin" in it.  The correct form to use in the UI is "plug-in."
Comment 1 Kaloyan Raev CLA 2011-04-19 11:37:08 EDT 
Created attachment 193595 [details]
patch
Comment 2 Kaloyan Raev CLA 2011-04-19 11:41:14 EDT 
    * Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 

This is a small post-M6 NLS change. It's better fix it now, before the first official release of Libra. 

    * Is there a work-around? If so, why do you believe the work-around is insufficient? 

No workaround. 

    * How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? 

Manually tested by me. The change is straight-forward. 

    * Give a brief technical overview. Who has reviewed this fix? 

Nothing technical here - just a small change in string messages.

    * What is the risk associated with this fix? 

No risk.
Comment 3 David Williams CLA 2011-04-19 11:51:43 EDT 
good catch, good to be consistent ... hope we are everywhere else! :)
Comment 4 Kaloyan Raev CLA 2011-04-19 12:06:06 EDT 
Pushed to the master branch in Git.
Comment 5 Kaloyan Raev CLA 2011-08-24 09:57:32 EDT 
Closing