Closed Bug 588191 Opened 14 years ago Closed 3 years ago

Replace term "Personal Toolbar" by "Bookmarks Toolbar" in Help

Categories

(SeaMonkey :: Help Documentation, defect)

defect
Not set
normal

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
seamonkey 2.83
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: InvisibleSmiley, Assigned: njsg)

References

Details

(Whiteboard: SM2.53.6)

Attachments

(1 file, 2 obsolete files)

The exact same bookmark folder is both called "Bookmarks Toolbar" (Bookmarks Manager, Sidebar) and "Personal Toolbar" (Bookmarks menu, Bookmarks toolbar button menu). This is confusing; at least one of them should be renamed so that they match. Help needs to be adapted then, too.

I'd go for "Bookmarks Toolbar".

Fun fact: Choose Properties on the "Personal Toolbar" folder. It shows "Bookmarks Menu" (same for Unsorted Bookmarks).
True, we currently have some fun mashup of "Personal Toolbar" values copied from old suite versions and "Bookmarks Toolbar" copied from Firefox.

I think that the FF choice of "Bookmarks Toolbar" is quite logical, and probably more logical than just "Personal Toolbar" as it's really just bookmarks, but maybe we should at least for now go with a compromise and call it "Personal Bookmarks" or "Personal Bookmarks Toolbar", depending on the available widths for string lengths.
Since this bug is not about renaming the Personal Toolbar itself but the special bookmarks folder, "Personal Toolbar Bookmarks" might be best (cf. "Unsorted Bookmarks"). I think "Personal Bookmarks" misses the relation to the Personal Toolbar, it sounds like "my bookmarks" (and all bookmarks are mine, right?).
Since the main work will be done in bug 588807 now, I'm transforming this into the bug for the Help update.
Component: Bookmarks & History → Help
Depends on: 588807
QA Contact: bookmarks → help
Severity: minor → normal
Summary: Use a common term for Bookmarks Toolbar and Personal Toolbar bookmark folders → Replace term "Personal Toolbar" by "Bookmarks Toolbar" in Help
Hello, I'm an absolute beginner and would like to work on this.

First version of a patch to rename the personal toolbar in help. I've also tried to update the surrounding text on bookmarks. (This should address bug 1288184 as well.)

Assignee: nobody → nunojsg
Status: NEW → ASSIGNED
Attachment #9192193 - Flags: review?(iann_bugzilla)

Comment on attachment 9192193 [details] [diff] [review]
588191-help-rename-bookmarks-toolbar

Missing </li> in suite/locales/en-US/chrome/common/help/customize_help.xhtml

  • <li>Properties for the currently selected bookmark will appear in the bottom
  • part of the window.

suite/locales/en-US/chrome/common/help/help-index1.rdf:

  •     nc:name="personal toolbar"
    
  •     nc:name="bookmarks toolbar"
    

If you search for bookmarks toolbar this is the only location in small caps. Would change it to Bookmarks Toolbar.

f+ for the moment.

Attachment #9192193 - Flags: review?(iann_bugzilla) → feedback+

A new version, with one more closing tag than the previous one (also capitalizes the help-index1.rdf entry).

Attachment #9192193 - Attachment is obsolete: true
Attachment #9192906 - Flags: feedback?(frgrahl)

Comment on attachment 9192906 [details] [diff] [review]
588191-help-rename-bookmarks-toolbar

"you can select <q>Bookmark this group of tabs</q>"

Capitalization was and is wrong here.

  • accessed using the Sidebar Bookmarks tab, the Library window, and"

Shouldn't this be "...window or" ?

Otherwise fine for me and r/a+ implied. IanN should take a final look for grammar errors.

Attachment #9192906 - Flags: feedback?(frgrahl) → feedback?(iann_bugzilla)
Comment on attachment 9192906 [details] [diff] [review]
588191-help-rename-bookmarks-toolbar

Other than what frg spotted there is also:

>+++ b/suite/locales/en-US/chrome/common/help/customize_help.xhtml
> <ol>
>   <li>Open the Bookmarks menu and choose Manage Bookmarks.</li>
>+  <li>Select the <q>Bookmarks Toolbar</q> folder.</li>
>+  <li>Choose New Folder from the File menu.</li>
>+  <li> Type a name for your new bookmark folder. By default, the name is
Nit: remove the space before Type
Attachment #9192906 - Flags: feedback?(iann_bugzilla) → feedback+

Fixed version. r/a+ from me.

Attachment #9192906 - Attachment is obsolete: true
Attachment #9193132 - Flags: review+
Attachment #9193132 - Flags: approval-comm-release+
Attachment #9193132 - Flags: approval-comm-esr68+
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Whiteboard: SM2.53.6
Target Milestone: --- → Future
Comment on attachment 9193132 [details] [diff] [review]
588191-help-rename-bookmarks-toolbar-2536.patch

Sorry, only just noticed this whilst doing localisations. For a follow-up patch:

>+++ b/suite/locales/en-US/chrome/common/help/customize_help.xhtml

>-    <li><a href="#personal_toolbar">Personal Toolbar</a></li>
>+    <li><a href="#personal_toolbar">Bookmarks Toolbar</a></li>
Also change href, so #bookmarks_toolbar

>-<h3 id="personal_toolbar">Personal Toolbar</h3>
>+<h3 id="personal_toolbar">Bookmarks Toolbar</h3>
Also change id, so "bookmarks_toolbar"

>-    <td><img src="images/personalbar.png" alt="Personal Toolbar"/></td>
>+    <td><img src="images/personalbar.png" alt="Bookmarks Toolbar"/></td>
Is image still correct? If so will need to be renamed to bookmarksbar.png and link changed.

>-<h4 id="adding_personal_toolbar_bookmarks">Adding Personal Toolbar
>+<h4 id="adding_personal_toolbar_bookmarks">Adding Bookmarks Toolbar
Also change id and any links to it.

>     <a href="#adding_bookmark_folders_to_the_personal_toolbar">Adding Bookmark
Also change href.
>-    Folders to the Personal Toolbar</a>.</li>
>+    Folders to the Bookmarks Toolbar</a>.</li>

> <p id="adding_bookmark_folders_to_the_personal_toolbar"><strong>Adding
Also change id and links to it.

>-        <p>For more information about adding bookmarks to the Personal Toolbar,
>-          see <a href="#adding_personal_toolbar_bookmarks">Adding Personal
>+        <p>For more information about adding bookmarks to the Bookmarks Toolbar,
>+          see <a href="#adding_personal_toolbar_bookmarks">Adding Bookmarks
Also change this href.

>-    <td><img src="images/personalbar.png" alt="Personal Toolbar"/></td>
>+    <td><img src="images/personalbar.png" alt="Bookmarks Toolbar"/></td>
See earlier comment.

>+++ b/suite/locales/en-US/chrome/common/help/glossary.xhtml

>-  bookmark icon in the <a href="#personal_toolbar">Personal Toolbar</a> or
>+  bookmark icon in the <a href="#personal_toolbar">Bookmarks Toolbar</a> or
Also change this href.

>-<dt id="personal_toolbar">Personal Toolbar</dt><dd>The customizable toolbar
>+<dt id="personal_toolbar">Bookmarks Toolbar</dt><dd>The customizable toolbar
Also change this id and hrefs to it.

>+++ b/suite/locales/en-US/chrome/common/help/help-glossary.rdf
>-        <rdf:li> <rdf:Description nc:name="Personal Toolbar" nc:link="glossary.xhtml#personal_toolbar"/> </rdf:li>
>+        <rdf:li> <rdf:Description nc:name="Bookmarks Toolbar" nc:link="glossary.xhtml#personal_toolbar"/> </rdf:li>
Should be moved to the B section as it is alphabetically ordered, plus the link needs to be changed.

>+++ b/suite/locales/en-US/chrome/common/help/help-index1.rdf

>        <rdf:Description ID="toolbar:personal_toolbar"
>-         nc:name="personal toolbar"
>+         nc:name="Bookmarks Toolbar"
>          nc:link="customize_help.xhtml#personal_toolbar"/>
Also ID and link need changing.

>+++ b/suite/locales/en-US/chrome/common/help/suite-toc.rdf
>-        <rdf:li> <rdf:Description ID="cust-personal" nc:name="Personal Toolbar"  nc:link="customize_help.xhtml#personal_toolbar"/> </rdf:li>
>+        <rdf:li> <rdf:Description ID="cust-personal" nc:name="Bookmarks Toolbar"  nc:link="customize_help.xhtml#personal_toolbar"/> </rdf:li>
Also ID and link need changing.
Target Milestone: Future → seamonkey 2.83
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: