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

feat: set focus on toolbar buttons when they are clicked to unfocus a… #1475

Merged
merged 2 commits into from
Aug 30, 2019

Conversation

fxisco
Copy link
Contributor

@fxisco fxisco commented Aug 23, 2019

Feature

This PR helps the toolbar button to gain focus when is clicked. This way any input that is being edited will loose focus and as result will be saved.

Issue: #1474

Copy link
Contributor

@manuelurenah manuelurenah left a comment

Choose a reason for hiding this comment

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

🏄

@@ -405,7 +407,6 @@
const activeNav = savedTab || 'all-pages';

this.$store.dispatch('showNavBackground', true);

Copy link
Contributor

Choose a reason for hiding this comment

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

Please leave an empty line before a return statement to make ir more readable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ohhhh there was an empty space that my linter removed. Thank you man.

@coveralls
Copy link

coveralls commented Aug 23, 2019

Coverage Status

Coverage remained the same at 33.91% when pulling 1a2cdd2 on focus-when-toolbar-button-click into 41ffb17 on master.

@fxisco fxisco force-pushed the focus-when-toolbar-button-click branch from a390f5e to 1a2cdd2 Compare August 30, 2019 15:02
@fxisco fxisco merged commit 13e9158 into master Aug 30, 2019
@fxisco fxisco deleted the focus-when-toolbar-button-click branch August 30, 2019 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants