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

fix: 🐛 Fields scroll behind form modal header #921

Merged
merged 4 commits into from
Aug 24, 2023

Conversation

Mudassar-Raza
Copy link
Contributor

@Mudassar-Raza Mudassar-Raza commented Aug 21, 2023

Fields were going behind from modal header

When viewing a form modal, when you scroll a list of fields, the fields are scrolling on top of the header section.

AC:
When you scroll on a form modal the fields do not show in the header

Loom: https://www.loom.com/share/cc4810764e18427190b28f9cb84317d2?sid=900f374e-e0a9-405b-8363-303fec387d8a

Closes: #920

Fields were going behind from modal header

BREAKING CHANGE: 🧨 N

✅ Closes: #920
@Mudassar-Raza Mudassar-Raza linked an issue Aug 21, 2023 that may be closed by this pull request
@laurelbean
Copy link
Contributor

@Mudassar-Raza If you put anything in the Breaking Change field when making a commit then it treats it like it is a breaking change. Can you please redo the commit and just press enter to skip the Breaking Change option?

Fields scroll behind from modal header

✅ Closes: #920
Fields scroll behind form modal header

✅ Closes: #920
@Mudassar-Raza
Copy link
Contributor Author

@laurelbean I made the commit with skipped breaking changes, kindly review it

@laurelbean laurelbean merged commit eab8299 into master Aug 24, 2023
kaylamc2 pushed a commit that referenced this pull request Aug 24, 2023
### [11.0.2](v11.0.1...v11.0.2) (2023-08-24)

### Bug Fixes

* 🐛 Fields scroll behind form modal header ([#921](#921)) ([eab8299](eab8299)), closes [#920](#920)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug - Fields scroll behind form modal header
4 participants