Closed
Bug 1270578
Opened 8 years ago
Closed 4 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•8 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•8 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•8 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•8 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•8 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•8 years ago
|
||
@ Akshay Jain: "Bug 1268031 - composer_help.xhtml: Typo" alos might be such a "lost or missing blank" issue.
Comment 9•8 years ago
|
||
Hello Rainer, I will work on Bug 1268031 too.
Comment 10•8 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•8 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•8 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•4 years ago
|
Keywords: good-first-bug
Whiteboard: [easyconfirm][good first bug] → [easyconfirm]
Updated•4 years ago
|
Assignee: ajain2 → nobody
Status: ASSIGNED → NEW
Assignee | ||
Comment 13•4 years ago
|
||
Attachment #9145558 -
Flags: review?(iann_bugzilla)
Assignee | ||
Comment 14•4 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•4 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•4 years ago
|
||
Attachment #9145571 -
Flags: feedback?(iann_bugzilla)
Comment 17•4 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•4 years ago
|
||
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/4083da0d3e3b
Add missing space ("editmenu"->"edit menu"). r=IanN
Status: ASSIGNED → RESOLVED
Closed: 4 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
•