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

Why do I see python in the second row here, but nothing else? #2454

Closed
smarterclayton opened this issue Nov 4, 2017 · 16 comments
Closed

Why do I see python in the second row here, but nothing else? #2454

smarterclayton opened this issue Nov 4, 2017 · 16 comments

Comments

@smarterclayton
Copy link
Contributor

image

Very confusing, Needs to be clear that my currently opened drawer has visual dominance over other items in the page. The opacity effect on the other tiles is insufficient to convey that the drawer is dominating them. I would expect the actual background to lose opacity or fade towards the backend.

It took me 5-20 seconds to understand that the other items are disabled tiles, which was way too long and distracted me from my goal of trying to select a target.

@smarterclayton
Copy link
Contributor Author

smarterclayton commented Nov 4, 2017

Also, visually the drawer has a massive amount of whitespace, and the filter row adds a ton more whitespace, to the point where the tiles have no visual connection to the sub selections in the tile.

This page is distracting and does not readily communicate to me what I need to click on. It really needs to be tightened up - the parent tiles need to clearly own the drawer, and the things in the drawer need to feel like individual clickable items. The icons look lost and uncentered in the drawer, and the text below them looks unbalanced (given the spacing of the icons varies wildly).

For a component that is our first connection to a new user, this needs to be a lot tighter and feel more tactile. Nothing in the drawer feels clickable.

@jorgemoralespou
Copy link

@smarterclayton you should search before opening issues ;-)
I think this is a duplicate of #2435

Agree that the service catalog needs some visual improvements.

@spadgett
Copy link
Member

spadgett commented Nov 6, 2017

@openshift/team-ux-review

@smarterclayton
Copy link
Contributor Author

I'm going to use this one because I think my picture is better. :)

@smarterclayton
Copy link
Contributor Author

I think my problem here is that there are only about 400 pixels (20x20 of the chevron) that indicate that the giant white area is tied to PHP. Something like 1% of the screen is telling me what is going on, and that's not enough for such a fundamental action.

@jennyhaines
Copy link

@smarterclayton - I see what you're saying, and I'm going to design a high-fidelity solution around these points.

@serenamarie125
Copy link

@smarterclayton Totally agree that we need a tighter connection between the selected subcategory and connected content. Also, at some point it seems that unselected subcategories are dimmed out, but they look disabled.

@spadgett
Copy link
Member

spadgett commented Nov 6, 2017

@jennyhaines @serenamarie125 Are we only looking at solutions that keeps the tiles? Can we consider other designs as well?

I also find the tiles confusing, and the implementation is complicated and error-prone since it can't be achieved with HTML and CSS alone.

@jennyhaines
Copy link

@spadgett - For the short term (before this Wednesday), I'll work on the connection between subcategories and their associated content. (And I was told you'd fix the opacity issue with the unselected categories).

I'll take a look at an alternative solution to tiles in a later story.

@jennyhaines
Copy link

jennyhaines commented Nov 6, 2017

I think this would work in the short term (to make it into the release by EOD Wednesday) @smarterclayton @spadgett ... The UX group vetted this with me. I plan on looking later into an optimal long term solution that would make it into a future release. What are your thoughts, and would these changes be doable by EOD Wednesday?

subcategories_revised

stacked

@serenamarie125
Copy link

@jennyhaines can you possibly enumerate what you've done here ? I'll take a stab:

  1. unselected subcategory is full color, but has a different tile background
  2. hover state of subcategory has hand cursor, white background, and blue text
  3. selected state of subcategory has white background and blue text & rather than showing a caret, visually connects the 2 rectangles
  4. decreased the top margin above the filter bar
  5. added a horizontal separator below the filter bar

@beanh66
Copy link

beanh66 commented Nov 7, 2017

@serenamarie125 selected state looks like it also has a blue highlight line at the top

@jennyhaines
Copy link

@serenamarie125 @beanh66 - Exactly. Thank you for listing those points out. I've just added a bit more detail here:

  1. unselected subcategory is still full color, but has a grey tile background (#ededed)
  2. hover state of subcategory has hand cursor, white background, and blue text. (no blue top border)
  3. selected state of subcategory has white background and blue text & rather than showing a caret, and the selected tile is visually connected to the white results container.
  4. the selected state tile also has a 2px blue border along the top to differentiate it more.
  5. decreased the top margin above the filter button to 15px
  6. added a horizontal separator 15px below the filter bar (1px, #ededed )

Thank you @rhamilto !

@serenamarie125
Copy link

FYI @spadgett @jwforres, we are using this issue to tackle the immediate need of fixing the visual connection between subcategory and associated content for 3.7. How will we track revisiting the "subcategories as tiles" ? Do we need a trello card or Issue so this doesn't get lost?

@jwforres
Copy link
Member

jwforres commented Nov 7, 2017

openshift-merge-robot added a commit that referenced this issue Nov 7, 2017
Automatic merge from submit-queue.

Update catalog subcategory styles

Partial fix for #2454

![screen shot 2017-11-07 at 2 15 57 pm](https://user-images.githubusercontent.com/895728/32513539-7e9ef31a-c3c8-11e7-94bc-23725fd6812f.PNG)
@spadgett
Copy link
Member

spadgett commented Nov 7, 2017

#2467 with the style changes for 3.7 has merged, and we have a Trello card to revisit this design going forward.

https://trello.com/c/wxJzUBFV

Closing this issue in favor of the Trello card.

@spadgett spadgett closed this as completed Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants