-
-
Notifications
You must be signed in to change notification settings - Fork 63
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
wrap it in a chrome extension! #11
Comments
This is a very great feature |
@benmonro created an official one: https://github.com/testing-library/which-query |
That's an official |
Cool. What features are you putting into that one? |
@benmonro , how about a preview? 😎 https://twitter.com/meijer_s/status/1269703405150314498?s=20 |
That's cool! It does feel like we should combine them into one extension though. Just my 2 cents |
Yeah mine also has an elements inspector panel and adds things to the console. Would definitely love to combine forces on this. I think it'd be best for the community imo |
@smeijer i can make you admin on that repo in case that's a concern. Let me know how we can come together on this. My goal is just to make the best experience for everyone and avoid fragmenting |
No worries. Let me wrap this thing up, and see where we can meet. The biggest concern that I have with being an "official extension" isn't about being an admin or not. It's about the limitations. In my opinion, the official one should stick to the strict recommendations. While an unofficial extension, can provide some fallback queries. Less perfect, but still workable. ( I still haven't tried |
That's fair. How would you feel if I integrated some of what you've done into the official one? I was wanting to add a devtool page and what you have here is perfect.... |
About which parts are you thinking specifically? Just functionality / code? Or also the look and feel of testing-playground.com? If you tell me your plans with which-query, I might submit a PR once this one is done. But having two identical extensions doesn't make a lot of sense. I've also added a pane to the element inspector. That's where my work in this extension started. But I moved to a dedicated panel later on. I can share a short video of that tomorrow. |
Yeah my plan was to build something very similar to what you have. Perhaps just the suggestion part. The goal if which query is to improve the a11y of the tests through the queries so I probably would leave the bottom half out (though if you want to combine them into one extension I'd be fine keeping it) |
I've checked out The only thing that I haven't implemented, is a context menu. Would it not make more sense to merge that functionality in the testing-playground extension? To answer your question:
To be honest, I wouldn't be thrilled. I've been working hard on this project. And ending up with two equal extensions doesn't make a lot of sense. Here a small preview of the added sidebar panel: |
Kind of bummed to hear that. I also worked hard on which query and shared my ideas and progress with you the whole way. Was really hoping we could Come together for the sake of community but hey it's your decision. |
I understand. And I appreciate the work you've done by implementing the query suggestions from testing-playground into the core of testing-library. I also like that testing-playground has inspired you to turn I still believe it's possible to come together. As I said earlier, wouldn't it make sense to merge the functionality of |
Closed by #145 |
I believe we can wrap this project into a chrome extension. That way people can use the thing on their own sites, without needing to copy / paste html fragments. How awesome would that be?!
Resources
The text was updated successfully, but these errors were encountered: