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

Disable background scroll when Dialog open #3906

Open
1 task done
liamsms opened this issue Jul 29, 2024 · 16 comments · May be fixed by #3907
Open
1 task done

Disable background scroll when Dialog open #3906

liamsms opened this issue Jul 29, 2024 · 16 comments · May be fixed by #3907
Assignees

Comments

@liamsms
Copy link
Contributor

liamsms commented Jul 29, 2024

Area

UI Components

The problem

The current behavior of the Dialog component allows for background scrolling when the Dialog is open. This can sometimes lead to unintentionally scrolling the background content while interacting with the Dialog. It would be nice to have an option to disable this.

The solution

Add a new prop to Dialog which disables background scrolling. eg disableBackgroundScrolling: boolean

Alternatives and examples

.

Are you a JPMorgan Chase & Co. employee?

  • I am an employee of JPMorgan Chase & Co.
@liamsms liamsms added status: awaiting triage Automatically added to new issues. Should be removed once they have been triaged. type: enhancement 💡 New feature or request labels Jul 29, 2024
@joshwooding
Copy link
Contributor

This seems like a defect and should be fixed.

@liamsms liamsms linked a pull request Jul 29, 2024 that will close this issue
@mark-tate
Copy link
Contributor

@tomhazledine @Fercas123 for awareness of the issue, what is our recommended solution

@mark-tate
Copy link
Contributor

grouped dialog issues into one sprint, needs further grooming, so dates subject to change

@mark-tate
Copy link
Contributor

Latte Goal: review PR as it overlaps with Overlay

@dplsek
Copy link

dplsek commented Oct 25, 2024

Pending update from @Fercas123 moving to Lungo until we know otherwise

@mark-tate
Copy link
Contributor

Lungo Goal: review PR

@mark-tate
Copy link
Contributor

Contribution:

Macc: follow up with Liam

@mark-tate
Copy link
Contributor

Mocha Goal: follow up with Liam

@mark-tate
Copy link
Contributor

one more dev review required... has been updated by Liam with our feedback

@bhoppers2008
Copy link

(ZC) Reviewed but changes required. No further action this sprint.

@mark-tate
Copy link
Contributor

Red Eye: feedback needs to be added to PR by EOS

@dplsek
Copy link

dplsek commented Jan 9, 2025

@mark-tate to update when he returns

@joshwooding joshwooding removed the status: awaiting triage Automatically added to new issues. Should be removed once they have been triaged. label Jan 10, 2025
@bhoppers2008
Copy link

@joshwooding to pick up.

@mark-tate
Copy link
Contributor

Tecno Goal: review and turn into a known state by EOS (@joshwooding)

@mark-tate
Copy link
Contributor

PR review in progress

@mark-tate
Copy link
Contributor

will be reviewed by EOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

5 participants