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

feat: Added bitbucket icon #484

Closed
wants to merge 1 commit into from
Closed

feat: Added bitbucket icon #484

wants to merge 1 commit into from

Conversation

bdbch
Copy link
Contributor

@bdbch bdbch commented Sep 22, 2018

Added bitbucket icon as requested in #479

Preview:
image

@codecov
Copy link

codecov bot commented Sep 22, 2018

Codecov Report

Merging #484 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #484   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           5      5           
  Lines          34     34           
  Branches        3      3           
=====================================
  Hits           34     34

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8aec53b...c6805a5. Read the comment docs.

@mittalyashu
Copy link
Contributor

I don't know, if it aligns with BitBucket brand guidelines, we need to ask their design team for the same.

@mittalyashu
Copy link
Contributor

Icon Suggestion
Figma_ZcBPs9EDPx Figma_mQvD66Mmp8

@locness3, what do you think?

@locness3
Copy link

idk

@colebemis
Copy link
Member

Thanks for opening this PR, @bdbch! But I'm going to close this because we've decided to stop adding logo icons to Feather (see #763 for more details).

@colebemis colebemis closed this Apr 4, 2020
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.

None yet

4 participants