Page MenuHomePhabricator

Show disambiguation extract in preview instead of custom disambiguation view
Closed, ResolvedPublic

Assigned To
Authored By
hueitan
Oct 18 2021, 7:28 AM
Referenced Files
F42399538: 2024-03-05_09-02-38.png
Mar 5 2024, 5:07 PM
F42399536: 2024-03-05_09-02-29.png
Mar 5 2024, 5:07 PM
F42399493: 2024-03-05_08-59-54.png
Mar 5 2024, 5:01 PM
F42399491: 2024-03-05_08-59-43.png
Mar 5 2024, 5:01 PM
F42399403: 2024-03-05_08-51-16.png
Mar 5 2024, 5:01 PM
F42399396: 2024-03-05_08-50-32.png
Mar 5 2024, 5:01 PM
F42398580: Screenshot 2024-03-05 at 23.30.49.png
Mar 5 2024, 3:33 PM
F42398583: image.png
Mar 5 2024, 3:33 PM

Description

In many cases, summary api returns the disambiguation page extract description, somehow it can be useful instead of just redirect link.

We can't promise it always has the extract information, therefore the existing redirect link/text will be provided when extract is not available.

Storybook to try: https://wikimedia.github.io/wikipedia-preview/main/storybook/?path=/story/targets--hyperlink&args=title:seo

with extract infowhen no extract info
image.png (606×738 px, 67 KB)
Screenshot 2024-03-05 at 23.30.49.png (784×1 px, 187 KB)
image.png (477×920 px, 32 KB)

Here's the link for more disambiguation pages
https://en.wikipedia.org/wiki/Wikipedia:Links_to_disambiguating_pages

QA Results - Storybook

Event Timeline

SBisson edited projects, added Inuka-Team; removed Inuka-Team (Kanban).
SBisson edited projects, added Inuka-Team (Next); removed Inuka-Team.
PWaigi-WMF edited projects, added Inuka-Team (Kanban); removed Inuka-Team (Next).
PWaigi-WMF moved this task from Backlog to Definition on the Inuka-Team (Kanban) board.
PWaigi-WMF moved this task from Definition to Ready for Dev on the Inuka-Team (Kanban) board.
SBisson renamed this task from [Suggestion] Show disambiguation extract in preview instead of redirect disambiguation suggested text when available to Show disambiguation extract in preview instead of custom disambiguation view.Feb 20 2024, 1:36 PM

If anyone can find a disambiguation page with no extract data, please post it here and I will update the test spec in the code.

@hueitan Here are a few summary Api returns disambiguation page extract descriptions besides a redirect link as seen in the screenshots below. I will move this to design sign-off. Thanks for all your work and for updating the task!

Status: ✅PASS
Environment: WordPress 6.4.3; Wikipedia Preview 1.12
OS: macOS Sonoma 14.2.1
Browser: Chrome 122
Skins. N/A
Device: MBA M2
Emulated Device:: n/a
Test Links:
https://wikimedia.github.io/wikipedia-preview/main/storybook/?path=/story/targets--hyperlink&args=title:seo
https://en.wikipedia.org/wiki/Seo
https://wikimedia.github.io/wikipedia-preview/main/storybook/?path=/story/targets--hyperlink&args=title:net
https://en.wikipedia.org/wiki/Net
https://wikimedia.github.io/wikipedia-preview/main/storybook/?path=/story/targets--hyperlink&args=title:zip
https://en.wikipedia.org/wiki/Zip

✅AC1: https://phabricator.wikimedia.org/T293594

Disambiguate WordStorybookWikipedia
SEO
2024-03-05_08-50-32.png (1×1 px, 104 KB)
2024-03-05_08-51-16.png (1×1 px, 321 KB)
Net
2024-03-05_08-59-43.png (998×1 px, 95 KB)
2024-03-05_08-59-54.png (1×1 px, 295 KB)
Zip
2024-03-05_09-02-29.png (978×1 px, 90 KB)
2024-03-05_09-02-38.png (1×1 px, 335 KB)

@hueitan change in this task looks fine to me though I also think once we deploy changes mentioned in T356846 then changes as part of this task won't be of any use?

@hueitan change in this task looks fine to me though I also think once we deploy changes mentioned in T356846 then changes as part of this task won't be of any use?

Yes, WPP won't show disambiguation page anymore.

It's still available for those who don't use wordpress plugin but the WP library itself.