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

Touch Bar support #345

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Touch Bar support #345

wants to merge 5 commits into from

Conversation

greg
Copy link
Contributor

@greg greg commented Dec 21, 2016

Basic Touch Bar support: compose button, and prev/next conversation navigation.

Also fixes prev/next navigation and changes keyboard shortcuts to ⌘[ and ], as ⌃[⇧]⇥ conflicts with the search feature (and the page overrides) (see independent PR #410 for this change)

- change keyboard shortcuts as the previous conflicted with the search
feature (which overrode)
- Compose button, and prev/next conversation navigation
Selected conversation would not be found correctly if there were
non-selected unread conversations.
@rsms
Copy link
Owner

rsms commented Feb 18, 2017

Since I don't have a computer with TouchBar, I unfortunately have no way of testing this.

@greg
Copy link
Contributor Author

greg commented Feb 18, 2017

After building and running in Xcode, you can enable the Touch Bar simulator from the Window menu (⌘⇧5).

@sindresorhus
Copy link

@rsms You can also test it with Touch Bar Simulator (No Xcode needed).

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