iTwin / appui

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

Update alert dialog styling (backport #1114) #1117

Closed mergify[bot] closed 1 week ago

mergify[bot] commented 1 week ago

Changes

This PR closes #851 by updating the styling of the alert dialog used via IModelApp.notifications.outputMessage APIs. Dialog title is now changed based on message priority as well.

Before After
Screenshot 2024-11-13 at 13 22 28 Screenshot 2024-11-13 at 14 28 30

Additionally, a select component will be used to display enum arguments of related storybook story.

Testing

Added additional stories.


This is an automatic backport of pull request #1114 done by Mergify.

mergify[bot] commented 1 week ago

Cherry-pick of 30e6cc13085855ec20ca55181dd9b18ccb8128d2 has failed:

On branch mergify/bp/release/5.0.x/pr-1114
Your branch is up to date with 'origin/release/5.0.x'.

You are currently cherry-picking commit 30e6cc130.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
    modified:   common/api/appui-react.api.md
    new file:   common/changes/@itwin/appui-react/fix-851_2024-11-13-12-51.json
    new file:   common/changes/@itwin/core-react/fix-851_2024-11-13-12-51.json
    modified:   docs/storybook/src/Utils.tsx
    modified:   docs/storybook/src/frontstage/notifications/OutputMessage.stories.tsx
    modified:   docs/storybook/src/frontstage/notifications/OutputMessage.tsx
    modified:   e2e-tests/tests/dialog/standard-message-box.test.ts-snapshots/standard-message-box-test-1-chromium-linux.png
    modified:   ui/appui-react/src/appui-react/UiFramework.json
    modified:   ui/appui-react/src/test/dialog/StandardMessageBox.test.tsx
    modified:   ui/core-react/src/core-react/messagebox/MessageBox.scss
    modified:   ui/core-react/src/core-react/messagebox/MessageBox.tsx
    modified:   ui/core-react/vitest.config.ts

Unmerged paths:
  (use "git add <file>..." to mark resolution)
    both modified:   ui/appui-react/src/appui-react/messages/MessageManager.tsx

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally