(Translated by https://www.hiragana.jp/)
⚓ T249548 Allow reference heading to scroll with content
Page MenuHomePhabricator
Create Task
Maniphest T249548

Allow reference heading to scroll with content
Closed, ResolvedPublic2 Estimated Story Points
Actions

Description

Description

After playing around with the beta feature I've realized that having a fixed heading doesn't add much value to the experience of viewing a reference, and it comes at the expense of the reading experience. By allowing the header to scroll with the content we would free-up space and improve the reading experience.

Video:

Screen Recording 2020-04-06 at 4.12.07 PM.mov1 MBDownload

Basic prototype: https://codepen.io/alexhollender/pen/abOeNxB

Details

SubjectRepoBranchLines +/-
Scroll reference heading as wellmediawiki/extensions/Popupsmaster+22 -22
Customize query in gerrit

Related Objects

Event Timeline

• alexhollender_WMF created this task.Apr 6 2020, 8:12 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 6 2020, 8:12 PM
Lena_WMDE moved this task from Backlog to Before Default on the Reference Previews board.Oct 14 2020, 3:14 PM
ECohen_WMDE moved this task from Before Default to Ready for pickup on the Reference Previews board.Oct 16 2020, 8:32 AM
Lena_WMDE set the point value for this task to 2.Oct 19 2020, 9:14 AM
Lena_WMDE added a project: WMDE-QWERTY-Sprint-2020-10-21.Oct 21 2020, 8:26 AM
Lena_WMDE moved this task from Ready for pickup to Doing on the Reference Previews board.
awight claimed this task.Oct 26 2020, 2:21 PM
awight moved this task from Sprint Backlog to Doing on the WMDE-QWERTY-Sprint-2020-10-21 board.
awight removed awight as the assignee of this task.Oct 26 2020, 2:52 PM
awight moved this task from Doing to Sprint Backlog on the WMDE-QWERTY-Sprint-2020-10-21 board.
awight subscribed.
gerritbot added a comment.Oct 27 2020, 1:34 PM

Change 636655 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@master] Scroll reference heading as well

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

gerritbot added a project: Patch-For-Review.Oct 27 2020, 1:34 PM
thiemowmde claimed this task.Oct 27 2020, 1:34 PM
thiemowmde moved this task from Sprint Backlog to Review on the WMDE-QWERTY-Sprint-2020-10-21 board.
thiemowmde added a comment.Oct 27 2020, 2:05 PM

Here is an example how the scrollbar looks after this change. Chrome (Ubuntu):

Screenshot from 2020-10-27 14-59-38.png (432×378 px, 41 KB)

Firefox (Ubuntu):

Screenshot from 2020-10-27 15-01-05.png (425×363 px, 34 KB)

Can someone post a screenshot from an Apple machine?

thiemowmde mentioned this in T249530: Move scrollbar to outer container.Oct 27 2020, 2:08 PM
awight moved this task from Review to Demo on the WMDE-QWERTY-Sprint-2020-10-21 board.Oct 28 2020, 9:25 AM
gerritbot added a comment.Oct 28 2020, 10:33 AM

Change 636655 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Scroll reference heading as well

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

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.16; 2020-11-03).Oct 28 2020, 11:00 AM
Maintenance_bot removed a project: Patch-For-Review.Oct 28 2020, 11:10 AM
thiemowmde closed this task as Resolved.Nov 3 2020, 12:59 PM
Lena_WMDE moved this task from Demo to Done on the WMDE-QWERTY-Sprint-2020-10-21 board.Nov 3 2020, 1:08 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits