Skip to content
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

Add mouse scroll wheel mapping #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xxv
Copy link

@xxv xxv commented Oct 23, 2017

This scrolls with the natural scrolling style, moving with the movement
of the page, not the scrollbar.

This scrolls with the natural scrolling style, moving with the movement
of the page, not the scrollbar.
Copy link
Contributor

@algernon algernon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I agree that having a scroll wheel on the factory firmware would be awesome, doing so has a few implications, like having to update the printed & laminated sheet too, as that includes mouse keys as well. Not doing that may end up surprising new users.

Another thing I'm unsure about is the order of up / down scroll. It seems that scrolling down is by the "mouse up" key. That seems counter-intuitive to me.

In short, I'm a bit torn here. On one hand, scroll wheel is <3. On another, the order, and the implications make this change less practical.

But it's up to @obra in the end.

@obra
Copy link
Member

obra commented Nov 26, 2017

I think we want to create a second model 01 firmware demo repo with a 'showy' firmware that has examples of all the bells and whistles.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants