Page MenuHomePhabricator

Add pulsating dot with help popup to `mul` in desktop termbox
Closed, ResolvedPublic

Description

See the parent task for more information. This task is for adding the pulsating dot and showing a popup when it is clicked, with the help message and link. The “don’t show this again” checkbox is left for a separate subtask (T340644).

Event Timeline

Change 933678 had a related patch set uploaded (by Guergana Tzatchkova; author: Guergana Tzatchkova):

[mediawiki/extensions/Wikibase@master] Add pulsating dot for extra info on mul language row

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

Change 935133 had a related patch set uploaded (by Guergana Tzatchkova; author: Guergana Tzatchkova):

[wikibase/termbox@master] Add messages for popup for multiple languages

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

Change 935133 abandoned by Guergana Tzatchkova:

[wikibase/termbox@master] Add messages for popup for multiple languages

Reason:

messages already added in I9b8b40b9dc114341c4d830280b5698fd45b7c450 as part of the "wikibase.view.ControllerViewFactory" module

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

Change 933678 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Add pulsating dot for extra info on mul language row

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

It's working! \o/

One small thing: The position of the dot seems to be off a bit on my system (see screenshot). Would this be easy to fix?

image.png (208×426 px, 11 KB)

One more thing: Every time I click on "All entered languages" it shows me an additional dot. 😅

image.png (370×602 px, 25 KB)

That should have been fixed yesterday (when Don’t add multiple pulsating dots) was merged) – is it still happening for you?

Yes, it's still happening for me on Test Wikidata. It is not happening on Beta. Should the dot even be active on Test?

while this made for a somewhat pleasing visual effect

I should read your patch descriptions more often 😂

Oh yes, I forgot this is also testable on Test Wikidata. Yeah, it should be fixed there next week.

Why should the dot not be on Beta?

I meant on Test (changed it but your answer was too quick xD). I was not sure if we were still testing the soft rollout on Test or not. But thinking about it, it actually makes sense to have the dot during the soft rollout. So nevermind.

Oh no, I am only realizing now, that the dot is also visible in viewing mode. It should only be visible in editing mode. Can we still fix this?

This one can probably be closed?

Fixing the dot position seems to be still open:

image.png (122×352 px, 5 KB)

The dot belongs to the label of the input box ("multiple languages") and not to the input box itself. @Sarai-WMDE can help you, when in doubt.

This one can probably be closed?

Fixing the dot position seems to be still open:

image.png (122×352 px, 5 KB)

I don’t have access to that file.

On my end the dot also wraps around, which makes it hard for me to debug the issue Sarai pointed out:

image.png (54×139 px, 2 KB)

What would your suggestion be?

A separate task for it would probably be best. Right now it’s not even clear to me what the problem is.

Manuel moved this task from Todo/Backlog to Our work done on the Wikidata Dev Team (Sprint-∞) board.

Ok, I see. In case this is non-trivial, what we have is good enough.