Fix scrolling to invalid field on invalid form

Open
#5 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
35/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
typescript
Domain
frontend

Research direction

Find the form-validation and scrolling entry point used when saving an invalid form, then reproduce the case where the first invalid field is at the top of the page. Check the current y-position handling around the sticky header; done means scrolling works for both top-of-page and lower invalid fields.

Written by the indexing model from the issue text.

Description

bug enhancement

When attempting to save if the form is invalid, the page will scroll to the first invalid field in the form. The scrolling does not work properly when the field in question is at the top of the page (because of the sticky header). There should be a minimum y value check to enable the scrolling.

Dominant language
TypeScript
Stars
0
Forks
1
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ecrin-github/crPMT

All issues in ecrin-github/crPMT

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.