Page MenuHomePhabricator

Update NSIS icons and imagery
ClosedPublic

Authored by movrcx on May 4 2018, 19:03.

Details

Reviewers
schancel
deadalnix
Group Reviewers
Restricted Project
Restricted Owners Package(Owns No Changed Paths)
Commits
rSTAGING48f33b511ae1: Update NSIS icons and imagery
rABC48f33b511ae1: Update NSIS icons and imagery
Summary

Changes Bitcoin Core logos to Bitcoin-abc and Bitcoin Cash

Test Plan

make deploy, attach screenshots

Diff Detail

Repository
rABC Bitcoin ABC
Branch
master
Lint
Lint SkippedExcuse: Might break .xpm files
Unit
No Unit Test Coverage
Build Status
Buildable 2504
Build 3132: Bitcoin ABC Buildbot
Build 3131: arc lint + arc unit

Event Timeline

movrcx created this revision.May 4 2018, 19:03
Owners added a reviewer: Restricted Owners Package.May 4 2018, 19:03
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptMay 4 2018, 19:03
movrcx added a comment.May 4 2018, 19:34

This diff updates the NSIS icons and imagery only. See D1370 for installation directory and uninstall registry key changes.

Where does bitcoin256.png comes from ? The edges are all fubared.

schancel accepted this revision.May 14 2018, 22:59
schancel added a subscriber: schancel.

I'm good with this change, it might be nice to rename the files to indicate that they are not bitcoin logos anymore.

This revision is now accepted and ready to land.May 14 2018, 22:59
movrcx updated this revision to Diff 3972.May 24 2018, 18:08

Recreate icons based off of original vector file

deadalnix accepted this revision.May 25 2018, 14:37
This revision was automatically updated to reflect the committed changes.