-
Notifications
You must be signed in to change notification settings - Fork 43
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 app name filter to archetypes page & drawer link #1673
Conversation
5f85287
to
8324593
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On the PR description,
- What part of 0.3.1 priorities #1672 is this PR addressing?
- Can you add a short caption for the screen shots or add an arrow to show what has changed? Relying on my memory of how something currently looks, or trying to run the app to that point with similar data, to decipher the new screenshot is rough to my less-than-ideally-caffeinated brain.
All of the code changes look ok. Really my apprehension is with swapping out a list of archetypes for a single link. Feels like a loss of information in the details drawer.
...rc/app/pages/applications/components/application-detail-drawer/application-detail-drawer.tsx
Outdated
Show resolved
Hide resolved
key: applicationName, | ||
value: applicationName, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Having the key and the value for the selectOptions
be the same is currently annoying me for no good reason. I mean as long as the key value matches up with what is returned from getItemValue()
all is well.
Updated description. Sorry about that!
The concern with the chips in drawer was scalability. Maybe we can find a happy medium to keep some sort of chip list / detail in drawer along with the link. |
Sticking with using labels to display the archetypes, this looks promising: https://www.patternfly.org/components/label#label-group-with-overflow |
@JustinXHale Looking for input here. I think it was Ramon who requested the link instead of chips here since there may be 100s of tags. Even with the overflow, that might be asking too much of our side drawer to render all of those chips in a readable manner. |
Give me a couple days to look into this. I need to pay with some designs and see how we can take scalability into account when there can be a 100 tags |
Specifically for this PR, it is considering the display of the archetypes that an application is associated to. I would be surprised if there are 100 different archetypes for an application, even if the number isn't really limited. The drawer has 3 things for the application archetypes:
Maybe a table layout in another tab like we have in dependency detail drawer? Maybe it is worth thinking about how similar the application drawer's details/archetypes section and tags tabs should look? cc: @JustinXHale |
Wondering if we should consider the inverse case - applications for a given archetype being > 100 applications:
Like this?
Yeah I am not sure what we expect to see for tags. Currently we have this layout which already seems a bit stretched for an app with 40 tags:
|
@ibolton336 i think a label group is great. that was my first thought, just wasnt sure if it would work if its 100 labels. But if that wont be an issue, i think the pattern is perfectly acceptable. We probably need to think about after how many labels to we want to show the more tag, i'm thinking 5. |
4ebdfb3
to
da72937
Compare
Latest changes move the code for link instead of label list from the app drawer to the archetype drawer as outlined here: |
da72937
to
9a73a89
Compare
Signed-off-by: ibolton336 <[email protected]>
Signed-off-by: Ian Bolton <[email protected]>
9a73a89
to
b8f74f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack
There was a suggestion to update the drawer chips to a link since as the list of chips grows, it will not be sustainable to show all chips in drawer. The change here moves to a pre-filter for the archetypes category in the applications table which is linkable from the archetype drawer. https://issues.redhat.com/browse/MTA-2283 <img width="1677" alt="Screenshot 2024-02-21 at 4 51 48 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/f787e32b-df98-48de-92a8-91521059df69"> --------- Signed-off-by: ibolton336 <[email protected]> Signed-off-by: Ian Bolton <[email protected]>
There was a suggestion to update the drawer chips to a link since as the list of chips grows, it will not be sustainable to show all chips in drawer. The change here moves to a pre-filter for the archetypes category in the applications table which is linkable from the archetype drawer.
https://issues.redhat.com/browse/MTA-2283