Skip to content

Add a Safari Pinned Tab Icon #1909

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

Merged
merged 1 commit into from
Aug 7, 2017
Merged

Conversation

ptescher
Copy link
Contributor

@ptescher ptescher commented Aug 3, 2017

@ptescher ptescher changed the title Add a Safari Creating Pinned Tab Icon Add a Safari Pinned Tab Icon Aug 3, 2017
@sg00dwin
Copy link
Member

sg00dwin commented Aug 3, 2017

@pat2man
The mask-icon.svg looks fine. You just need to change the link path from images/openshift-logo.svg to images/mask-icon.svg

@ptescher
Copy link
Contributor Author

ptescher commented Aug 3, 2017

@sg00dwin 😬 rebased.

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

It looks like Travis is complaining about a diff in your dist files. Can you try grunt build again?

<svg width="16px" height="16px" viewBox="0 0 16 16" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
<!-- Generator: Sketch 46 (44423) - http://www.bohemiancoding.com/sketch -->
<title>mask-icon</title>
<desc>Created with Sketch.</desc>
Copy link
Member

Choose a reason for hiding this comment

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

If it's not against the T&C for Sketch, I'd rather remove the description here and generator comment above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for checking that. I'm not worried about it then

Icon adopted from openshift-logo.svg

Reference document:
https://developer.apple.com/library/content/documentation/AppleApplicati
ons/Reference/SafariWebContent/pinnedTabs/pinnedTabs.html
@ptescher
Copy link
Contributor Author

ptescher commented Aug 4, 2017

@spadgett ran grunt build and rebased, passing.

@spadgett
Copy link
Member

spadgett commented Aug 7, 2017

Thanks @pat2man

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 5720905

@openshift-bot
Copy link

openshift-bot commented Aug 7, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/67/) (Base Commit: 7a25f13) (PR Branch Commit: 5720905)

@openshift-bot openshift-bot merged commit 8b2ea90 into openshift:master Aug 7, 2017
@ptescher ptescher deleted the mask-icon branch August 8, 2017 00:39
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.

4 participants