Last modified: 2012-04-16 09:16:00 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 T29447, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 27447 - Added option to toolbar-item to switch off text-selection
Added option to toolbar-item to switch off text-selection
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
Page editing (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-02-16 10:37 UTC by Carsten Nielsen
Modified: 2012-04-16 09:16 UTC (History)
2 users (show)

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


Attachments
Patch for automatic-selection-enhancement (3.30 KB, patch)
2011-02-16 10:37 UTC, Carsten Nielsen
Details
Patch for automatic-selection-enhancement (3.28 KB, patch)
2011-02-16 10:46 UTC, Carsten Nielsen
Details
Update to previous patch to get rid of php-warning (1.43 KB, patch)
2011-02-17 07:59 UTC, Carsten Nielsen
Details

Description Carsten Nielsen 2011-02-16 10:37:04 UTC
Created attachment 8150 [details]
Patch for automatic-selection-enhancement

A client of mine was a bit annoyed by the editor:
Every time he inserted - lets say - an image, the default contents of the image-fields "thumbs|width|caption...." were automatically marked/selected. So with a tip on the keyboard all "helping" text was gone. 

I added an option to the toolbar-items to optional deactivate this behaviour.

Patch attached.
Comment 1 Carsten Nielsen 2011-02-16 10:46:16 UTC
Created attachment 8151 [details]
Patch for automatic-selection-enhancement

corrected patch
Comment 2 Mark A. Hershberger 2011-02-16 23:06:30 UTC
Applied in r27472
Comment 3 Mark A. Hershberger 2011-02-17 00:44:10 UTC
argh, keep getting these commit revisions wrong.  r82308.

There is currently a FIXME on the code with this comment:

> Notice: Undefined index: select in /home/reedy/mediawiki/trunk/phase3/includes/EditPage.php on line 2390

MediaWiki is expected to run without warnings under E_STRICT.  Can you update your patch?

If you'd like to make more contributions to MediaWiki, I recommend you ask for commit privs.
Comment 4 Carsten Nielsen 2011-02-17 07:59:06 UTC
Created attachment 8156 [details]
Update to previous patch to get rid of php-warning

I've only added a diff to the previous patch as this patch has already been committed as far I can see?
Comment 5 Mark A. Hershberger 2011-02-17 18:44:59 UTC
Thanks, applied in r82343
Comment 6 Roan Kattouw 2011-02-20 19:50:54 UTC
(In reply to comment #0)
> Every time he inserted - lets say - an image, the default contents of the
> image-fields "thumbs|width|caption...." were automatically marked/selected. So
> with a tip on the keyboard all "helping" text was gone. 
> 
Ctrl+Z should bring it right back.
Comment 7 Brion Vibber 2011-07-20 23:57:57 UTC
Indeed this seems like an undesirable change to make on the default buttons which are *exactly* intended to produce sample text that's selected so your typing removes it.

Further, r82343 was never actually functional to begin with... It's been reverted in r92715.

The r82308 changes are still in, so you can pass the extra parameter on the JS side for your custom toolbar items, but the old classic toolbar is being phased out for WikiEditor anyway.

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


Navigation
Links