Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Add alternative object details designs #65

Merged
merged 1 commit into from
Jan 1, 2019

Conversation

cshinn
Copy link
Contributor

@cshinn cshinn commented Nov 29, 2018

This doc has a few designs for object details pages (service page specifically, but something very similar could be done for workloads, etc).
Each step builds on the previous one, so we can take the changes as far as makes sense at a given time. This PR does not add new content (with the exception of the missing sidecar indicator) but is a rearrangement of existing things on the page

Closes #61

Copy link
Contributor

@abonas abonas left a comment

Choose a reason for hiding this comment

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

please squash the typo fix commit with previous one :)

- Move health into header (details still available on hover)
- More space for ports and endpoints lists

## Big changes big difference
Copy link
Contributor

Choose a reason for hiding this comment

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

this is great as well, I think it should be a different item. As I discussed with @serenamarie125 I'm in favor of this, I'd like to understand better how long does it take to convert pf3 to pf4. Also worth to understand - why pf3 is not getting this style change so all projects can enjoy it "for free"? With the current situation, every project has to do separate work to make it look like this.

Copy link
Contributor

Choose a reason for hiding this comment

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

@abonas we need to set up a meeting with the appropriate PatternFly team members. Who are the appropriate people to include on the Service Mesh side?

Copy link
Contributor

Choose a reason for hiding this comment

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

@abonas we need to set up a meeting with the appropriate PatternFly team members. Who are the appropriate people to include on the Service Mesh side?

it depends what is the goal of the meeting - can you elaborate?


## Sidecar field
![](img/sidecar-suggestion.png)
- We could add a `sidecar` field to the item details to illustrate whether the sidecar is missing or represent
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this it the less optimal place to be part of, and at the bottom of the details.
It is also conceptually not "the same" as the other details here, because it is something "applied to" and not really "part of" the current entity.

Copy link
Contributor

Choose a reason for hiding this comment

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

I am in favor of either putting the sidecar indicator in line with the header or below the header, but not as part of details section.

## Moderate style changes
Adds more space to the page and brings it in line with other cloud software

![](img/moderate-changes.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

LOVE that. can you elaborate on the technical aspect? where do we get details about which font/what kind of styling to apply?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This view is essentially using the exact same styles as OKD uses for its object detail views. I'm not sure how difficult it would be to reuse their stylesheets, but I'm sure we can put together a spec if this is the road we want to go down

![](img/sidecar-suggestion.png)
- We could add a `sidecar` field to the item details to illustrate whether the sidecar is missing or represent

## Minor page structure improvements
Copy link
Contributor

Choose a reason for hiding this comment

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

@cshinn I will add another story for you to investigate a design utilizing the PF3 React data grid component rather than the list view which is currently being used. I think this would also be helpful, what do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That sounds good. Is that the component used in OKD right now?

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@serenamarie125 serenamarie125 merged commit 289cd05 into kiali:master Jan 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants