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

feat(Dialog): add new variant confirm #1302

Merged
merged 3 commits into from
Feb 28, 2022
Merged

feat(Dialog): add new variant confirm #1302

merged 3 commits into from
Feb 28, 2022

Conversation

dinarosv
Copy link
Contributor

Added variant confirm to Dialog.
See the Figma file for the different variant design.

@dinarosv dinarosv requested a review from tujoworker February 22, 2022 09:51
@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 22, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b6e1a03:

Sandbox Source
eufemia-starter Configuration

@gatsby-cloud
Copy link

gatsby-cloud bot commented Feb 22, 2022

Gatsby Cloud Build Report

DNB Eufemia Portal

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 9m

Performance

Lighthouse report

Metric Score
Performance 🔶 64
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 92

🔗 View full report

@dinarosv dinarosv force-pushed the feat/dialog-confirm branch 4 times, most recently from 0fce1d7 to 91a2d27 Compare February 23, 2022 08:04
@dinarosv dinarosv force-pushed the feat/dialog-confirm branch from 4575c97 to 4debf44 Compare February 23, 2022 13:09
Copy link
Member

@tujoworker tujoworker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me now. Just a last question, should we provide the Dialog with description or not?

packages/dnb-eufemia/src/components/dialog/Dialog.tsx Outdated Show resolved Hide resolved
@dinarosv
Copy link
Contributor Author

Looks good to me now. Just a last question, should we provide the Dialog with description or not?

Think we should do that. There were also a few other things I will add, that we talked about, before I merge :)

@dinarosv dinarosv force-pushed the feat/dialog-confirm branch 2 times, most recently from a287c58 to 38891ca Compare February 24, 2022 08:36
@dinarosv dinarosv requested a review from tujoworker February 24, 2022 09:32
@dinarosv dinarosv force-pushed the feat/dialog-confirm branch from 921293d to 93b6aae Compare February 28, 2022 07:12
@dinarosv dinarosv force-pushed the feat/dialog-confirm branch from 93b6aae to b6e1a03 Compare February 28, 2022 07:13
@dinarosv dinarosv merged commit bdb7bb4 into main Feb 28, 2022
@dinarosv dinarosv deleted the feat/dialog-confirm branch February 28, 2022 07:35
tujoworker pushed a commit that referenced this pull request Mar 2, 2022
# [9.24.0](v9.23.1...v9.24.0) (2022-03-02)

### Bug Fixes

* **Dialog:** set role alertdialog for confirmation variant ([8ec99af](8ec99af))
* **Dropdown:** fix empty data entry handling ([4e4762b](4e4762b))

### Features

* **Avatar:** add hasLabel to omit group warning ([#1318](#1318)) ([ed43481](ed43481))
* **Dialog:** add new variant confirmation ([#1302](#1302)) ([bdb7bb4](bdb7bb4))
* **FormStatus:** add new marketing state/status ([#1317](#1317)) ([f5e63a9](f5e63a9))
* **NumberFormat:** add "compact" property – format a number compact with an abbreviation ([0e26449](0e26449))
* **Tag:** omit non-existing group warning with new prop: hasLabel ([079cdcd](079cdcd))
@tujoworker
Copy link
Member

🎉 This PR is included in version 9.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants