Drop the 'My' reference in 'My Sidebar' (used in Netscape 7 only).

VERIFIED FIXED

Status

SeaMonkey
Help Documentation
VERIFIED FIXED
14 years ago
13 years ago

People

(Reporter: stephend, Assigned: stephend)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
The UI for Seamonkey (and Firebird) uses plain 'Sidebar', not 'My Sidebar' as
was used by the commercial Netscape 7.x application.

Here is the LXR string search for the -text- occurrences of 'My Sidebar':
http://lxr.mozilla.org/seamonkey/search?string=my+sidebar

Here is the LXR string search for the -link- occurrences of 'my_sidebar':
http://lxr.mozilla.org/seamonkey/search?string=my_sidebar

I've fixed both cases.

Patch coming up.
(Assignee)

Comment 1

14 years ago
Created attachment 133160 [details] [diff] [review]
Patch
(Assignee)

Comment 2

14 years ago
Note that Daniel has a patch for some of this plus additional cleanup in bug
153671, but I prefer this, as it's more comprehensive and scoped.
Status: NEW → ASSIGNED

Comment 3

14 years ago
Comment on attachment 133160 [details] [diff] [review]
Patch

r=rlk@trfenv.com. good job!
Attachment #133160 - Flags: review+
(Assignee)

Updated

14 years ago
Summary: Drop the 'My' reference in 'My Sidebar' (occurs in Netscape 7 only). → Drop the 'My' reference in 'My Sidebar' (used in Netscape 7 only).
(Assignee)

Comment 4

14 years ago
Fixed.  Thanks for the quick turnaround.

This will also make Daniel's patch smaller (once he CVS up[s])...
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 5

14 years ago
reopening. These lines have to be changed:

<a name="My_Sidebar:removing_tabsIDX"></a> ...
<a name="My_Sidebar:customizing_tabsIDX">
<a name="My_Sidebar:reorganizing_tabsIDX">

Keller, can we have sr? One-person review is not working too well. Let Brant and
I do the reviews, and you and Neil do the super-reivews. As long as people /w
CVS accounts review all patches, we should be okay.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 6

14 years ago
> Keller, can we have sr? One-person review is not working too well. Let Brant and
> I do the reviews, and you and Neil do the super-reivews. As long as people /w
> CVS accounts review all patches, we should be okay.

If you wanna give r=, do ahead. I'm not that good at reviewing documentation, I
realize that. I think I'm going to stop doing r= for documentation changes.

Comment 7

14 years ago
OK, I cleaned up after stephend.
Changes daniel noted checked in.
Status: REOPENED → RESOLVED
Last Resolved: 14 years ago14 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

14 years ago
Doh, sorry all.  For some reason, global search and replace missed those in that
file...

Comment 9

14 years ago
v
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.