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 'With icon' variant of accordion #1264

Open
maryshak1996 opened this issue Jun 12, 2023 · 10 comments
Open

Add 'With icon' variant of accordion #1264

maryshak1996 opened this issue Jun 12, 2023 · 10 comments
Labels
Feature end user feature that requires design to define this, i.e. changes or augments the user experience.

Comments

@maryshak1996
Copy link

I'm not sure if this isn't a variant of the 'accordion' or if it's more a variant of the 'expandable' component, but I wanted to contribute this concept that we are using in our console landing page templates (having it be a PF component would make it even easier for folks to adhere to our landing page guidance).
Several examples of usage for this component can be seen in this Sketch here. Here is the component itself, consisiting of ...

  1. Relevant icon
  2. Toggle header
  3. Expandable content
  4. (Optional) Right-aligned quick action in the toggle header
image
@srambach srambach added the Feature end user feature that requires design to define this, i.e. changes or augments the user experience. label Jun 14, 2023
@andrew-ronaldson
Copy link

Thanks for this Mary. I see this as a variant but I'll look more closely at your Sketch file. Are the newly added icons just a visual feature or are they clickable? Saw the blue and brain went to a link.

@maryshak1996
Copy link
Author

@andrew-ronaldson They aren't clickable, but rather a stylistic decision to add some visual interest since this pattern will be used on ConsoleDot to show key features and use cases of a service (so we want to add some visual weight there)

@andrew-ronaldson
Copy link

@mcoker any thoughts on this addition to the component? Seems doable as an optional feature.

@mcoker
Copy link
Contributor

mcoker commented Aug 28, 2023

I can't see the sketch file, but the screenshot looks like an expandable data-list and table, minus the blue border on the left. That border used to be on tables/data-lists, but it was removed.

Adding an icon would be really straight forward, it could be as simple as a prop on an <AccordionItem />, and we just add a new element to support it for spacing/alignment.

Adding additional actions is a bit of a bigger ask since in the accordion, all of the styling on the toggle is on a <button> that serves as the toggle. The actions would need to go beside the toggle, but the toggle should still have some styles on interaction, even when interacting with the adjacent actions, similar to a menu item with actions.

@andrew-ronaldson
Copy link

Correct me if I'm wrong @maryshak1996 but I think this seems like a prop that adds a decorator which is separate from any actions so it would just follow the accordion button. Maybe this would need a tooltip to identify the icon as a specific service?

@maryshak1996
Copy link
Author

hey yes that's correct @andrew-ronaldson :) it's just a decorator icon! I don't even think it would need a tooltip since the text next to it would provide that context

@mcoker
Copy link
Contributor

mcoker commented Aug 29, 2023

@maryshak1996
Copy link
Author

@mcoker yup exactly! If it was possible to choose which side (left or right) the expandable icon could be located on the accordion toggle that would be great, but definitely not a hill I'm going to die on! When the accordion is at full-width, it feels like the expandable toggle is pretty far away from the title it's expanding is all.

@mcoker
Copy link
Contributor

mcoker commented Aug 29, 2023

@maryshak1996 cool! @andrew-ronaldson both of those enhancements should be pretty straight forward.

@andrew-ronaldson
Copy link

@lboehling @blfetche do we want to get this in the penta work?

@andrew-ronaldson andrew-ronaldson added this to the Prioritized backlog milestone Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature end user feature that requires design to define this, i.e. changes or augments the user experience.
Projects
Status: Ready to assign
Development

No branches or pull requests

4 participants