[go: up one dir, main page]

Page MenuHomePhabricator

Make the ReferencePreviews feature compatible with sub-references (Cite Extends)
Closed, ResolvedPublic

Description

User story
As a reader when I hover over a sub-reference I want to see the text for the parent reference along with the text for the sub-reference.

For the first approach use this layout as depicted in the screenshot:

Screenshot from 2024-05-24 14-52-50.png (303×556 px, 78 KB)

Please note, design will be adapted once UX is back from vacation.

Implementation details:

  • One possible solution is to teach the Cite extension to understand nested references, and show the parent as well.

Things to be done

  • Identify sub-references when scraping the preview text from the wiki page
  • Identify the parent reference
  • Scrape content of the parent reference
  • Combine both in the preview (TBD by design)

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
thiemowmde renamed this task from Make Popups and MobileFrontend compatible with BookReferencing to Make the ReferencePreviews feature of Popups compatible with BookReferencing.Jan 15 2020, 6:47 PM
thiemowmde updated the task description. (Show Details)
Tobi_WMDE_SW set the point value for this task to 5.
Tobi_WMDE_SW subscribed.

We'll start implementing points 1-3 while waiting on design feedback for point 4.

Change 566710 had a related patch set uploaded (by WMDE-Fisch; owner: WMDE-Fisch):
[mediawiki/extensions/Popups@master] [WIP] Include parent text on previews of extended references

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

The algorithm part of the patch including a test is ready. Can be reviewed concerning that aspect. Waiting for the final mock on this now.

1) Sub Book References with Existing Style - Desktop.jpg (881×1 px, 940 KB)

Here is the prototype no.1, a sub book reference example with the existing design.

2.3.png (172×322 px, 12 KB)

2.2.png (172×322 px, 15 KB)

2.1.png (172×322 px, 15 KB)

3. Wikidata Design Adapted.png (453×443 px, 17 KB)

2.4.png (232×316 px, 20 KB)

Here are some other ideas which are more experimental: (File names are numbered in order for further discussion)

Reference Preview.png (232×316 px, 17 KB)
@thiemowmde Here is the updated prototype after your input from today.

Here is the latest prototype, since the popup is next to the reference number it can be a good idea to remove it from the pop-up. Also instead of extra icons, we can just show the page number with a line break.

Reference Preview.png (227×317 px, 16 KB)

@Erdinc_Ciftci_WMDE, is the line "S. 22-77" indented by a few pixels? It looks like this in the image, but might be unintentional.

@thiemowmde Yes, but I'm not sure if it's a good idea. I just felt the need to separate "Sub Reference Info" from the "Reference Info" and maybe @Hanna_Petruschat_WMDE can give feedback on that since she is better on visual styling.

I thought that was a mistake with an unintentional space before the page number. I would not leave this there. It's not clear what it should mean in my opinion and is definitely not obvious enough for an indentation. I also think, we don't need it.

Ok, thanks for the clarification! It's no indention then.

The reference number is probably useless, but I think we should easily identify the specific part.

Indeed, the reader who watches successively several references does not want to be obliged to read again the full reference content, they should be able to visually identify the specific part (the subreference content).

So, according to me, this indentation should be more pronounced, instead of removing it.

Change #566710 abandoned by Jdlrobson:

[mediawiki/extensions/Popups@master] Include parent text on previews of extended references

Reason:

Hello this is an automated message.
I am abandoning this patch as it over a year old, and is not currently in a mergeable state. This has nothing to do with the quality of the patch.

If you still care about this patch, please feel free to restore it and rebase it, and we can happily continue the conversation to help you get it merged.

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

Lina_Farid_WMDE renamed this task from Make the ReferencePreviews feature of Popups compatible with BookReferencing to Make the ReferencePreviews feature compatible with sub-references (Cite Extends).May 24 2024, 12:25 PM
Lina_Farid_WMDE removed a project: Page-Previews.
Lina_Farid_WMDE updated the task description. (Show Details)
WMDE-Fisch updated the task description. (Show Details)
WMDE-Fisch removed the point value for this task.

Change #1049968 had a related patch set uploaded (by Mareike Heuer; author: Mareike Heuer):

[mediawiki/extensions/Cite@master] Display reference preview with sub reference

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

Change #1049968 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] Display reference preview with sub reference

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