Update default wordmark and logo
ClosedPublic
Actions

Authored by avivey on Aug 22 2022, 19:12.

Details

Reviewers

speck

Group Reviewers

O1: Blessed Committers

Commits

rP377ac059d6da: Update default wordmark and logo

Summary

The current default wordmark is "Phabricator" which is trademarked and the
default logo is also copyright.

(This change was made by @speck directly in the deployed instnace, bringing it into master now).

Test Plan

should be fine.

Diff Detail

Repository

rP Phorge

Branch

phorge (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 80
Build 80: arc lint + arc unit

Event Timeline

avivey created this revision.Aug 22 2022, 19:12

Owners added a reviewer: O1: Blessed Committers.Aug 22 2022, 19:12

Herald added subscribers: Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptAug 22 2022, 19:12

avivey requested review of this revision.Aug 22 2022, 19:12

Harbormaster completed remote builds in B78: Diff 257.Aug 22 2022, 19:12

looks like the favicon doesn't actually work :/

Could it be cached?

webroot/rsrc/css/application/base/main-menu-view.css
50	There’s probably an html file that needs to use this updated css name

celerify

Harbormaster completed remote builds in B79: Diff 258.Aug 22 2022, 19:15

I didn't want to do this, but I'm actually going to test this change.

Favicons are somehow cached server side:
<link rel="icon" id="favicon" href="http://localhost.localhost:8080/file/data/jvpvboup37hwjgpn3viv/PHID-FILE-xhmdncm6ydmubbkrfdj4/favicon" />

And they are actually located at https://we.phorge.it/source/phorge/browse/master/resources/builtin/favicon/ , not this file.
I'll remove this piece for now - I think it's not actually trademarked/copyrighted, so we can keep it for a little while longer.