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

Enhance UX for the browser action, possibility to disable popup #82

Merged
merged 5 commits into from
Sep 1, 2017

Conversation

silverqx
Copy link
Contributor

Added possibility to enable / disable XDebug Session only by left click on xdebug helper icon.

@silverqx
Copy link
Contributor Author

It's done, I tested edge cases too, like when you have selected trace, then go to options, you disable popup and then click on icon, so in this case will be disabled.

I tested keyboard shortcuts too.

I will test it again tomorrow and this PR will be ready, pls look at it and give some feedback, thx.

@silverqx
Copy link
Contributor Author

Tested again today and every thing looks good 🏠

@silverqx silverqx changed the title [WIP] Enhance UX for the browser action Enhance UX for the browser action, possibility to disable popup Jul 28, 2017
@gazugafan
Copy link

Can confirm it works great for me. Just what I was looking for after switching to Vivaldi, and here it is already done! Hope it gets merged into the master branch!

@mac-cain13
Copy link
Owner

Thanks for the reminder, I should really merge and release this. I will hit the merge button right now, and try to release it asap. :)

@mac-cain13 mac-cain13 merged commit 9025661 into mac-cain13:master Sep 1, 2017
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.

3 participants