Status

REOPENED
a year ago
9 months ago

People

(Reporter: isaacschemm, Assigned: isaacschemm)

Tracking

Trunk
seamonkey2.54

SeaMonkey Tracking Flags

(seamonkey2.54 fixed)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Assignee)

Description

a year ago
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Firefox/52.0 SeaMonkey/2.49.1
Build ID: 20170617112514

Steps to reproduce:

* Go to the "Web Development Tools" section in SeaMonkey Help
* Click on "Firefox Developer Tools site" (https://developer.mozilla.org/docs/Tools)


Actual results:

WebIDE is not mentioned


Expected results:

Since WebIDE appears in the SeaMonkey Web Development tool, but not on the Fx devtools homepage, we should probably mention it separately in Help.

The WebIDE page (https://developer.mozilla.org/docs/Tools/WebIDE) is apparently focused on Firefox OS development. Since Firefox OS is discontinued, but you can also use WebIDE to debug Firefox for Android or other apps, the remote debugging guide (https://developer.mozilla.org/docs/Tools/Remote_Debugging) might be a better resource to link to.
(Assignee)

Updated

a year ago
Component: General → Help Documentation
(Assignee)

Comment 1

a year ago
WebIDE is slated to go away eventually, and remote debugging will be moved elsewhere (see bug 1314811). But we can update the SeaMonkey documentation to remove it when that time comes.
(Assignee)

Comment 2

a year ago
Created attachment 8895357 [details] [diff] [review]
1387565-mention-webide-in-help.patch
Attachment #8895357 - Flags: review?
Attachment #8895357 - Flags: feedback?(iann_bugzilla)
Attachment #8895357 - Flags: feedback?(frgrahl)
Comment on attachment 8895357 [details] [diff] [review]
1387565-mention-webide-in-help.patch

Looks good. It should be "Firefox Developer Tools WebIDE" or just "Developer Tools WebIDE".

Additonal not. The Developer Tools will be moved into system addon which might break them in SeaMonkey very soon. So this help might need to change again before the next planned release 2.56.
Attachment #8895357 - Flags: feedback?(frgrahl) → feedback+
(Assignee)

Comment 4

a year ago
Created attachment 8896091 [details] [diff] [review]
1387565-mention-webide-in-help-v2.patch

Changed "Firefox WebIDE" to "Developer Tools WebIDE".
Attachment #8895357 - Attachment is obsolete: true
Attachment #8895357 - Flags: review?
Attachment #8895357 - Flags: feedback?(iann_bugzilla)
Attachment #8896091 - Flags: review?(frgrahl)

Comment 5

a year ago
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/d74685ff0eae
Mention WebIDE in Help. r=frg
Status: UNCONFIRMED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Isaac, could you keep an eye on it if it needs changes in later releases.

Thanks.
status-seamonkey2.54: --- → fixed
Target Milestone: --- → seamonkey2.54
Comment on attachment 8896091 [details] [diff] [review]
1387565-mention-webide-in-help-v2.patch

Hmm I thought I set r+. doh
Attachment #8896091 - Flags: review?(frgrahl) → review+

Updated

10 months ago
Assignee: nobody → isaacschemm
Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: FIXED → ---

Comment 8

10 months ago
Comment on attachment 8896091 [details] [diff] [review]
1387565-mention-webide-in-help-v2.patch

>+++ b/suite/locales/en-US/chrome/common/help/developer_tools.xhtml

>+  <li id="webide">
>+    <strong>WebIDE</strong>: &brandShortName; includes Developer Tools WebIDE,
>+	which can be used to debug other web sites or Gecko-based applications
>+	running on the same computer, or on another device (such as a smartphone)
>+	through a USB or network connection. To learn more, visit the
>+	<a href="https://developer.mozilla.org/docs/Tools/Remote_Debugging">Remote
>+	Debugging</a> page at MDN. Note that you may not be able to debug
>+	applications that are newer than the included version of WebIDE.
>+  </li>
Nit: Spaces not tabs should be used for indentation.

>+++ b/suite/locales/en-US/chrome/common/help/help-index1.rdf
>@@ -613,16 +613,21 @@
>          nc:link="mailnews_getting_started.xhtml#set_as_default"/>
>      </rdf:li>
>      <rdf:li>
>        <rdf:Description ID="Developer_Tools"
>          nc:name="Developer Tools"
>          nc:link="developer_tools.xhtml#devtools"/>
>      </rdf:li>
>      <rdf:li>
>+       <rdf:Description ID="WebIDE"
>+         nc:name="WebIDE"
>+         nc:link="developer_tools.xhtml#webide"/>
>+     </rdf:li>
>+     <rdf:li>
>        <rdf:Description ID="device_manager"
>          nc:name="Device Manager"
>          nc:link="using_certs_help.xhtml#about_security_devices_and_modules"/>
>      </rdf:li>
Nit: this file should be in alphabetical order, so this should be under W not D.

Please can a new patch be generated to fix these issues? Thanks.
Flags: needinfo?(isaacschemm)
(Assignee)

Comment 9

10 months ago
Created attachment 8924192 [details] [diff] [review]
1387565-mention-webide-in-help-v3.patch

Updated patch with WebIDE in correct position alphabetically and fixed whitespace
Flags: needinfo?(isaacschemm)
Attachment #8924192 - Flags: review?(iann_bugzilla)

Comment 10

9 months ago
Comment on attachment 8924192 [details] [diff] [review]
1387565-mention-webide-in-help-v3.patch

>+++ b/suite/locales/en-US/chrome/common/help/help-index1.rdf
>@@ -2207,16 +2207,21 @@
>          nc:link="certs_prefs_help.xhtml#privacy_and_security_preferences_certificates"/>
>      </rdf:li></rdf:Seq>
>    </nc:subheadings>
> </rdf:Description>
> 
> <rdf:Description about="help-indexAZ.rdf#w">
>    <nc:subheadings>
>      <rdf:Seq><rdf:li>
>+       <rdf:Description ID="WebIDE"
>+         nc:name="WebIDE"
>+         nc:link="developer_tools.xhtml#webide"/>
>+     </rdf:li>
>+     <rdf:li>
This needs to come after "web pages"
>        <rdf:Description ID="web_pages"
>          nc:name="web pages"/>
>      </rdf:li>
>      <rdf:li>
>        <rdf:Description ID="Working_Offline"
>          nc:name="Working Offline"
>          nc:link="mailnews_offline.xhtml#working_offline"/>
>      </rdf:li></rdf:Seq>

Once this patch is correct, the previous one needs to be backed out before this lands or this patch needs to include the back out.

r=me with that fixed.
Attachment #8924192 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 11

9 months ago
Created attachment 8927689 [details] [diff] [review]
1387565-mention-webide-in-help-v4.patch

Put WebIDE after Web Pages in index
Attachment #8924192 - Attachment is obsolete: true
Attachment #8927689 - Flags: review?(iann_bugzilla)

Comment 12

9 months ago
Comment on attachment 8927689 [details] [diff] [review]
1387565-mention-webide-in-help-v4.patch

r=me thanks
Attachment #8927689 - Flags: review?(iann_bugzilla) → review+
You need to log in before you can comment on or make changes to this bug.