Refactor the UI of the Auto-Approve Menu #3910
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a breaking change in that it removes a setting, but I made it as an illustration for someone to experience how this change would feel versus the current implementation.
Thus, DO NOT MERGE.
I believe the removed setting may be better served in the Settings pane Auto-Approved section anyway.
Related GitHub Issue
Proposal part of: #3909
Description
Re-ordered
Test Procedure
See issue description
Type of Change
srcor test files.Pre-Submission Checklist
npm run lint).console.log) has been removed.npm test).mainbranch.npm run changesetif this PR includes user-facing changes or dependency updates.Screenshots / Videos
Documentation Updates
Additional Notes
Get in Touch
Important
Refactors
AutoApproveMenuinAutoApproveMenu.tsxby reordering elements, removing a setting, and changing UI layout.AutoApproveMenuinAutoApproveMenu.tsxby reordering UI elements.upwhen expanded.This description was created by
for 3b2eaea. You can customize this summary. It will automatically update as commits are pushed.