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

Add gallery to the about page #366

Merged
merged 4 commits into from
Oct 14, 2024
Merged

Add gallery to the about page #366

merged 4 commits into from
Oct 14, 2024

Conversation

fhlavac
Copy link
Collaborator

@fhlavac fhlavac commented Oct 10, 2024

RHCLOUD-35173

closes #324

Added gallery to the "About" page

image

@edonehoo

@patternfly-build
Copy link

patternfly-build commented Oct 10, 2024

includeSubsections={true}
hasGridText={true}
isFullWidth={false}
/>

**Note:** This extension replaces [`RedHatInsights/frontend-components`](https://github.com/RedHatInsights/frontend-components). If you previoulsy used `frontend-components`, you can refer to [this migration guide](https://github.com/patternfly/react-component-groups/blob/main/migration.md) to help you transition to `react-component-groups`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**Note:** This extension replaces [`RedHatInsights/frontend-components`](https://github.com/RedHatInsights/frontend-components). If you previoulsy used `frontend-components`, you can refer to [this migration guide](https://github.com/patternfly/react-component-groups/blob/main/migration.md) to help you transition to `react-component-groups`.
**Note:** This extension replaces [`RedHatInsights/frontend-components`](https://github.com/RedHatInsights/frontend-components). If you previoulsy used `frontend-components`, you can refer to [this migration guide](https://github.com/patternfly/react-component-groups/blob/main/migration.md) to help you transition to `react-component-groups`.

What do you think about moving this up to line 11 or so? Right below the note about the package location. Just to make sure it's easy to spot

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sounds good!

The creation and maintenance of this extension is a collaborative effort between the PatternFly and Red Hat Hybrid Cloud Console teams. All component groups receive accessibility and design reviews, and we are working to improve internationalization, add unit test coverage, and ensure that CSS overrides all correctly reference the PatternFly CSS API.
The creation and maintenance of this extension is a collaborative effort between the PatternFly and Red Hat Hybrid Cloud Console teams. All component groups receive accessibility and design reviews, and we are working to improve internationalization, add test coverage, and ensure that CSS overrides all correctly reference the PatternFly CSS API.

Below, you can see a gallery of categories representing the components:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Below, you can see a gallery of categories representing the components:
There are a few different functional categories of component groups:

<SectionGallery
section="Component groups"
galleryItemsData={galleryData}
placeholderText="Search component groups by name"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
placeholderText="Search component groups by name"
placeholderText="Search component groups by name"

maybe we can just remove the search bar for now, since there's not a lot to filter through? but if that's a lot of work, it's doing no harm by being there!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@edonehoo the gallery component does not allow it yet. We may want to talk to @nicolethoen about the possibility of adding that configuration.

@@ -0,0 +1,17 @@
{
"content-containers": {
"summary": "Provide layout components to organize and display content in structured and flexible formats."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"summary": "Provide layout components to organize and display content in structured and flexible formats."
"summary": "Organize and display information in structured or flexible containers."

"summary": "Provide layout components to organize and display content in structured and flexible formats."
},
"controls": {
"summary": "Include components that manage user interactions and input, enabling intuitive navigation and selection."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"summary": "Include components that manage user interactions and input, enabling intuitive navigation and selection."
"summary": "Enable more effective navigation and selection by providing users with options for interaction and input."

"summary": "Include components that manage user interactions and input, enabling intuitive navigation and selection."
},
"error-communication": {
"summary": "Contain various error and alert states to communicate issues or restrictions."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"summary": "Contain various error and alert states to communicate issues or restrictions."
"summary": "Communicate issues or restrictions through error and alert states."

"summary": "Contain various error and alert states to communicate issues or restrictions."
},
"helpers": {
"summary": "Offer utility components that enhance usability and provide additional guidance or information within the UI."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"summary": "Offer utility components that enhance usability and provide additional guidance or information within the UI."
"summary": "Provide users with utilities that enhance usability and provide additional guidance or information within the UI."

"summary": "Offer utility components that enhance usability and provide additional guidance or information within the UI."
},
"status-and-state-indicators": {
"summary": "Visualize entity states, statuses, and loading indicators to inform users of system conditions or processes."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"summary": "Visualize entity states, statuses, and loading indicators to inform users of system conditions or processes."
"summary": "Inform users of system conditions or processes through visualizations of entity states, statuses, and loading indicators."

@fhlavac
Copy link
Collaborator Author

fhlavac commented Oct 14, 2024

@edonehoo your comments should be addressed, thank you! I also found a way of displaying also the components in the gallery. Just it does not seem to support displaying per category (just all components or none).

@nicolethoen do you think it would be possible to improve the Section Gallery to accept also a subsection? We would like achieve something like this
image
(each components category having a separate gallery with contained components)

@nicolethoen
Copy link
Contributor

@fhlavac I'd say it's definitely possible to improve the Section Gallery, really in any way :)
You could open an issue - though I would wager if you want to see any enhancements in the next few weeks, you might consider contributing the enhancement?

The code lives here:
https://github.com/patternfly/patternfly-org/blob/main/packages/documentation-framework/components/sectionGallery/sectionGallery.js

@fhlavac
Copy link
Collaborator Author

fhlavac commented Oct 14, 2024

@nicolethoen thank you, sounds good. So @edonehoo are we good to merge this as a step 1? Or would you prefer to display all components without categories instead?

I'll then open tickets to enhance SectionGallery and create galleries per category here.

@edonehoo
Copy link
Collaborator

@fhlavac I think this is good to start with, because it provides definitions for the categories, which we don't do anywhere else. Meanwhile, the individual components all at least have docs for context.

Copy link
Collaborator

@edonehoo edonehoo left a comment

Choose a reason for hiding this comment

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

🎉

…roups/about-component-groups.md

Co-authored-by: Erin Donehoo <[email protected]>
@fhlavac fhlavac merged commit 5080157 into patternfly:main Oct 14, 2024
6 checks passed
@fhlavac fhlavac deleted the galleryx branch October 14, 2024 13:14
Copy link

🎉 This PR is included in version 6.0.0-prerelease.6 🎉

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.

Add gallery to the About section
4 participants