Closed Bug 397846 Opened 18 years ago Closed 18 years ago

Remove "Navigator" branding from SeaMonkey UI

Categories

(SeaMonkey :: General, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0a1

People

(Reporter: kairo, Assigned: kairo)

Details

Attachments

(3 files)

We still have the browser called "Navigator" in some parts of UI, though people usually refer to it as "browser". Additionally, "Navigator" was actually Netscape branding and our branding should be "SeaMonkey", so just as MailNews is called "SeaMonkey Mail & Newsgroups", we should call the browser the "SeaMonkey browser", IMHO.
Here's the main (most visible) part of this patch, i.e. changing the occurrences in suite/locales - with the exception of help, which is probably a bit bigger patch-wise, and which I'd like to do a separate patch for.
Attachment #282836 - Flags: superreview?(neil)
Attachment #282836 - Flags: review?(neil)
Comment on attachment 282836 [details] [diff] [review] Part 1: suite/locales, except help (checked in) Is it worth renaming the entities as well?
Attachment #282836 - Flags: superreview?(neil)
Attachment #282836 - Flags: superreview+
Attachment #282836 - Flags: review?(neil)
Attachment #282836 - Flags: review+
The renaming of entities due to semantic changes is only needed after we have source L10n (bug 286110) so that localizers are forced to take action on their side as well and get those strings corrected. As we only have one locale in a testing mode with this approach yet, and that is my de one, we don't need to do such a change right now - unless you want to get "navigator" out of the entity names...
Comment on attachment 282836 [details] [diff] [review] Part 1: suite/locales, except help (checked in) This part is now checked in.
Attachment #282836 - Attachment description: Part 1: suite/locales, except help → Part 1: suite/locales, except help (checked in)
Here's the help part of this change - I hope I did this correctly :)
Attachment #283005 - Flags: review?
Attachment #283005 - Flags: review? → review?(stefanh)
...and here's the patch for the very few strings in editor we need to change.
Attachment #283009 - Flags: superreview?(neil)
Attachment #283009 - Flags: review?(neil)
Attachment #283005 - Flags: superreview?(neil)
Attachment #283009 - Flags: superreview?(neil)
Attachment #283009 - Flags: superreview+
Attachment #283009 - Flags: review?(neil)
Attachment #283009 - Flags: review+
Comment on attachment 283005 [details] [diff] [review] Part 2: help changes > <li>For <q>HTTP address of your home page</q>, enter the complete URL >- that you would enter in the Navigator browser to view pages at this >+ that you would enter in the browser to view pages at this > site. Do not include a filename or subdirectory as part of the URL. I thought we normally rewrapped paragraphs, although I see from other paragraphs that this hasn't always been done when text is removed. >@@ -1624,15 +1629,15 @@ > <rdf:li> > <rdf:Description ID="preferences:languages" > nc:name="languages preferences" > nc:link="cs_nav_prefs_navigator.xhtml#languages"/> > </rdf:li> > <rdf:li> > <rdf:Description ID="preferences:Navigator" >- nc:name="Navigator preferences" >+ nc:name="Browser preferences" > nc:link="cs_nav_prefs_navigator.xhtml#navigator_preferences"/> > </rdf:li> > <rdf:li> > <rdf:Description ID="preferences:passwords" > nc:name="passwords preferences" > nc:link="passwords_help.xhtml#passwords"/> > </rdf:li> Now that our Index panel has been unceremoniously dropped I guess it doesn't make a lot of difference what you do here, but these and some other entries are no longer in alphabetical order.
Attachment #283005 - Flags: superreview?(neil) → superreview+
Comment on attachment 283009 [details] [diff] [review] Part 3: editor/ui/locales (checked in) Checked in that part as well.
Attachment #283009 - Attachment description: Part 3: editor/ui/locales → Part 3: editor/ui/locales (checked in)
Comment on attachment 283005 [details] [diff] [review] Part 2: help changes FYI, I wont have the time to look at this until the weekend.
Comment on attachment 283005 [details] [diff] [review] Part 2: help changes <ol> <li>In a Composer window, open the File menu, and choose Recent Pages, then select the document from the list. <p>Alternatively, browse to the location of the document you want to update by entering the document&apos;s HTTP - address (the document&apos;s web address) in the Navigator browser&apos;s + address (the document&apos;s web address) in the browser&apos;s Location field.</p></li> Please change "Location field" to "Location Bar". <p>Alternatively, browse to the location of the document you want to update by entering the document&apos;s HTTP address (the document&apos;s web - address) in the Navigator browser&apos;s Location field.</p> + address) in the browser&apos;s Location field.</p> </li> Same here. <p>For example, when you try to locate a published file by typing the - filename&apos;s web address into Navigator&apos;s Location field, you + filename&apos;s web address into the browser&apos;s Location field, you must enter the filename exactly as you created it, using the same combination of uppercase and lowercase letters.</p> Same here. Please also do the same change here : http://lxr.mozilla.org/seamonkey/source/suite/locales/en-US/chrome/common/help/composer_help.xhtml#1452 <p>The History preferences panel allows you to configure three history settings - on Navigator.</p> + on the browser.</p> I'd say "in the browser" or "for the browser". -<p>When you start &brandShortName;, you see Navigator, your browser. +<p>When you start &brandShortName;, you see your browser. A <q>What&apos;s New</q> page appears automatically in the browser window when you first launch &brandShortName;.</p> Please remove the whole thing. We don't need to inform the user that she/he see the browser upon launch (that might also not be true). Also, the "What's New" stuff is an artifact. -<p>To activate any of these Navigator warnings, select the corresponding +<p>To activate any of these browser warnings, select the corresponding checkbox:</p> Just make that "To activate any of these warnings, select the corresponding checkbox:" Index: mozilla/suite/locales/en-US/chrome/common/help/help-index1.rdf =================================================================== RCS file: /cvsroot/mozilla/suite/locales/en-US/chrome/common/help/help-index1.rdf,v retrieving revision 1.56 diff -u -7 -p -r1.56 help-index1.rdf --- mozilla/suite/locales/en-US/chrome/common/help/help-index1.rdf 28 Oct 2006 19:49:00 -0000 1.56 +++ mozilla/suite/locales/en-US/chrome/common/help/help-index1.rdf 1 Oct 2007 13:31:25 -0000 @@ -165,14 +165,64 @@ You forgot to change this: 1285 <rdf:li> 1286 <rdf:Description ID="Navigator" 1287 nc:name="Navigator" 1288 nc:link="nav_help.xhtml"/> 1289 </rdf:li> <nc:subheadings> <rdf:Seq><rdf:li> <rdf:Description ID="browser:changing_your_home_page" nc:name="changing your home page" nc:link="customize_help.xhtml#changing_your_home_page"/> </rdf:li> <rdf:li> + <rdf:Description ID="browser:controlling_popups" + nc:name="controlling popups" + nc:link="cs_priv_prefs_popup.xhtml#controlling_popups"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:Download_Manager" + nc:name="Download Manager" + nc:link="nav_help.xhtml#plugins_and_downloads"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:full_screen_mode" + nc:name="Full screen mode" + nc:link="nav_help.xhtml#full_screen_mode"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:helper_applications" + nc:name="helper applications" + nc:link="cs_nav_prefs_navigator.xhtml#helper_applications"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:home_page" + nc:name="home page" + nc:link="nav_help.xhtml#viewing_your_home_page"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:keyboard_shortcuts" + nc:name="keyboard shortcuts" + nc:link="shortcuts-navigator.xhtml"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:opening_pages" + nc:name="opening pages" + nc:link="nav_help.xhtml#moving_to_another_page"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:preferences" + nc:name="Browser Preferences" + nc:link="cs_nav_prefs_navigator.xhtml#navigator_preferences"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:set_as_wallpaper" + nc:name="set as wallpaper" + nc:link="nav_help.xhtml#setting_image_as_wallpaper"/> + </rdf:li> + <rdf:li> + <rdf:Description ID="browser:start_page" + nc:name="start page" + nc:link="customize_help.xhtml#specifying_a_starting_page"/> + </rdf:li> + <rdf:li> <rdf:Description ID="browser:start_up_components" nc:name="start up components" nc:link="customize_help.xhtml#specifying_which_components_open_at_launch"/> </rdf:li></rdf:Seq> </nc:subheadings> </rdf:Description> @@ -945,15 +995,15 @@ <rdf:li> <rdf:Description ID="keyboard_shortcuts:Mail_and_Newsgroups" nc:name="Mail and Newsgroups keyboard shortcuts" nc:link="shortcuts-mailnews.xhtml"/> </rdf:li> <rdf:li> <rdf:Description ID="keyboard_shortcuts:Navigator" - nc:name="Navigator keyboard shortcuts" + nc:name="Browser keyboard shortcuts" nc:link="shortcuts-navigator.xhtml"/> </rdf:li></rdf:Seq> </nc:subheadings> </rdf:Description> <rdf:Description about="help-indexAZ.rdf#l"> <nc:subheadings> @@ -1299,62 +1349,17 @@ <nc:subheadings> <rdf:Seq><rdf:li> <rdf:Description ID="Navigator:changing_your_home_page" nc:name="changing your home page" nc:link="customize_help.xhtml#changing_your_home_page"/> </rdf:li> This one should also be changed and moved, right? <rdf:li> - <rdf:Description ID="Navigator:controlling_popups" - nc:name="controlling popups" - nc:link="cs_priv_prefs_popup.xhtml#controlling_popups"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:Download_Manager" - nc:name="Download Manager" - nc:link="nav_help.xhtml#plugins_and_downloads"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:full_screen_mode" - nc:name="Full screen mode" - nc:link="nav_help.xhtml#full_screen_mode"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:helper_applications" - nc:name="helper applications" - nc:link="cs_nav_prefs_navigator.xhtml#helper_applications"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:home_page" - nc:name="home page" - nc:link="nav_help.xhtml#viewing_your_home_page"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:keyboard_shortcuts" - nc:name="keyboard shortcuts" - nc:link="shortcuts-navigator.xhtml"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:opening_pages" - nc:name="opening pages" - nc:link="nav_help.xhtml#moving_to_another_page"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:preferences" - nc:name="Navigator Preferences" - nc:link="cs_nav_prefs_navigator.xhtml#navigator_preferences"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:set_as_wallpaper" - nc:name="set as wallpaper" - nc:link="nav_help.xhtml#setting_image_as_wallpaper"/> - </rdf:li> - <rdf:li> - <rdf:Description ID="Navigator:start_page" - nc:name="start page" - nc:link="customize_help.xhtml#specifying_a_starting_page"/> + <rdf:Description ID="Navigator:start_up_components" + nc:name="start up components" + nc:link="customize_help.xhtml#specifying_which_components_open_at_launch"/> This one already exists (as "Browser:start_up_components" ) </rdf:li></rdf:Seq> </nc:subheadings> </rdf:Description> <rdf:Description about="#Newsgroups"> <nc:subheadings> <rdf:Seq><rdf:li> @@ -1624,15 +1629,15 @@ <rdf:li> <rdf:Description ID="preferences:languages" nc:name="languages preferences" nc:link="cs_nav_prefs_navigator.xhtml#languages"/> </rdf:li> <rdf:li> <rdf:Description ID="preferences:Navigator" - nc:name="Navigator preferences" + nc:name="Browser preferences" nc:link="cs_nav_prefs_navigator.xhtml#navigator_preferences"/> </rdf:li> <rdf:li> <rdf:Description ID="preferences:passwords" nc:name="passwords preferences" nc:link="passwords_help.xhtml#passwords"/> </rdf:li> r=me with the mentioned things fixed.
Attachment #283005 - Flags: review?(stefanh) → review+
Checked in the help part with changes requested by stefanh, including the complete removal of the "Navigator" topic from help-index1.rdf, which is apparently still used for searching.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: