Page MenuHomePhabricator

Stop using jquery.tipsy for HTMLCheckMatrix labels (seen on Special:Preferences with Echo)
Closed, ResolvedPublic

Assigned To
Authored By
matmarex
Nov 8 2015, 4:22 PM
Referenced Files
F36972783: image.png
May 2 2023, 7:22 PM
F25275421: image.png
Aug 23 2018, 10:43 PM
F8610968: firefox_2017-07-03_10-59-56.png
Jul 3 2017, 7:00 PM
F8610920: firefox_2017-07-03_10-54-15.png
Jul 3 2017, 6:54 PM
F2946835: pasted_file
Nov 10 2015, 2:15 AM
Tokens
"Like" token, awarded by TerraCodes.

Description

Stop using jquery.tipsy for HTMLCheckMatrix labels.

pasted_file (323×329 px, 22 KB)

resources/src/mediawiki.htmlform/checkmatrix.js
		$matrixTooltips = $root.find( '.mw-htmlform-matrix .mw-htmlform-tooltip' );
		if ( $matrixTooltips.length ) {
			mw.loader.using( 'jquery.tipsy', function () {
				$matrixTooltips.tipsy( { gravity: 's' } );
			} );
		}

Related Objects

Event Timeline

matmarex raised the priority of this task from to Needs Triage.
matmarex updated the task description. (Show Details)
matmarex added a project: MediaWiki-HTMLForm.

@matmarex: What else to use, if someone wanted to work on this? Is this an good first task / GCI task or rather not? Technical-Debt?

Replicating the current behavior using OOjs UI would require T88630 first (or a custom solution).

matmarex renamed this task from Stop using jquery.tipsy for HTMLCheckMatrix labels to Stop using jquery.tipsy for HTMLCheckMatrix labels (seen on Special:Preferences with Echo).Nov 10 2015, 2:14 AM
matmarex updated the task description. (Show Details)
matmarex added a project: JavaScript.
matmarex set Security to None.

(Or redesigning this so that information is not hidden in popups/tooltips.)

Is Echo currently the only user of this? We could probably surface this info in a different way...

I think so. It was also considered in T53941 (see the patch there).

Is this what causes the warning

This page is using the deprecated ResourceLoader module "jquery.tipsy".

at
https://pt.wikipedia.org/wiki/Special:Preferences
?

(Or redesigning this so that information is not hidden in popups/tooltips.)

How does

firefox_2017-07-03_10-54-15.png (903×839 px, 81 KB)
look?

or bolded?

firefox_2017-07-03_10-59-56.png (908×809 px, 80 KB)

or bolded?

Should we remove the redundant "Notify me" everywhere?

Jdlrobson subscribed.

Can this be ported to OOUI now?

I think it can be ported to OOUI at any moment, provided someone does the work.

As far as I know there isn't any remaining use of 'checkmatrix' field in non-OOUI mode on Wikimedia wikis, so this wasn't a priority for anyone.

In case someone wants to work on this now, here's the minimal code to test this feature (on a special page etc.):

$form = HTMLForm::factory( 'table', [
	'foo' => [
		'type' => 'checkmatrix',
		'rows' => [ 'A' => 'a' ],
		'columns' => [ 'B' => 'b' ],
		'tooltips' => [ 'A' => 'aaaa' ],
	]
], $this->getContext() );
$form->show();

Which displays a tiny form like this, currently still using a 'jquery.tipsy' tooltip:

image.png (209×377 px, 8 KB)

Considering that (probably) nothing uses this code now, it might be acceptable to just remove the JS code that adds the tipsy tooltips, and leave behind plain HTML title attributes.

Agreed. I think that is acceptable.

Change 915866 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Remove last usage of jquery.tipsy in core

https://gerrit.wikimedia.org/r/915866

Change 915866 merged by jenkins-bot:

[mediawiki/core@master] Remove last usage of jquery.tipsy in core

https://gerrit.wikimedia.org/r/915866

matmarex assigned this task to Jdlrobson.