Closed
Bug 1270578
Opened 9 years ago
Closed 5 years ago
cs_nav_prefs_navigator.xhtml: Typo "Editmenu"
Categories
(SeaMonkey :: Help Documentation, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: RainerBielefeldNG, Assigned: njsg)
References
(Blocks 1 open bug)
Details
(Whiteboard: SM2.53.3)
Attachments
(4 files)
48.60 KB,
image/png
|
Details | |
46.74 KB,
image/png
|
Details | |
861 bytes,
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-release+
iannbugzilla
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
11.07 KB,
patch
|
iannbugzilla
:
feedback+
iannbugzilla
:
approval-comm-release+
iannbugzilla
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
Steps how to reproduce with English SeaMonkey 2.45a1 (Windows NT 6.1; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0 Build 20160308001946 (Default Classic Theme) on German WIN7 64bit:
1. Browser → F1
» Help appears
2. Type search string "Browser Preferences" → Click hit
"Browser Preferences"
» Related Help appears
3. Click TOC "History"
Bug: Listing below in item 1. dontains word "Editmenu", what is
uncommon. All other paragraphs use "Edig menu"
b) Suggestion for Fix: replace "Editmenu" by "Edit menu"
Reporter | ||
Updated•9 years ago
|
Summary: cs_nav_prefs_navigator.xhtml: Mentioned "Sidebar Advanced Search Mode" does not exist → cs_nav_prefs_navigator.xhtml: Typo "Editmenu"
Comment 1•9 years ago
|
||
Can I work on this? Could you assign it to me?
Sure, thanks!
Assignee: nobody → ajain2
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 3•9 years ago
|
||
Hello, Could you help me get started or tips on how to resolve it?
I am newbie here. Thanks
As a minimum, you will need the sources from "comm-central" which is a Mercurial repository. For instructions how to get started, see https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_SeaMonkey_build (depending on your platform). If you are on Linux, you should find everything needed in your distro's repository.
For simple changes like this one, you probably don't need a full build. You could just get the sources, make the change, then create the patch. However, it's always a good idea to double-check that what you've done actually yields the intended result (which sometimes it doesn't).
The code specific to SeaMonkey is in the suite/ directory on the top level of the repository. It contains a folder locales/ which has all labels and texts for the English versions (all other languages have their individual repositories).
See https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch for a general description of the process. Once you are ready to submit a patch, follow the steps in (even though the red banner at the top of this page says it's outdated) https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F and attach the patch here for review.
Hope this helps and isn't too overwhelming yet. :-)
Comment 5•9 years ago
|
||
Thanks a lot for this. I appreciate it. I will start working on it.
You are welcome. Another useful resources to just browse and search the code is http://mxr.mozilla.org/comm-central/ which allows you to find specific files or text snippets you are looking for.
Reporter | ||
Comment 7•9 years ago
|
||
This source code might be a little worrying vbecause there is some OS dependent switch in that area:
c) Source cs_nav_prefs_navigator.xhtml (does NOT work)
<ol>
<li>Open the <span class="mac">&brandShortName;</span>
<span class="noMac">Edit</span> menu and choose Preferences.</li>
d) Source nav_help.xhtml (DOES work)
<ol>
<li>Open the
<span class="mac">&brandShortName;</span><span class="noMac">Edit</span>
menu and choose Preferences.</li>
At least I currently do not understand why 'c' does not work bud 'd' does
Reporter | ||
Comment 8•9 years ago
|
||
@ Akshay Jain:
"Bug 1268031 - composer_help.xhtml: Typo" alos might be such a "lost or missing blank" issue.
Comment 9•9 years ago
|
||
Hello Rainer, I will work on Bug 1268031 too.
Comment 10•9 years ago
|
||
JFTR: We have suite/locales/en-US/chrome/common/help/helpFileLayout.css which should import mozilla/toolkit/components/help/content/platformClasses.css (that's where the platform-specific classes are defined). I don't see any reason why that shouldn't work.
The difference between 'c' and 'd' is that 'd' has a new line after the "noMac" span element.
Comment 11•9 years ago
|
||
User agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40
Build identifier: 20160120191716
Verified it on Macintosh where it have text "SeaMonkey Menu"
Comment 12•9 years ago
|
||
User agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40
Build identifier: 20160120202951
Verified it on Windows version. It says "Editmenu". I am working on it. Should I build it after making the changes?
Updated•5 years ago
|
Keywords: good-first-bug
Whiteboard: [easyconfirm][good first bug] → [easyconfirm]
![]() |
||
Updated•5 years ago
|
Assignee: ajain2 → nobody
Status: ASSIGNED → NEW
Assignee | ||
Comment 13•5 years ago
|
||
Attachment #9145558 -
Flags: review?(iann_bugzilla)
Assignee | ||
Comment 14•5 years ago
|
||
This is also present in l10n (in untranslated English files, at least), should I make a patch for these as well?
Comment 15•5 years ago
|
||
Comment on attachment 9145558 [details] [diff] [review]
cs_nav_prefs_navigator.xhtml "Editmenu"->"Edit menu" patch
Yes to l10n. I'm presuming those that have localised to something other than English did not include the typo :)
r/a=me
Attachment #9145558 -
Flags: review?(iann_bugzilla)
Attachment #9145558 -
Flags: review+
Attachment #9145558 -
Flags: approval-comm-release+
Attachment #9145558 -
Flags: approval-comm-esr60+
Assignee | ||
Comment 16•5 years ago
|
||
Attachment #9145571 -
Flags: feedback?(iann_bugzilla)
Comment 17•5 years ago
|
||
Comment on attachment 9145571 [details] [diff] [review]
l10n: cs_nav_prefs_navigator.xhtml "Editmenu"->"Edit menu" patch
[Triage Comment]
LGTM f/a=me
Attachment #9145571 -
Flags: feedback?(iann_bugzilla)
Attachment #9145571 -
Flags: feedback+
Attachment #9145571 -
Flags: approval-comm-release+
Attachment #9145571 -
Flags: approval-comm-esr60+
Comment 18•5 years ago
|
||
Pushed by [email protected]:
https://hg.mozilla.org/comm-central/rev/4083da0d3e3b
Add missing space ("editmenu"->"edit menu"). r=IanN
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Keywords: good-first-bug
Whiteboard: [easyconfirm] → SM2.53.3
You need to log in
before you can comment on or make changes to this bug.
Description
•