Fixing scroll references for Advanced and Security settings #19771
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.
Related: #19479
Explanation
When anchoring to a settings search result that exists on the Advanced and Security tabs, many of the settings weren't scrolled to correctly, or at all. This is because there was an off by one error in the references ordering in the tab components when compared to the route positions in the settings constants. This PR fixes those off by ones, and in addition for Security constants, standardizes and fixes misspellings in the hash routes.
Before
(See video in referenced issue)
After
settings-fix.mov
Manual Testing Steps
Show balance and token price checker
,Show hex data
,Choose your network
, etc)Pre-merge author checklist
Pre-merge reviewer checklist
If further QA is required (e.g. new feature, complex testing steps, large refactor), add the
Extension QA Board
label.In this case, a QA Engineer approval will be be required.
cc: @NiranjanaBinoy