Last modified: 2014-06-15 23:31:08 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T47850, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45850 - Code editor should not be cuffed to the enhanced toolbar
Code editor should not be cuffed to the enhanced toolbar
Status: PATCH_TO_REVIEW
Product: MediaWiki extensions
Classification: Unclassified
CodeEditor (Other open bugs)
unspecified
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 55419 (view as bug list)
Depends on: 26918
Blocks:
  Show dependency treegraph
 
Reported: 2013-03-07 16:10 UTC by Danny B.
Modified: 2014-06-15 23:31 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Danny B. 2013-03-07 16:10:57 UTC
Code editor does the line numbering, syntax highlighting and indentation, and it should stay that way. No need for *code* editor to provide *writing* tools.

When editing module with the code editor, user will hardly ever need clickable features from the WikiEditor (signature, links, bold, headline etc...), thus why such resource consuming tool should be loaded when not used.

Actually if there should be any clickable features, then rather a list of keywords for instance and other things relevant to the *coding*. (Nice feature would be list of function definitions eg. but that's for another bug, just an example here.)

Bug 26918 mentions editors should be standalone. And user should be able to choose which editor to load on given page / namespace.
Comment 1 Andre Klapper 2013-03-07 18:50:41 UTC
What is the current problem, and what is your request?
Comment 2 Danny B. 2013-03-21 14:39:00 UTC
When editing the Module, even when the enhanced toolbar is turned off in prefs, code editor still uses that and even if the code editor is turned off, still the enhanced toolbar is shown instead of the classical one.

Code editor switch button must be independent on the toolbar.

Enhanced toolbar must not be loaded when the code editor is off.
Comment 3 Derk-Jan Hartman 2013-12-31 13:41:30 UTC
*** Bug 55936 has been marked as a duplicate of this bug. ***
Comment 4 Isarra 2014-03-09 22:00:53 UTC
*** Bug 55419 has been marked as a duplicate of this bug. ***
Comment 5 Gerrit Notification Bot 2014-04-28 13:37:20 UTC
Change 130068 had a related patch set uploaded by TheDJ:
Persistent disabling of CodeEditor

https://gerrit.wikimedia.org/r/130068
Comment 6 Derk-Jan Hartman 2014-04-29 20:00:10 UTC
OK, so the thing is that yes, in theory it can do that. But in practice, I don't want it to support too many modes. It's not like it gets too much eyes to begin with, it makes it more difficult to support.

Also I want to add a few buttons (bug 59924) etc and I have a patch to hide some of the  normal wikieditor buttons that won't really apply (https://gerrit.wikimedia.org/r/#/c/118993/), so I do actually want some sort of toolbar.

I'm therefore proposing to make the coupling between the two even stricter (fixes of bug 46779) and better persisted and will close this as WONTFIX.

People without WikiEditor enabled will therefore also not be able to use CodeEditor.
Comment 7 Danny B. 2014-04-29 20:32:23 UTC
There is absolute nonsense to provide dozens of buttons for visual editing which are useless for code editing.

Also, in the sense of bug 26918, every editor should be standalone.

When I don't want to use WikiEditor, I don't want to use it. No matter on which page I am, including css/js/lua. Wikieditor consumes lots of resources and that's why I (and many other people) have it in preferences turned off. MediaWiki must treat my preferences and not push me to consume my resources when I want to edit code pages.

However, while on css/js/lua page, if I would like to use the code editor, I want to turn on just the *code* editor, without those useless buttons OR have plain textarea. Now I am pushed to have the WikiEditor and you even plan to push people to have either nothing or both. That's wrong approach. The choice must be CodeEditor or plain textarea. No WikiEditor involved. If anybody will miss any button, CodeEditor can be easily customized like WikiEditor is.
Comment 8 Gerrit Notification Bot 2014-06-15 23:25:54 UTC
Change 139760 had a related patch set uploaded by TheDJ:
[WIP] Try to split out wikiEditor vs codeEditor toolbar

https://gerrit.wikimedia.org/r/139760
Comment 9 Derk-Jan Hartman 2014-06-15 23:31:08 UTC
Did some work on splitting this further out:

https://gerrit.wikimedia.org/r/139759
https://gerrit.wikimedia.org/r/139760

This is work in progress, i'm moving to a situation where in the case of disable enhanced toolbar, the code editor is able to build it's own toolbar, instead of adding to the WikiEditor toolbar.

It still would use the WikiEditor framework to communicate with ACE. The toolbar for ACE is one that uses the WikiEditor toolbar api. Still needs to take into account 'usecodeeditor' preference and needs a designated button in the editor to 'enable' the code editor if it is currently disabled.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links