Skip to content
This repository has been archived by the owner on Jun 14, 2019. It is now read-only.

integration list - what about using 2 rows to show all the data #356

Open
gashcrumb opened this issue May 24, 2019 · 5 comments
Open

integration list - what about using 2 rows to show all the data #356

gashcrumb opened this issue May 24, 2019 · 5 comments

Comments

@gashcrumb
Copy link
Collaborator

This is a...


[x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ ] Documentation issue or request

Description

@dongniwang @seanforyou23 @mcoker

Could we use like 2 rows within a list item to show all the data and stuff we need to show for an integration on the integration list page? For example if we had more than one row in a list item, this name maybe wouldn't have to wrap:

image

and we'd probably have an easier time managing how the app looks at various widths.

@gashcrumb
Copy link
Collaborator Author

If I make it a bit narrower, I do get rows, and look the list items are almost more consistent in height:

image

Name still wraps though. In the old app we truncated strings, but that's such a workaround.

@riccardo-forina
Copy link
Collaborator

riccardo-forina commented May 24, 2019

What if we made use of the expanded state a list item could man have instead? I’m not sure we need all the info all the time on the screen, and we could actually tuck more borrowing from the details page.

Edit: this assuming the expanded state will be supported by pf4 too of course

@dongniwang
Copy link
Contributor

Do we already not show the description in the list view?

Yeah, I was going to suggest we use the expandable list view but not sure if that's what you mean by 2 rows?

The other thing is we can probably update the "configuration required" to just the warning icon with a tooltip.

@gashcrumb
Copy link
Collaborator Author

an expandable list view would be kinda cool, definitely starting with putting the description in there.

What I mean by 2 rows is along the lines of what you see here

@elvisisking
Copy link
Contributor

Would setting the stacked property to true in the ListView.Item work?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants