-
-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Turn scroll indicator into a draggable bar. #14
Comments
I was looking at gitter's scrollbar and they use webkit scrollbars: https://css-tricks.com/examples/WebKitScrollbars/ Would this be a better solution? |
I think that is a good idea. It could detect device and based on display different scrollbars. |
Note that the webkit scrollbar is actually a native scrollbar - so as far as I know (from the limited research I did) we wouldn't be able to control the height and scroll position of the element. Gitter does use this, but they don't have a true virtual scroll - they append to their chat list when the scroll hits the bottom or top (meaning that the scrollbar jumps). We may be able to emulate a scrollbar that we can control, and have it show up as a scrollbar for assistive technologies (using the WAI-ARIA role). |
@jedd-ahyoung Correct. We can't use a native scrollbar. Maybe do something similar to the scrollbar on OS X which is just an mobile like indicator and on mouseover turn it to a more draggable friendly style and handle dragging. |
We might want to get @joelcoxokc involved in this too. He's working on Aurelia interface and may have some feedback on how to handle emulating the scroll bars. |
@martingust @EisenbergEffect What about just making the scrollbar a pluggable component? This way we're not relying on someone else's implementation. I would imagine this project would get forked a few times if it wasn't extendible... same could be said for any component that has sub-components. @jedd-ahyoung I'd very much like to see any component implement WAI-ARIA properly. Even some kind of validation process for aurelia components to meet that and other standards would be good. |
@rquast I'd definitely like to push for accessbility in Aurelia. I don't want to derail this thread, but I've been doing some tests with Voiceover and it seems that there are some issues with everything being so dynamic. That said, I've been planning to add some more accessibility issues for this plugin. |
Here's a non-native scrollbar example I found which looks pretty similar to what exists already in the code. https://codemirror.net/demo/simplescrollbars.html .. they just use mousedown, mousemove and mouseup events on the scroll div element. I think it would be pretty simple to make the existing scroll do the same thing by adding those extra events to the scrollhandler. https://codemirror.net/addon/scroll/simplescrollbars.js |
More discussion on this #23 |
I don't know what the default is for mobile apps, but a desktop app needs a scroll bar rather than the indicator div.
The text was updated successfully, but these errors were encountered: