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 _execute_browser_action shortcut key #135

Merged
merged 2 commits into from Jan 9, 2018

Conversation

1 participant
@matatk
Owner

matatk commented Jan 8, 2018

Suggested by Coolfields Consulting; fixes #133.

On Chrome on Mac, there seems to be a bug with the popup not sizing correctly sometimes—seems this is only realistic for the browser to fix, though (and this PR doesn't make the effects worse for Landmarks, and it does provide a helpful keyboard shortcut, so no reason not to merge).

matatk added some commits Jan 8, 2018

Add _execute_browser_action shortcut key
Fixes #133

Unfortunately on Chrome there seems to be a bug with the popup not
sizing correctly sometimes:
https://bugs.chromium.org/p/chromium/issues/detail?id=457887
Run README through Pandoc for tidying
Nice to see it convert some dashes :-).

@matatk matatk merged commit f76525c into master Jan 9, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@matatk matatk deleted the add-extension-popup-shortcut branch Jan 9, 2018

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