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

Dialog #20953

Closed
25 of 33 tasks
varholak-peter opened this issue Dec 7, 2021 · 4 comments · Fixed by #25181
Closed
25 of 33 tasks

Dialog #20953

varholak-peter opened this issue Dec 7, 2021 · 4 comments · Fixed by #25181

Comments

@varholak-peter
Copy link
Contributor

varholak-peter commented Dec 7, 2021

Preparation:

Implementation

[link to react-* package folder]

Validation

  • Started validating
  • Add and validate in UI Builder
  • Add and validate in docs site
  • Validate with token pipeline
  • Validate in product
  • Finalize migration guide
    • Author codemods
  • Deliverable: Preview component ready for broader/3rd party use
@layershifter
Copy link
Member

@bsunderhus is there an issue to implement scroll lock in Dialog? (something like https://github.com/theKashey/react-remove-scroll)

@bsunderhus
Copy link
Contributor

bsunderhus commented Aug 12, 2022

@bsunderhus is there an issue to implement scroll lock in Dialog? (something like https://github.com/theKashey/react-remove-scroll)

There isn't at the moment, we could track this possibility! @jurokapsiar

@layershifter
Copy link
Member

@bsunderhus is there an issue to implement scroll lock in Dialog? (something like https://github.com/theKashey/react-remove-scroll)

There isn't at the moment, we could track this possibility! @jurokapsiar

FYI: We have it in v0 (https://fluentsite.z22.web.core.windows.net/0.63.0/components/dialog/definition) so it's about feature parity.

@bsunderhus
Copy link
Contributor

bsunderhus commented Aug 12, 2022

FYI: We have it in v0 (https://fluentsite.z22.web.core.windows.net/0.63.0/components/dialog/definition) so it's about feature parity.

#24355

@msft-fluent-ui-bot msft-fluent-ui-bot added Status: Fixed Fixed in some PR and removed Status: In PR labels Oct 13, 2022
@microsoft microsoft locked as resolved and limited conversation to collaborators Nov 12, 2022
@paulgildea paulgildea added the B20 label Feb 3, 2023
@paulgildea paulgildea changed the title Dialog Convergence Dialog Feb 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.