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): adding styles for native dialog #2772

Merged
merged 35 commits into from
Oct 8, 2024
Merged

Conversation

gfellerph
Copy link
Member

@gfellerph gfellerph commented Mar 7, 2024

Styles for the dialog element, replacing ngb-modal.

Copy link

changeset-bot bot commented Mar 7, 2024

🦋 Changeset detected

Latest commit: 3021bb0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
@swisspost/design-system-documentation Minor
@swisspost/design-system-styles Minor
@swisspost/design-system-components-angular-workspace Patch
@swisspost/design-system-components Minor
@swisspost/design-system-demo Patch
@swisspost/internet-header Patch
@swisspost/design-system-intranet-header-workspace Patch
@swisspost/design-system-nextjs-integration Patch
@swisspost/design-system-styles-primeng-workspace Patch
@swisspost/design-system-intranet-header Minor
@swisspost/design-system-styles-primeng Minor
@swisspost/design-system-components-react Minor
@swisspost/design-system-components-angular Minor
@swisspost/design-system-intranet-header-showcase Patch
@swisspost/design-system-tokens Minor
@swisspost/design-system-icons Minor
@swisspost/design-system-migrations Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@swisspost-bot
Copy link
Contributor

swisspost-bot commented Mar 7, 2024

Preview environment ready: https://preview-2772--swisspost-web-frontend.netlify.app
Preview environment ready: https://preview-2772--swisspost-design-system-next.netlify.app

transform: scale(1);
opacity: 1;

@starting-style {
Copy link
Member Author

Choose a reason for hiding this comment

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

Both Safari (WebKit/standards-positions#210) and Firefox (mozilla/standards-positions#833) are supportive of the spec, therefore I consider it safe to use for progressive enhancement.

@gfellerph gfellerph linked an issue Apr 15, 2024 that may be closed by this pull request
@swisspost-bot
Copy link
Contributor

swisspost-bot commented May 29, 2024

Related Previews

Copy link

sonarcloud bot commented May 29, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
2 New Code Smells (required ≤ 0)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@gfellerph gfellerph added the v9 label Jul 22, 2024
@gfellerph gfellerph marked this pull request as ready for review August 9, 2024 13:27
@gfellerph gfellerph requested a review from a team as a code owner August 9, 2024 13:27
@gfellerph gfellerph added v8 and removed v9 labels Aug 9, 2024
@alizedebray
Copy link
Contributor

@gfellerph Is this still a draft?

@gfellerph
Copy link
Member Author

@gfellerph Is this still a draft?

no

Copy link

socket-security bot commented Aug 15, 2024

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

Copy link
Contributor

@alizedebray alizedebray left a comment

Choose a reason for hiding this comment

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

Nice! Happy to say goodbye to two ngb components at once 🌟

@gfellerph gfellerph removed the v8 label Sep 10, 2024
Copy link
Contributor

@alizedebray alizedebray left a comment

Choose a reason for hiding this comment

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

Looks great!
There is only this small display issue with the last example:
image

@gfellerph gfellerph mentioned this pull request Oct 8, 2024
Copy link

sonarcloud bot commented Oct 8, 2024

@gfellerph gfellerph merged commit da7b10f into main Oct 8, 2024
11 checks passed
@gfellerph gfellerph deleted the 1536-component-dialog branch October 8, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Component: Dialog
3 participants