Skip to content

chore(Button): updated how ariaexpanded and class are set - #7807

Merged
mcoker merged 2 commits into
patternfly:mainfrom
thatblindgeye:buttonAriaExpandedReplace
Sep 8, 2025
Merged

mcoker merged 2 commits into
patternfly:mainfrom
thatblindgeye:buttonAriaExpandedReplace

Conversation

@thatblindgeye

Copy link
Copy Markdown
Contributor

No description provided.

@thatblindgeye
thatblindgeye requested a review from mcoker September 8, 2025 13:04
@patternfly-build

patternfly-build commented Sep 8, 2025 •

Copy link
Copy Markdown
Collaborator

@mcoker mcoker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - just one question on the use of concat.

{{/if}}
{{#if (concat button--IsExpanded button--IsAriaExpanded)}}
aria-expanded="{{button--IsExpanded}}"
{{#if (concat button--AriaExpanded)}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you still need the concat helper?

Suggested change
{{#if (concat button--AriaExpanded)}}
{{#if button--AriaExpanded}}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Only for you to catch it 😭

button--IsExpanded=table-tr--IsExpanded
button--IsAriaExpanded=table-tr--IsExpanded
button--AriaExpanded=table-tr--IsExpanded
button--modifier=(ternary table-tr--IsExpanded "pf-m-expanded" "")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@mcoker mcoker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

send it!

@mcoker
mcoker merged commit ed6d9be into patternfly:main Sep 8, 2025
4 checks passed
@patternfly-build

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 6.3.0-prerelease.60 🎉

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants