Last modified: 2014-11-17 10:36:43 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 T32796, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 30796 - "Enable dialogs for inserting links, tables and more" preference is confusing
"Enable dialogs for inserting links, tables and more" preference is confusing
Status: REOPENED
Product: MediaWiki extensions
Classification: Unclassified
WikiEditor (Other open bugs)
unspecified
All All
: Normal enhancement with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-07 08:09 UTC by Amir E. Aharoni
Modified: 2014-11-17 10:36 UTC (History)
4 users (show)

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


Attachments
removes the preferences for dialogs and sets it to global (2.36 KB, patch)
2012-02-25 18:07 UTC, Nischay Nahata
Details
2nd patch (1.62 KB, patch)
2012-03-03 08:18 UTC, Nischay Nahata
Details
removes messages from i18n file (73.90 KB, patch)
2012-03-07 15:26 UTC, Nischay Nahata
Details

Description Amir E. Aharoni 2011-09-07 08:09:28 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.
Comment 1 Brion Vibber 2011-09-07 19:07:11 UTC
I'd strongly recommend dropping the preference as well.
Comment 2 Mark A. Hershberger 2011-11-14 20:51:24 UTC
Seems like it'd be a simple patch -- not that I know anything about it.
Comment 3 Nischay Nahata 2012-02-25 13:15:35 UTC
Can you tell which PHP file controls the preferences settings??
Comment 4 Nischay Nahata 2012-02-25 18:07:59 UTC
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
Comment 5 Sumana Harihareswara 2012-02-26 03:58:58 UTC
Added keywords to indicate that patch awaits review.  Thanks, Nischay.
Comment 6 Amir E. Aharoni 2012-03-02 19:49:04 UTC
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.
Comment 7 Nischay Nahata 2012-03-03 08:18:00 UTC
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
Comment 8 Amir E. Aharoni 2012-03-07 11:12:12 UTC
* 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.
Comment 9 Nischay Nahata 2012-03-07 15:26:15 UTC
Created attachment 10189 [details]
removes messages from i18n file
Comment 10 Nischay Nahata 2012-04-05 17:35:42 UTC
applied in https://gerrit.wikimedia.org/r/#change,4339
Comment 11 Roan Kattouw 2012-04-24 01:09:27 UTC
(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 .

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


Navigation
Links