Closed Bug 221950 Opened 21 years ago Closed 21 years ago

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

Categories

(SeaMonkey :: Help Documentation, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stephend, Assigned: stephend)

Details

Attachments

(1 file)

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.
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 on attachment 133160 [details] [diff] [review]
Patch

r=rlk@trfenv.com. good job!
Attachment #133160 - Flags: review+
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).
Fixed.  Thanks for the quick turnaround.

This will also make Daniel's patch smaller (once he CVS up[s])...
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
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 → ---
> 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.
OK, I cleaned up after stephend.
Changes daniel noted checked in.
Status: REOPENED → RESOLVED
Closed: 21 years ago21 years ago
Resolution: --- → FIXED
Doh, sorry all.  For some reason, global search and replace missed those in that
file...
v
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: