Page MenuHomePhabricator

CX2: Machine translation tools should not appear in toolbars inside modal dialogs
Closed, ResolvedPublic

Description

A picture is worth of thousand words (credits to @Petar.petkovic)

image.png (687×704 px, 126 KB)

I guess the fact that this happens makes sense in a way because it is a toolbar after all, but obviously it shouldn't happen.

Event Timeline

Pginer-WMF moved this task from Needs Triage to CX2 on the ContentTranslation board.

Keep in mind "Reset translation" appears in VE toolbar as well, not only inside modal dialogs.

ve-menu-item-in-more-tools.png (534×382 px, 16 KB)

Change 440993 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/ContentTranslation@master] Disable catchall for MT tools

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

Change 440993 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Disable catchall for MT tools

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

Etonkovidova subscribed.

Checked the fix in cx2-testing:

Screen Shot 2018-06-25 at 4.36.57 PM.png (556×548 px, 44 KB)

Screen Shot 2018-06-25 at 4.44.40 PM.png (562×1 px, 75 KB)

Vvjjkkii renamed this task from CX2: Machine translation tools should not appear in toolbars inside modal dialogs to ygbaaaaaaa.Jul 1 2018, 1:05 AM
Vvjjkkii reopened this task as Open.
Vvjjkkii removed Nikerabbit as the assignee of this task.
Vvjjkkii raised the priority of this task from Medium to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed a subscriber: Aklapper.
CommunityTechBot renamed this task from ygbaaaaaaa to CX2: Machine translation tools should not appear in toolbars inside modal dialogs.Jul 2 2018, 1:31 PM
CommunityTechBot closed this task as Resolved.
CommunityTechBot assigned this task to Nikerabbit.
CommunityTechBot lowered the priority of this task from High to Medium.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot edited subscribers, added: Aklapper; removed: Petar.petkovic.