iTwin / appui

Monorepo for iTwin.js AppUi
MIT License
8 stars 2 forks source link

Dialog css layer fix #885

Closed ignas-k closed 3 weeks ago

ignas-k commented 3 weeks ago

Changes

This reverts the unwanted change (introduced in #883) which would make modal dialogs use iTwinUI v2 styling. Apparently, iTwinUI Modal has different behaviour from Dialog which causes problems with our theme manager because we support v2 theme providers. So for now I reverted to using Dialog instead of Modal. We can try using Modal in AppUI 5.0 when we stop supporting iTwinUI v2.

Testing

Tested in standalone test-app.