Page MenuHomePhabricator

Clearer checkbox meaning on Special:ManageMessageGroups
Closed, ResolvedPublic


Having two checkboxes "Ignore" and "Import and ask updates" for each message is very confusing: in particular, one wonders what happens if both are selected. How can a message be both ignored and imported?

Event Timeline

Nikerabbit added a project: good first task.
Nikerabbit added a subscriber: Nikerabbit.

Sounds like a radio button is appropriate to use here when there are multiple options.

			$label = $this->msg( 'translate-manage-action-ignore' )->text();
			$actions = Xml::checkLabel( $label, "i/$id", "i/$id" );

			if ( $group->getSourceLanguage() === $code ) {
				$label = $this->msg( 'translate-manage-action-fuzzy' )->text();
				$actions .= ' ' . Xml::checkLabel( $label, "f/$id", "f/$id", true );

I could take this one if it's still relevant and Nemo_bis isn't working on it

I will not be working on this for the next couple months,

I'm trying to reproduce this, but I can' t.

My page says:

There are no changes to process.

Not sure where I need to make the changes. I've read this:

and a bunch of other pages like:

Omidfi removed Omidfi as the assignee of this task.Oct 8 2017, 4:53 AM

I didn't get lucky with this one, I can't get the configuration work. I leave it for now so that it becomes open in case someone elase wants to work on it.

Change 505385 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[mediawiki/extensions/Translate@master] Replace checkbox with radio buttons on Special:ManageMessageGroups

Replaced the check-boxes with the radio buttons to make the UI more aligned with the actual behavior. Also added another option "Import" which is displayed when there is a change in a non-source translation.

image.png (639×1 px, 42 KB)

Also moved the ignore button to the end as it made more sense to have the default selected options shown first.

Change 345596 abandoned by Abijeet Patro:
Radio button for ignore vs. fuzzy on Special:ManageMessageGroups

Abandoned in favor of I9c7e7e18ed4996d36e48253a66c5789afe4f9e86

Added the import button for source language also. This will import the changes, without marking translations in other language as fuzzy.

Tweaked the CSS as per discussion with Niklas, and tested across themes. Here are a few screenshots,

T160606-timeless.png (789×1 px, 62 KB)

T160606-monobook.png (700×1 px, 55 KB)

T160606-vector.png (745×1 px, 39 KB)

Change 505385 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Replace checkbox with radio buttons on Special:ManageMessageGroups

abi_ removed a project: Patch-For-Review.

Updating status as patch has been merged.

Have been using this updated UI on for a few days now. Changes have been working as expected.

Today, also used the UI to ignore a change.

image.png (562×1 px, 68 KB)

abi_ moved this task from QA to Done on the User-abi_ board.