Remove the textarea fallback from the Source Editor

RESOLVED FIXED in Firefox 13

Status

()

Firefox
Developer Tools
P1
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: msucan, Assigned: rc)

Tracking

({dev-doc-complete})

Trunk
Firefox 13
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sourceeditor][killthem])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
The current textarea fallback cannot keep up with the changes planned ahead for the Source Editor (things like autocomplete, debugger ruler, and so on).

We should remove the textarea fallback code, since we won't be able to maintain it and our developer tools cannot use it (for example the debugger and the style editor).

Ehsan: can you please point us to specific requirements (if any) we should meet before we remove the textarea fallback? Is the polish we did in the past sufficient to allow us to do that? (now that Orion is enabled by default)

Thank you very much!
(Reporter)

Updated

6 years ago
Priority: -- → P2
(Reporter)

Updated

6 years ago
Blocks: 717384
If the devtools team is satisfied with the current quality of the Orion service, I'm fine with it!  :-)
(Assignee)

Comment 2

6 years ago
it's painful maintaining the two. We should do this, but I would like some feedback from David Bolter and the a11y group before final decision. CCing...

Comment 3

6 years ago
(In reply to Rob Campbell [:rc] (robcee) from comment #2)
> it's painful maintaining the two. We should do this, but I would like some
> feedback from David Bolter and the a11y group before final decision. CCing...

Marco, ping?

Comment 4

6 years ago
The current style editor is not at all useable for someone using a screen reader. The cursor appears to be stuck on two 0x00fc characters, the style information is a big chunk of spans or so underneath a div inside the accessible tree, no line numbers and no way to discern which belongs to what.
Marco, sorry I led you astray. A better way to test Orion is: tools, web developer, scratchpad.

Where is the pref to set it to use the fallback editor?
(In reply to David Bolter [:davidb] from comment #5)
> Where is the pref to set it to use the fallback editor?

devtools.editor.component => textarea
(Reporter)

Comment 7

6 years ago
(In reply to David Bolter [:davidb] from comment #5)
> Marco, sorry I led you astray. A better way to test Orion is: tools, web
> developer, scratchpad.
> 
> Where is the pref to set it to use the fallback editor?

In about:config, the devtools.editor.component pref. One can change it to "orion" or to "textarea", depending on what one intends to use.

Thank you Marco and David for looking into it! Please let us know the results of your testing and any recommendations you have.
Thanks Cedric.

Mihai, I understand well the pain of multiple code paths.

I want to better understand the maintenance/upkeep pain details in this case, because not being able to use the developer tools is also pain. I might be able to find a resource if necessary.

In the meantime I'm waiting on my IBM contact for Orion a11y roadmap info.
(Reporter)

Comment 9

6 years ago
(In reply to David Bolter [:davidb] from comment #8)
> Thanks Cedric.
> 
> Mihai, I understand well the pain of multiple code paths.
> 
> I want to better understand the maintenance/upkeep pain details in this
> case, because not being able to use the developer tools is also pain. I
> might be able to find a resource if necessary.
> 
> In the meantime I'm waiting on my IBM contact for Orion a11y roadmap info.

That's much appreciated! I would very much like Orion to become more accessible.

The divs and the spans are there, as Marco points out. I assume that with some aria attributes and other accessibility-related improvements Orion can be much more usable. Until then, I agree, it's not as easy to use Orion as it is the textarea fallback.

Maintenance is a problem indeed. However we are gradually working on adding source code editor-specific features. For example now I am working on the debugger gutter (bug 707987). This leaves us in a state where we can no longer use the textarea fallback, unless we go ahead and re-implement parts of Orion, even Orion itself. That is what we want to avoid in the first place by using Orion and by not forking the project (we make upstream contributions to make sure the Orion code runs as good as possible within the Firefox environment).

Similar to the case of the JS Debugger we have the Style Editor which currently fails to work with the textarea fallback.

We can't easily keep the textarea fallback usable while going forward with features we need built into the developer tools.

Please let me know if you need further details or if you have any other questions. Thank you!
(Assignee)

Updated

6 years ago
Assignee: nobody → rcampbell
Status: NEW → ASSIGNED
Priority: P2 → P1
(Assignee)

Updated

6 years ago
Whiteboard: [sourceeditor] → [sourceeditor][killthem]
(Assignee)

Comment 10

6 years ago
Created attachment 594224 [details] [diff] [review]
remove text area

did I miss anything?
Attachment #594224 - Flags: review?(mihai.sucan)
(Assignee)

Comment 11

6 years ago
Created attachment 594235 [details] [diff] [review]
remove text area

restored ui check
Attachment #594224 - Attachment is obsolete: true
Attachment #594224 - Flags: review?(mihai.sucan)
Attachment #594235 - Flags: review?
(Reporter)

Comment 12

6 years ago
Comment on attachment 594235 [details] [diff] [review]
remove text area

Review of attachment 594235 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. Thank you!

::: browser/app/profile/firefox.js
@@ +1097,3 @@
>  //   "orion" - this is the Orion source code editor from the Eclipse project. It
>  //   provides programmer-specific editor features such as syntax highlighting,
>  //   indenting and bracket recognition. It may not be appropriate for all

Probably we want to update the orion component description, since the a11y notes are no longer relevant because we remove the textarea fallback. Thoughts?
Attachment #594235 - Flags: review? → review+
(Assignee)

Comment 13

6 years ago
(In reply to Mihai Sucan [:msucan] from comment #12)
> Comment on attachment 594235 [details] [diff] [review]
> remove text area
> 
> Review of attachment 594235 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good. Thank you!
> 
> ::: browser/app/profile/firefox.js
> @@ +1097,3 @@
> >  //   "orion" - this is the Orion source code editor from the Eclipse project. It
> >  //   provides programmer-specific editor features such as syntax highlighting,
> >  //   indenting and bracket recognition. It may not be appropriate for all
> 
> Probably we want to update the orion component description, since the a11y
> notes are no longer relevant because we remove the textarea fallback.
> Thoughts?

yeah, good call, I'll fix, update the patch and land it.

thanks!
(Assignee)

Comment 14

6 years ago
https://hg.mozilla.org/integration/fx-team/rev/250f556a5d49
Whiteboard: [sourceeditor][killthem] → [sourceeditor][killthem][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/250f556a5d49
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [sourceeditor][killthem][fixed-in-fx-team] → [sourceeditor][killthem]
Target Milestone: --- → Firefox 13
(Reporter)

Comment 16

6 years ago
Please document the removal of the textarea fallback component from the Source Editor in Firefox 13. Thank you!
Keywords: dev-doc-needed
This is now mentioned on Firefox 13 for developers; there is no other relevant place to mention it.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.