8000 fix(expandable-section): adjust styles, icon, update screenshots by mcoker · Pull Request #7006 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

fix(expandable-section): adjust styles, icon, update screenshots#7006

Merged
mattnolting merged 3 commits intopatternfly:v6from
mcoker:backstop-8.26.24
Aug 28, 2024
Merged

fix(expandable-section): adjust styles, icon, update screenshots#7006
mattnolting merged 3 commits intopatternfly:v6from
mcoker:backstop-8.26.24

Conversation

@mcoker
Copy link
Contributor
@mcoker mcoker commented Aug 27, 2024

fixes #7007

I was updating the backstop screenshots and noticed these issues with multiple file upload/expandable section. This PR

  • Removes align-items: start on expandable section to allow both the toggle and content elements to occupy the width of their container
  • Updates the loading pficon in the toggle to use a FA icon. The pficon was throwing off the toggle alignment
  • Updates backstop images visual regression screenshots. Mostly just the expandable section changes that have already been reviewed and signed off on. The border in one of the wizard screenshots has errored before, I suspect that's an occasional rendering issue/false positive. And looks like there may have been a missing screenshot for the other wizard one?

@mcoker mcoker changed the title Backstop 8.26.24 fix(expandable-section): adjust styles, icon, update screenshots Aug 27, 2024
@patternfly-build
Copy link
Collaborator
patternfly-build commented Aug 27, 2024

{{{multiple-file-upload-status-progress-icon--attribute}}}
{{/if}}>
<i class="pf-v6-pficon pf-v6-pficon-in-progress"></i>
{{! <i class="pf-v6-pficon pf-v6-pficon-in-progress"></i> }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Did we want to keep this comment to revert back to this element in the future?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@thatblindgeye yep, though I just updated the comment to add TODO text that describes why it's commented.

@kmcfaul kmcfaul linked an issue Aug 27, 2024 that may be closed by this pull request
@mcoker mcoker requested a review from mattnolting August 28, 2024 14:22
@andrew-ronaldson andrew-ronaldson self-requested a review August 28, 2024 15:47
Copy link
Collaborator
@andrew-ronaldson andrew-ronaldson left a comment

Choose a reason for hiding this comment

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

Cool beans

@mattnolting mattnolting merged commit 666c754 into patternfly:v6 Aug 28, 2024
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.226 🎉

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.

Expandable section bugs in multiple file upload

5 participants

0