Closed
Bug 717373
Opened 13 years ago
Closed 12 years ago
Remove the textarea fallback from the Source Editor
Categories
(DevTools :: General, defect, P1)
DevTools
General
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 13
People
(Reporter: msucan, Assigned: rcampbell)
References
Details
(Keywords: dev-doc-complete, Whiteboard: [sourceeditor][killthem])
Attachments
(1 file, 1 obsolete file)
26.67 KB,
patch
|
msucan
:
review+
|
Details | Diff | Splinter Review |
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•13 years ago
|
Priority: -- → P2
Comment 1•13 years ago
|
||
If the devtools team is satisfied with the current quality of the Orion service, I'm fine with it! :-)
Assignee | ||
Comment 2•13 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•12 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•12 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.
Comment 5•12 years ago
|
||
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?
Comment 6•12 years ago
|
||
(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•12 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.
Comment 8•12 years ago
|
||
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•12 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•12 years ago
|
Assignee: nobody → rcampbell
Status: NEW → ASSIGNED
Priority: P2 → P1
Assignee | ||
Updated•12 years ago
|
Whiteboard: [sourceeditor] → [sourceeditor][killthem]
Assignee | ||
Comment 10•12 years ago
|
||
did I miss anything?
Attachment #594224 -
Flags: review?(mihai.sucan)
Assignee | ||
Comment 11•12 years ago
|
||
restored ui check
Attachment #594224 -
Attachment is obsolete: true
Attachment #594224 -
Flags: review?(mihai.sucan)
Attachment #594235 -
Flags: review?
Reporter | ||
Comment 12•12 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•12 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•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/250f556a5d49
Whiteboard: [sourceeditor][killthem] → [sourceeditor][killthem][fixed-in-fx-team]
Comment 15•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/250f556a5d49
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [sourceeditor][killthem][fixed-in-fx-team] → [sourceeditor][killthem]
Target Milestone: --- → Firefox 13
Reporter | ||
Comment 16•12 years ago
|
||
Please document the removal of the textarea fallback component from the Source Editor in Firefox 13. Thank you!
Keywords: dev-doc-needed
Comment 17•12 years ago
|
||
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
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•