Skip to content

Add better description how header title and content slots are aligned #4302

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

Open
wants to merge 1 commit into
base: latest
Choose a base branch
from

Conversation

TatuLund
Copy link
Contributor

No description provided.

Copy link

AI Language Review

The only change made in the Usage file of the AsciiDoc is the addition of the following sentence under the "Header" section:

"When there is title text or element set via API, like in the example below, the content slot is automatically right aligned, otherwise it is left aligned."

This new addition clarifies the alignment behavior of the content slot in the dialog header when a title is set via the API.

No issues require improvement based on this modification.

@TatuLund TatuLund requested a review from rolfsmeds May 23, 2025 05:20
@@ -63,6 +63,8 @@ The Dialog component has static header and footer areas, and a scrolling content

The header contains an optional title element, and a slot next to it for custom header content, such as a close button.

When there is title text or element set via API, like in the example below, the content slot is automatically right aligned, otherwise it is left aligned.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
When there is title text or element set via API, like in the example below, the content slot is automatically right aligned, otherwise it is left aligned.
If the dialog has a title (set via text or custom element), the header content slot is right-aligned. If no title is present, the header content slot is left-aligned.

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.

2 participants