Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

fix: replace default react favicon with tags icon #699

Merged
merged 7 commits into from
Mar 22, 2019

Conversation

elizabethhalper
Copy link
Collaborator

Replace default favicon and electron icon with tags icon

@codecov
Copy link

codecov bot commented Mar 21, 2019

Codecov Report

Merging #699 into dev-preview4 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           dev-preview4     #699   +/-   ##
=============================================
  Coverage         87.18%   87.18%           
=============================================
  Files               117      117           
  Lines              3807     3807           
  Branches            692      692           
=============================================
  Hits               3319     3319           
  Misses              488      488

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 e855394...36c6e81. Read the comment docs.

Copy link
Contributor

@PIC123 PIC123 left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Member

@luisamiranda luisamiranda left a comment

Choose a reason for hiding this comment

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

It works on my machine! 😆

@elizabethhalper elizabethhalper merged commit b77372a into dev-preview4 Mar 22, 2019
@wbreza wbreza deleted the elhalper/16537/tags-icon branch April 8, 2019 21:15
wbreza pushed a commit that referenced this pull request Apr 12, 2019
fix: replace default react favicon with tags icon  (#699)

Get rid of React/Electron's default icons and replace them with the tags icon used throughout the app.

AB#16537
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants