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

Inspector: Store as global Variable #1583

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chaseWillden
Copy link
Contributor

Adds the ability to right click any DOM node and store it as a global variable for console usage.
I personally use this all the time in the other browsers.

Screenshot 2024-09-30 at 9 48 29 PM Screenshot 2024-09-30 at 9 49 34 PM

@warpdesign
Copy link
Contributor

How about defining $0 alias? This automatically points to the selected DOM element in Chrome/Safari console.

@AtkinsSJ
Copy link
Member

AtkinsSJ commented Oct 1, 2024

How about defining $0 alias? This automatically points to the selected DOM element in Chrome/Safari console.

We actually have that implemented but it's currently broken. It's impossible to add a test for so it kept rotting. 😢

@chaseWillden
Copy link
Contributor Author

How about defining $0 alias? This automatically points to the selected DOM element in Chrome/Safari console.

We actually have that implemented but it's currently broken. It's impossible to add a test for so it kept rotting. 😢

Just want to understand next steps for PRs like this one. Is it recommended to create a PR and wait till someone merges? Do I need to reach out to the owner of this area? What's the normal process for this type of stuff?

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