Last modified: 2014-11-17 10:36:43 UTC
The wording of the preference "Enable dialogs for inserting links, tables and more" is very confusing. It does several very different things: * Turns the link button from just adding [[]] to a whole dialog. * Turns the table button from adding a 3x3 table to a whole dialog. * Add the Search and Replace button, that doesn't exit without it at all. * (Something else that i missed?) The Search and Replace should just always be enabled. It's a useful feature. The table dialog can probably always be on. The link dialog can also be always on, because people who like coding manually (like me) won't miss a button that just adds [[]] and people who are less technical would appreciate the dialog. In short, this preference should just be removed and its functionality should be on by default. See also Bug 28555.
I'd strongly recommend dropping the preference as well.
Seems like it'd be a simple patch -- not that I know anything about it.
Can you tell which PHP file controls the preferences settings??
Created attachment 10099 [details] removes the preferences for dialogs and sets it to global set $wgWikiEditorFeatures[dialogs][global]=true in wikiEditor.php and removed dialogs preferences in wikiEditor.hooks.php
Added keywords to indicate that patch awaits review. Thanks, Nischay.
1. The patch should be created using svn diff, according to instructions here: https://www.mediawiki.org/wiki/Patch 2. The preference should be removed completely and not just hidden and made true.
Created attachment 10164 [details] 2nd patch This patch does the following 1. removes the preferences from the dialogs and toolbar in file WikiEditor.hooks.php 2. set global=true for dialogs and toolbar and user=false so that it is not user preference based anymore
* The corresponding messages in the i18n file must be removed. * As earlier, the showing of the dialogs and the buttons is not supposed to be conditional at all - they should just always be shown.
Created attachment 10189 [details] removes messages from i18n file
applied in https://gerrit.wikimedia.org/r/#change,4339
(In reply to comment #10) > applied in https://gerrit.wikimedia.org/r/#change,4339 Was broken, reverted in https://gerrit.wikimedia.org/r/#change,5135 .