Skip to content

fix(extension): toggle inspecting on button click #229

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

Merged
merged 2 commits into from
Jul 1, 2020

Conversation

marcosvega91
Copy link
Member

@marcosvega91 marcosvega91 commented Jul 1, 2020

fix #228

What: I have fixed an issue where the user can't disable inspecting after click on the icon

Why: User can't use the browser without a refresh

How: By saving state in a ref and toggle inspecting checking this

Checklist:

  • Tests
  • Ready to be merged

@marcosvega91
Copy link
Member Author

@smeijer do you why for my branches the CI fails to assign PR? For another PR I had the same issue

Copy link
Member

@smeijer smeijer left a comment

Choose a reason for hiding this comment

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

This will be running out of sync when I select an element on the page. Selecting elements, stops the inspecting but keeps this flag on false. Which requires me to click the button twice to start inspecting again.

@marcosvega91
Copy link
Member Author

right I didn't think about it. I have added a TOGGLE_INSPECTING event

Copy link
Member

@smeijer smeijer left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks.

@smeijer smeijer changed the title fix: toggle inspecting on button click fix(extension): toggle inspecting on button click Jul 1, 2020
@smeijer smeijer merged commit 3980a1b into testing-library:develop Jul 1, 2020
@smeijer
Copy link
Member

smeijer commented Jul 1, 2020

@smeijer do you why for my branches the CI fails to assign PR? For another PR I had the same issue

I have no idea. I did change something. So on your next PR, we'll know if it works. Unfortunately, I can't rerun it on this one with the new config.

@marcosvega91
Copy link
Member Author

Thanks 😄

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.

Can't disable "select element" mouseover
2 participants