Skip to content

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Oct 3, 2017

@spadgett
Copy link
Member Author

spadgett commented Oct 4, 2017

Icons for subcategories:

openshift web console 2017-10-04 09-43-19

@sspeiche
Copy link

sspeiche commented Oct 4, 2017

Ooohhhh purdy

@spadgett
Copy link
Member Author

spadgett commented Oct 5, 2017

@jeff-phillips-18 PTAL

@benjaminapetersen
Copy link
Contributor

[test] since Jenkins file changed

@benjaminapetersen
Copy link
Contributor

dist on this one, not tests

@spadgett
Copy link
Member Author

spadgett commented Oct 5, 2017

dist on this one, not tests

Yeah, needs a common and catalog release before I can add dist

@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Oct 6, 2017
Use color SVG icons when available for recognized icon classes,
including those from openshift-logos.
@openshift-bot
Copy link

Evaluated for origin web console test up to ad0d847

@spadgett
Copy link
Member Author

spadgett commented Oct 9, 2017

[merge]

@spadgett spadgett changed the title [WIP] Add SVG icons Add SVG icons Oct 9, 2017
@openshift-bot
Copy link

Evaluated for origin web console merge up to ad0d847

@openshift-bot
Copy link

openshift-bot commented Oct 9, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/326/) (Base Commit: eb33544) (PR Branch Commit: ad0d847)

@openshift-bot
Copy link

Origin Web Console Test Results: FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_web_console/312/) (Base Commit: eb33544) (PR Branch Commit: ad0d847)

@openshift-bot openshift-bot merged commit 2ef8242 into openshift:master Oct 9, 2017
@spadgett spadgett deleted the logos branch October 9, 2017 16:37
openshift-merge-robot added a commit that referenced this pull request Nov 1, 2017
Automatic merge from submit-queue.

Fix bug where custom img icons don't appear in catalog cards

Introduced with #2208

![screen shot 2017-11-01 at 4 49 26 pm](https://user-images.githubusercontent.com/895728/32297259-fe6cad74-bf24-11e7-9e96-e157228b4798.PNG)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants