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

Suggestion: [playground] Provide an option to reset to the default configuration #592

Open
1 task done
amareshsm opened this issue Jun 27, 2024 · 3 comments
Open
1 task done
Labels
enhancement New feature or request

Comments

@amareshsm
Copy link
Member

amareshsm commented Jun 27, 2024

What problem do you want to solve?

Currently, in the playground, if we modify some configurations and later wish to revert to the default settings, it can be challenging for the user to remember the newly added configurations and remove them correctly to return to the default Config.

For instance, if a user enables all the rules to check for all errors and then wants to revert to the default configuration, it will be very difficult for them.

What do you think is the correct solution?

We provide a 'Reset to Defaults' button in the UI, which restores the configuration to its default (initial) Configs.

Refs:
Screenshot 2024-06-27 at 11 45 19 PM
Screenshot 2024-06-27 at 11 47 04 PM

Participation

  • I am willing to submit a pull request for this change.

Additional comments

We provide a 'Reset to Default' button in the UI similar to the below image. Open for any other suggestions also.

Screenshot 2024-06-28 at 12 06 08 AM

@amareshsm amareshsm added the enhancement New feature or request label Jun 27, 2024
@snitin315
Copy link
Contributor

This seems helpful, I'm fine with this change. I'd like to know what other team members think.

// @eslint/eslint-team

@fasttime
Copy link
Member

Personally, I support this proposal 👍 I've often been the guy who was looking for that button, only to realize it wasn't there every time again. Even so, I'm aware of recommendations not to use reset buttons in forms for various reasons like this one, or this one from MDN (in the note), so I'd be interested in other opinions.

@JoshuaKGoldberg
Copy link
Contributor

👍 from me on the proposal in general, and also 👍 on @fasttime's comment that this should not fall into the common reset button accessibility traps. At the very least the solution should have an 'undo' option per https://www.w3.org/WAI/WCAG2/supplemental/patterns/o4p05-form-undo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Feedback Needed
Development

No branches or pull requests

4 participants