Skip to content

Improve provider selection page#8286

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
liggitt:idp-selection
Mar 30, 2016
Merged

Improve provider selection page#8286
openshift-bot merged 1 commit intoopenshift:masterfrom
liggitt:idp-selection

Conversation

@liggitt
Copy link
Copy Markdown
Contributor

@liggitt liggitt commented Mar 30, 2016

@liggitt
Copy link
Copy Markdown
Contributor Author

liggitt commented Mar 30, 2016

before:
screen shot 2016-03-29 at 10 46 42 pm

after:
screen shot 2016-03-29 at 10 44 38 pm

also, this changes the background to be darker on hover, which makes the darker hover link more readable

@liggitt
Copy link
Copy Markdown
Contributor Author

liggitt commented Mar 30, 2016

[test]

@spadgett
Copy link
Copy Markdown
Member

Code changes LGTM, definitely an improvement.

Just a small note that Patternfly styles say to use square borders instead of rounded.

@liggitt
Copy link
Copy Markdown
Contributor Author

liggitt commented Mar 30, 2016

squared border, [merge]

@openshift-bot
Copy link
Copy Markdown
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5477/) (Image: devenv-rhel7_3871)

@openshift-bot
Copy link
Copy Markdown
Contributor

Evaluated for origin merge up to 4858d15

@liggitt
Copy link
Copy Markdown
Contributor Author

liggitt commented Mar 30, 2016

flake on #8303, re[test]

@openshift-bot
Copy link
Copy Markdown
Contributor

Evaluated for origin test up to 4858d15

@openshift-bot
Copy link
Copy Markdown
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/2604/)

@openshift-bot openshift-bot merged commit ea6653c into openshift:master Mar 30, 2016
@liggitt liggitt deleted the idp-selection branch April 1, 2016 18:22
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