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

Author detail extravaganza #908

Merged
merged 2 commits into from
Jul 17, 2018

Conversation

newswangerd
Copy link
Member

Fixes: #721
Fixes: #819

@newswangerd newswangerd requested a review from chouseknecht July 16, 2018 20:42
@chouseknecht
Copy link
Contributor

@newswangerd

Vast improvement. Like it!

Please review with @trahman73. I think he may want to tweak some of the small screen sizes views. Maybe add some padding around some of the things via media queries.

Copy link
Contributor

@chouseknecht chouseknecht left a comment

Choose a reason for hiding this comment

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

LGTM, once @trahman73 reviews

@newswangerd
Copy link
Member Author

Large:
screen shot 2018-07-17 at 9 42 02 am

Medium:
screen shot 2018-07-17 at 9 42 21 am

Small:
screen shot 2018-07-17 at 9 42 42 am

@trahman73
Copy link

Large and medium views look good! The mobile view is limited by PatternFly's 3.x framework which they are addressing in PF4.

@newswangerd newswangerd merged commit b8c00ff into ansible:devel Jul 17, 2018
@newswangerd newswangerd deleted the author-detail-extravaganza branch July 17, 2018 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants