Skip to content

Conversation

@bucknatt
Copy link
Contributor

@bucknatt bucknatt commented Sep 11, 2025

📄 Description

  • Added <p>-tag around dialog content in stories, required for accessibility.
  • Separated <button>-tags to be on two different lines.

@bucknatt bucknatt requested review from a team as code owners September 11, 2025 13:51
@bucknatt bucknatt requested a review from leagrdv September 11, 2025 13:51
@bucknatt bucknatt linked an issue Sep 11, 2025 that may be closed by this pull request
@changeset-bot
Copy link

changeset-bot bot commented Sep 11, 2025

🦋 Changeset detected

Latest commit: 78ef21c

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

This PR includes changesets to release 1 package
Name Type
@swisspost/design-system-documentation Patch

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 Sep 11, 2025

Related Previews

@bucknatt bucknatt marked this pull request as draft September 12, 2025 08:17
@leagrdv leagrdv removed their request for review September 17, 2025 06:48
…es with `flexdirection: column` on dialog styles
@bucknatt bucknatt marked this pull request as ready for review September 18, 2025 11:59
@alionazherdetska alionazherdetska self-requested a review September 24, 2025 12:42
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.

The code snippet is still showing the two buttons on a single line:
image

@bucknatt bucknatt requested a review from alizedebray October 1, 2025 06:35
…on two seperated lines in a html formatting context
@bucknatt bucknatt requested a review from alizedebray October 7, 2025 07:37
@sonarqubecloud
Copy link

@bucknatt bucknatt merged commit c51cf55 into main Oct 14, 2025
13 checks passed
@bucknatt bucknatt deleted the 6180-fix-dialog-markup branch October 14, 2025 06:58
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.

Fix Dialog Markup

4 participants