Page MenuHomePhabricator

Topic containers not handling sub headings (<h3+> correctly)
Closed, ResolvedPublic

Description

For example the first discussion here gets no counters:

https://www.mediawiki.org/wiki/Talk:Typography_refresh

It also can't be subscribed to, even with topic containers disabled (T298617)

Event Timeline

Esanders renamed this task from Topic containers not handling sub headings (<h3+> correctly) to Topic containers & topic subscriptions not handling sub headings (<h3+> correctly).Jul 5 2022, 5:26 PM
Esanders updated the task description. (Show Details)

Change 811348 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/DiscussionTools@master] Fix fetching of oldest comment in a thread

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

The subscription part of this is T298617, so I'll limit this task to the topic containers part.

Esanders renamed this task from Topic containers & topic subscriptions not handling sub headings (<h3+> correctly) to Topic containers not handling sub headings (<h3+> correctly).Jul 6 2022, 1:35 PM
Esanders updated the task description. (Show Details)

From the patch:

Bartosz Dziewoński
This actually fixes T298617 as well, but I am not sure if that isn't too much. I wrote a longer comment on that task, but in short, I'm concerned that "merging in" a subsection into a topic can now change its name and break the subscriptions (as seen in several of the tests). I'd like us to do something smart about this situation first (e.g. notify subscribers when it happens).

Esanders
I think there are always going to be cases where people splice bits of threads together and/or add introductory comments which break subscriptions. I would be nice if we could resolve that, maybe with the comment metadata database, but for now I think this is at least consistent and understandable logic, and probably fixes more cases than it breaks.

Change 816226 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/DiscussionTools@master] Show topic container even if heading is unsubscribable

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

This patch ^ solves the topic container issue without addressing T298617.

Change 816226 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Show topic container even if heading is unsubscribable

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

@Esanders would it be accurate for me to think the reason that the first section on this page (== New font stack breakdown ==) does not get the Topic Container counters/metadata because the signature ( Edokter (talk) — 12:36, 12 April 2014 (UTC)) appears within an H4, in this case ====Helvetica====?

matmarex moved this task from QA to Ready for Sign Off on the Editing-team (Kanban Board) board.
matmarex subscribed.

@Esanders would it be accurate for me to think the reason that the first section on this page (== New font stack breakdown ==) does not get the Topic Container counters/metadata because the signature ( Edokter (talk) — 12:36, 12 April 2014 (UTC)) appears within an H4, in this case ====Helvetica====?

This looks fixed now. Probably the page's parser cache had to expire (or be purged).