| • iamjessklein | |
| Apr 29 2021, 12:30 PM |
| F34470525: Screenshot_2021-05-27 Talk Main Page – Patch Demo (683694,1).png | |
| May 26 2021, 11:22 PM |
Actual
When I get a notification (in the echo notification dropdown) saying that someone replied to my conversation on a talk page, it directs me to the thread but doesn't style the thread in any way to make clear to the contributor what part of the page they should look at.
Expected
The thread that's been updated should be highlighted.
The highlight is light blue and never fades.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T276497 Scale DiscussionTools to all projects | |||
| Open | None | T251207 [Epic] Scale DiscussionTools to all Wikipedias | |||
| Resolved | ppelberg | T284489 Deploy topic subscriptions (desktop) as opt-out feature at all projects | |||
| Resolved | ppelberg | T284490 Offer topic subscriptions as a Beta Feature at all projects | |||
| Resolved | ppelberg | T287276 Offer manual topic subscriptions as Beta Feature at all projects | |||
| Open | None | T233447 [OKR] Increase response rates | |||
| Open | None | T273920 [Interventions] Notifications | |||
| Open | None | T263820 [Intervention] Manual topic subscriptions | |||
| Resolved | • LZaman | T274188 Offer manual topic subscriptions as a Beta Feature at partner wikis | |||
| Resolved | matmarex | T281471 Topic Subscription Prototype: Highlight thread after following a notification |
Change 683694 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Highlight target comment when following a link or notification
Change 683694 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Highlight target comment when following a link or notification
Expected
[…]
The highlight […] never fades.
I don’t like when such highlights are “stuck” on the page until I navigate away; maybe I already do something entirely different. I like GitHub’s approach: when I open an anchor link (e.g. https://github.com/wikimedia/parsoid/pull/5#issuecomment-371981298), it highlights the comment, this highlight doesn’t fade out automatically, but it disappears immediately when I click anywhere on the page outside of that comment.
That's interesting, I never noticed that before. I think we're open to changing that, although it might also be weird in some cases (e.g. if the highlight would disappear when you click [reply]).
Also, to clarify, "the highlight never fades" only means that it's not on a timer (it doesn't fade by itself), but it is removed if you navigate to a different anchor on the page, e.g. by clicking a section in the table of contents.
Is this really how it should look like? The highlight expands in height to approximately the half of the previous comment. (Firefox 78.10.0esr, Debian 10.)
No, it shouldn't look like that. I didn't test it on Firefox, I didn't really expect it to behave differently.
It's actually interesting why that happens. We use the Range#getBoundingClientRect method to determine where to draw the highlight, using a range that covers the whole comment, and starts inside the "start marker" of the comment. The start marker node also serves as the anchor target, and for that purpose, it's positioned slightly above the comment text, so that tall characters don't touch the edge of the screen when your browser scrolls to it. It looks like Chromium ignores that element when computing the bounding rect, but Firefox does not. Firefox's behavior actually feels more correct here, now that I think about it.
We should be able to fix this by adjusting the range to start after the marker, instead of inside of it.
Change 696509 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Adjust comment ranges to exclude the start/end markers
Change 696509 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Adjust comment ranges to exclude the start/end markers
Change 696617 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Follow-up: Adjust comment ranges to exclude the start marker *only*
Change 696617 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Follow-up: Adjust comment ranges to exclude the start marker *only*
Also, I noticed more issues: the highlight doesn't move when opening reply tool, or collapsible stuff, or disappear when opening VE.
Looks like we need to put more thought into "permanent" highlights.
Change 697447 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Change how highlights are positioned to work better with unaware tools
Change 697447 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Change how highlights are positioned to work better with unaware tools
Test wiki on Patch Demo by JKlein (WMF) using patch(es) linked to this task was deleted:
Test wiki on Patch demo by JKlein (WMF) using patch(es) linked to this task was deleted: