Closed
Bug 362703
Opened 19 years ago
Closed 19 years ago
Update /support/firefox/menu to Fx2
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cilias, Assigned: cilias)
References
()
Details
Attachments
(1 file)
|
16.33 KB,
patch
|
djst
:
review+
|
Details | Diff | Splinter Review |
<http://www.mozilla.org/support/firefox/menu> hasn't been updated for Firefox 2.
| Assignee | ||
Comment 1•19 years ago
|
||
The patch for this is pretty much ready, except for a few issues.
- In the Help Viewer, the Edit-->Find_in_This_Page item contains a note about FAYT. Do you want that added to the menu page as well?
- I changed the "Character Coding" label to "Character Encoding". That's what it says on the Windows version. Is it different on other OSes?
- Any additions were based on what I saw on Windows XP. If there any changes, that are not visable on the Windows version, I need to know about them.
- view source mentions bug 172817, but I don't see a UI for it. Do you want to remove that sentence, or add a tip to the tips page, and change the sentence.
- Tools > Options references sometimes use <tt>, and sometimes use <span class=mnu">. Is the use of different tags intentional? If not, which tag do you want to use for all "Tools > Options" references?
Comment 2•19 years ago
|
||
(In reply to comment #1)
> The patch for this is pretty much ready, except for a few issues.
>
> - In the Help Viewer, the Edit-->Find_in_This_Page item contains a note about
> FAYT. Do you want that added to the menu page as well?
Not sure what you mean here. The entry in the help viewer seems up to date, let's use that text.
>
> - I changed the "Character Coding" label to "Character Encoding". That's what
> it says on the Windows version. Is it different on other OSes?
I think it's the same but I can only verify it in Linux.
>
> - Any additions were based on what I saw on Windows XP. If there any changes,
> that are not visable on the Windows version, I need to know about them.
The most notable difference is that Linux/Mac use Edit>Preferences and Windows uses Tools>Options.
>
> - view source mentions bug 172817, but I don't see a UI for it. Do you want to
> remove that sentence, or add a tip to the tips page, and change the sentence.
Good point! This should indeed be added to the Tips & Tricks section. You need to set view_source.editor.external to "true", and then set view_source.editor.path to e.g. "C:\Program\Texturizer\Texturizer.exe" or whatever is your preferred text editor. I just checked and indeed it works.
If you have time, please file a new bug for this, both for adding the tips and for adding a reference to it in menu.html. For now, just remove the text about a "future version".
>
> - Tools > Options references sometimes use <tt>, and sometimes use <span
> class=mnu">. Is the use of different tags intentional? If not, which tag do you
> want to use for all "Tools > Options" references?
<span class="mnu">. <tt> is just a mistake, really. :)
Thanks!
| Assignee | ||
Comment 3•19 years ago
|
||
Attachment #247656 -
Flags: review?(djst.mozilla)
Comment 4•19 years ago
|
||
Comment on attachment 247656 [details] [diff] [review]
Draft1
Well done!
Attachment #247656 -
Flags: review?(djst.mozilla) → review+
| Assignee | ||
Comment 5•19 years ago
|
||
Checking in menu.html;
/cvsroot/mozilla-org/html/support/firefox/menu.html,v <-- menu.html
new revision: 1.14; previous revision: 1.13
done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Product: mozilla.org → Websites
Updated•13 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•