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

[Debugger] Onboarding for Map Scopes #126

Open
digitarald opened this issue May 18, 2020 · 4 comments
Open

[Debugger] Onboarding for Map Scopes #126

digitarald opened this issue May 18, 2020 · 4 comments
Labels
P2 Secondary tasks for current Nightly cycle

Comments

@digitarald
Copy link
Contributor

Background

Map scopes transforms build/compressed into how the original source behaves. It's magical when enabled buy we can't enable it by default as it has performance overhead (especially on the first pause).

It makes several variable-related features more reliable:

  • Scopes pane
  • Variable hover preview (without it preview will not work at all for original variables)
  • Inline variable preview (same breakage as above)
  • With 78: Now also mapping in Logpoints (making logging work for original variables)

Problem

The checkbox is so subtle that it's easy to ignore, so users lose out on a better debugging experience.

Solution

Lightweight onboarding to opt users into enabling Map scopes when it's available for a file.

This could be a subtle bar in the Scopes pane on pausing or on top of the file as a lot of variable inspection happens in context of the code.

@digitarald digitarald added the P2 Secondary tasks for current Nightly cycle label May 18, 2020
@digitarald
Copy link
Contributor Author

@violasong
Copy link
Contributor

(Unused) example of onboarding styling we could use

image

@janodvarko
Copy link
Member

@violasong, thanks for the mockup, I like that

If I understand correctly, we'd display similar notification message explaining what "Map Scopes" does and perhaps link to MDN - directly within the "Scopes" panel at the top?

I am also attaching a screenshot of what we do for the multi-line Console editor

image

Honza

@olfek
Copy link

olfek commented Apr 6, 2023

The checkbox is so subtle that it's easy to ignore, so users lose out on a better debugging experience.

I just ran into this over at terser/terser#1367

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Secondary tasks for current Nightly cycle
Projects
None yet
Development

No branches or pull requests

4 participants