I'm not 100% sure but the recent occurrence might have been something like:

• open a page with both enabled
• disable navpopups in another tab
• hover

Maybe Hovercards checks whether NavPopups is enabled at the moment of hovering, not whether it was enabled at the time of the page load?

This should fix this: T135693

How is this different from T135628 ? Confused.

This is about the two showing up at the same time. That suggests some error in the logic that is likely to remain even if you flip the priorities.

Change 292487 had a related patch set uploaded (by Bmansurov):
Do not show Hovercards when NavPopups gadget is enabled on huwiki

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

Temporary fix has been created. A long term proper fix is still needed. Some suggestions from IRC:

17:41 tgr: in general, I think using wikidata is neat if the WD community is up for it
17:43 tgr: iw-link the gadget definition pages (e.g. https://en.wikipedia.org/wiki/MediaWiki:Gadget-Navigation_popups ) to each other and that gives you the RL module name

@Tgr please feel free to add your other suggestions here.

Change 292487 merged by jenkins-bot:
Do not show Hovercards when NavPopups gadget is enabled on huwiki

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

Change 292497 had a related patch set uploaded (by Dereckson):
Do not show Hovercards when NavPopups gadget is enabled on huwiki

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

Change 292497 merged by jenkins-bot:
Do not show Hovercards when NavPopups gadget is enabled on huwiki

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

@bmansurov @Tgr What about having a config variable like $wgPopupsDisableWhenGadget = "latszer"; with a default of "Navigation_popups"?

It will mean checking the gadget name on every wiki we want to deploy, but it is flexible to change per wiki.

I don't like adding more config variables, but I also don't like an extension checking adhoc for gadgets and we're doing it, so this seems like a sensible solution.

Thoughts?

It's probably more scalable to use a system message: define a message key like poups-navpoup-gadget-name with Navigation_popups as the English text and {{notranslate}} in qqq, ask editors (e.g. on the wikitech-ambassadors list) to put the gadget name on the corresponding MediaWiki page, and use $msg->inContentLanguage() to load it.

In T135630#2352431, @Jhernandez wrote:

@bmansurov @Tgr What about having a config variable like $wgPopupsDisableWhenGadget = "latszer"; with a default of "Navigation_popups"?

It will mean checking the gadget name on every wiki we want to deploy, but it is flexible to change per wiki.

I don't like adding more config variables, but I also don't like an extension checking adhoc for gadgets and we're doing it, so this seems like a sensible solution.

Thoughts?

What would happen when we want to enable Popups for all wikis? I think this approach maybe one of the last resorts.

Can someone update the description. I'm a little confused with why this wasn't fixed.
This shouldn't be possible - the Hovercards code is not invoked when the gadget is enabled.

Thanks @Tgr
The logic for checking if Hovercards is enabled is here:
https://github.com/wikimedia/mediawiki-extensions-Popups/blob/master/resources/ext.popups.targets/desktopTarget.js#L7

I don't see any check against Navigation_Popups - it seems strictly feature detection. What am I missing?

@bmansurov fixed this in 827e1db I think?

Yes, although not perfect, it solves the problem. The solution is not good because we need to check whether the NavPopups gadget is enabled each time the user hovers over a link. @Tgr, do you think we should find an alternative solution and repurpose this task or can we close it as invalid/fixed?

If you want to avoid loading the Hovercards JS when NavPopups is enabled, you need a better way to detect it, but that should probably have its own task. If you don't care about that, I don't see much disadvantage to checking on every hover.