Skip to content

Prevent operator icons from stretching in operator subscription page#1150

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
jeff-phillips-18:fixes
Jan 30, 2019
Merged

Prevent operator icons from stretching in operator subscription page#1150
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
jeff-phillips-18:fixes

Conversation

@jeff-phillips-18
Copy link
Copy Markdown
Member

@jeff-phillips-18 jeff-phillips-18 commented Jan 30, 2019

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 30, 2019
@spadgett
Copy link
Copy Markdown
Member

/assign @rhamilto

@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 30, 2019
@rhamilto
Copy link
Copy Markdown
Member

@jeff-phillips-18
Copy link
Copy Markdown
Member Author

I can do that. Or I can create a new style under co-clusterserviceversion-logo__icon in the case where we may want them to differ. Is it a bit odd to add the .co-catalog-item-icon__large class here?

@jeff-phillips-18
Copy link
Copy Markdown
Member Author

I've used the co-catalog-item-icon classes to style it correctly

@rhamilto
Copy link
Copy Markdown
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 30, 2019
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jan 30, 2019
@jeff-phillips-18
Copy link
Copy Markdown
Member Author

Add fix to https://jira.coreos.com/browse/CONSOLE-1237 as well since its in the same realm.

@rhamilto
Copy link
Copy Markdown
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 30, 2019
@openshift-merge-robot openshift-merge-robot merged commit 31e7ee9 into openshift:master Jan 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants