-
Notifications
You must be signed in to change notification settings - Fork 104
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
Appearance of expanded row in list view differs between design and implementation #633
Comments
@kybaker @jennyhaines Just a heads up on this one. I will try to set up a meeting with the three of us and @matthewcarleton to discuss. I think this is more complicated than just changing the Core implementation to reflect the design since this the gray may not play well with what is now a blue hover and selection state. |
@kybaker , @mcarrano , @matthewcarleton , and I just met about this, and the consensus was to have the previously determined #edf8ff blue be the color we use for the list view row color for both hover and expanded/open states. (No grey used whatsoever for list view rows.) |
Just to clarify @jennyhaines . I think that when you say selected state you mean expanded/open, right? According to my notes, we will remove the blue background color for selected rows and use only the checkbox to signify selection. So the target will be equivalent to the current React implementation here: https://rawgit.com/patternfly/patternfly-react/gh-pages/index.html?selectedKind=ListView&selectedStory=List%20of%20expandable%20items Do we all agree with that? |
@mcarrano - yes, correct! I just edited my above comment so there is no confusion. It should be equivalent to the link you provided (except the "fa-angle-down" caret should continue to stay that default dark gray/black color) |
Thanks, @jennyhaines . Also, would you be able to update the mockups on the Overview and Design pages in the pattern doc here to reflect that? http://www.patternfly.org/pattern-library/content-views/list-view/ ? @matthewcarleton I will open a separate issue in the core repo for updating the CSS. |
@mcarrano - yes- working on that now! |
thanks @mcarrano! |
There is a related PR to also implement these changes in PatternFly-Core: patternfly/patternfly#1033 |
There is currently a difference between the design and implementation of an expanded row in List view.
Design specifies that row should be light gray when expanded (http://www.patternfly.org/pattern-library/content-views/list-view/#design). Core implementation shows expanded row as light blue (http://www.patternfly.org/pattern-library/content-views/list-view/#code).
This is further complicated by changes made recently to change hover color from gray to light blue and the interaction of hover and selection. What is the expected result is a row is both open and selected? Need to consider how all of these states work together: hover, selection, and open.
The text was updated successfully, but these errors were encountered: