Skip to content

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