Closed Bug 1270578 Opened 5 years ago Closed 9 months ago
_nav _prefs _navigator .xhtml: Typo "Editmenu"
48.60 KB, image/png
46.74 KB, image/png
861 bytes, patch
|Details | Diff | Splinter Review|
11.07 KB, patch
|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"
Summary: cs_nav_prefs_navigator.xhtml: Mentioned "Sidebar Advanced Search Mode" does not exist → cs_nav_prefs_navigator.xhtml: Typo "Editmenu"
Can I work on this? Could you assign it to me?
Assignee: nobody → ajain2
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
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. :-)
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.
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
@ Akshay Jain: "Bug 1268031 - composer_help.xhtml: Typo" alos might be such a "lost or missing blank" issue.
Hello Rainer, I will work on Bug 1268031 too.
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.
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"
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?
Whiteboard: [easyconfirm][good first bug] → [easyconfirm]
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
Comment on attachment 9145571 [details] [diff] [review] l10n: cs_nav_prefs_navigator.xhtml "Editmenu"->"Edit menu" patch [Triage Comment] LGTM f/a=me
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.