Page MenuHomePhabricator

Echo overlay should not transfer scroll action to background window
Open, LowestPublic

Description

--Reposted conversation from email thread--

Check out the notification menu on pinterest new/you/messages has some interesting behavior, "new" and "you" never trigger page scrolling but "messages" does when you get to the end, it might be useful to take a look at what the difference in these sections is. This is relevant to the Winter TOC, Echo notifications, and possibly task recommendations. I'd like to see if we can emulate this behavior in all of our scrolling flyouts, it feels much more polished than the page suddenly scrolling when you get to the bottom of a scrolled list in a flyout.

If you don't have an account on pinterest I can demo for you, I'd record a video, but it really is more of a feeling, that video won't really convey. Since we're seeing more and more uses of this element in the interface, it might make sense for one of the 2 teams working on it to componentize something that we can use consistently.

When you're scrolling a list in the flyout, when you get to the end (on e.g. the "you" section), and just keep scrolling, you will never scroll the page no matter how much you scroll (as long as your mouse is still over the flyout)

FWIW, for Echo, it seems like pretty good behavior in Firefox 30.0. When I get to the end and keep scrolling, the first scroll or two at the end do nothing, but if I continue it will start scrolling the page.

I don't have time to work on this right now, though. I recommend you file an enhancement request in Bugzilla or float it on the design list.


Version: unspecified
Severity: enhancement

Details

Reference
bz70908

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:48 AM
bzimport set Reference to bz70908.
bzimport added a subscriber: Unknown Object (MLST).

Moving to Echo since it concerns Echo. Echo doesn't use mediawiki ui (it could and should) but the fix for the bug should start there.

Note MediaWiki UI in current form is simply css and html markup, scrolling is JavaScript behaviour.

Mattflaschen-WMF triaged this task as Lowest priority.Jan 9 2015, 6:39 PM
Mattflaschen-WMF renamed this task from Scrolling list overlays should not transfer scroll action to background windows to Echo overlay should not transfer scroll action to background window.Mar 15 2015, 3:53 AM
Mattflaschen-WMF set Security to None.

You don't want to investigate a reusable fix that would work on all scrolling views that exist on pages that scroll? Seems like this has been coming up more and more frequently.

Jorm removed a subscriber: Jorm.Mar 16 2015, 6:19 AM

Found this library which seems to allow for this https://github.com/inuyaksa/jquery.nicescroll might be overkill but seeing how its done here could be insightful

Found this library which seems to allow for this https://github.com/inuyaksa/jquery.nicescroll might be overkill but seeing how its done here could be insightful

Thanks for looking into this.

Legoktm moved this task from Backlog to Needs plan on the Notifications board.Jul 6 2015, 8:12 AM
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptJul 6 2015, 8:12 AM