Page MenuHomePhabricator

Assist de/it/pl/pt/ru-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice"
Closed, ResolvedPublic8 Estimated Story Points

Event Timeline

tarlocesilion raised the priority of this task from to Needs Triage.
tarlocesilion updated the task description. (Show Details)
tarlocesilion subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Krinkle renamed this task from Editnotice dial displays as empty to Editnotices with hidden content shows as "1 notice".Apr 14 2015, 3:18 PM
Krinkle updated the task description. (Show Details)

pl.wp and it.wp seem to be using the same system as en.wp had previously; de.wp has something similar, but slightly different. en.wp's implementation was modified per T91715 to avoid this issue.

Jdforrester-WMF renamed this task from Editnotices with hidden content shows as "1 notice" to Assist pl/it/de/ru/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".May 6 2015, 9:48 PM
Jdforrester-WMF triaged this task as Medium priority.
Jdforrester-WMF edited a custom field.
Krenair renamed this task from Assist pl/it/de/ru/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice" to Assist pl/it/de/ru/zh/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".May 19 2015, 4:56 PM
He7d3r renamed this task from Assist pl/it/de/ru/zh/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice" to Assist de/it/pl/pt/ru/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".Jun 16 2015, 5:18 PM
This comment was removed by He7d3r.
Krenair renamed this task from Assist de/it/pl/pt/ru/meta-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice" to Assist dewiki/itwiki/plwiki/ptwiki/ruwiki/enwikibooks/metawiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".Jun 24 2015, 3:37 PM

Please stop expanding the scope of this task. The task is to help five specific wikis.

Jdforrester-WMF renamed this task from Assist dewiki/itwiki/plwiki/ptwiki/ruwiki/enwikibooks/metawiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice" to Assist de/it/pl/pt/ru-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".Jun 24 2015, 3:38 PM

I'm increasingly convinced that we should go back to handling this correctly in the software.

I'm increasingly convinced that we should go back to handling this correctly in the software.

We already changed the software to support the concept of inherited edit notices (e.g. "MediaWiki:Editnotice-0") while retaining the ability for that inherited template to conditionally load an actual edit notice.

Edit notice "existence" used to be derived on the page having no wikitext (which necessitated VisualEditor to do visual text extraction).

MediaWiki core now parses the edit notice and if no HTML was produced by the wikitext, the edit notice is not exported. The client-side performance benefit is substantial in the VE loading process. The added server-side processing is not a concern.

This system is the best of both, but does require a small sacrifice from both parties. The software now does additional parsing and extraction. But the template is now required to not output any boilerplate that wouldn't be visible anyway.

Essentially the fix is to either remove the unnecessarily html, or move it to be part of the conditional logic.

Simple cases:

Advanced case of en.wikipedia.org is documented at https://phabricator.wikimedia.org/T91715#1160062.

Do any of these wikis still have the issue?

Jdforrester-WMF claimed this task.

Provisionally marking as closed, then.

Do any of these wikis still have the issue?

I still see this on ptwiki, because I use CSS like this to hide the notices:

[id*="editnotice-"]{
    display: none !important;
}

You should probably consider this bug expected behaviour in your case then @He7d3r .

Yeah. That is, unfortunately, explicitly not supported since T87412 (and that's the entire reason for this bug).