Page MenuHomePhorge

Update sprite sheets
ClosedPublic

Authored by MacFan4000 on Oct 4 2022, 14:39.
Tags
Referenced Files
Unknown Object (File)
Tue, Nov 22, 06:30
Unknown Object (File)
Tue, Nov 22, 03:13
Unknown Object (File)
Mon, Nov 21, 01:53
Unknown Object (File)
Fri, Nov 18, 13:38
Unknown Object (File)
Thu, Nov 17, 21:38
Unknown Object (File)
Thu, Nov 17, 07:21
Unknown Object (File)
Wed, Nov 16, 04:57

Details

Summary

This fully updates some login provider icons. Somehow this didn't work fully on previous updates. I used --force this time.

Test Plan

None just ran the script which regenerated the spritesheets.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Cigaryno requested changes to this revision.Oct 6 2022, 19:00

Could you do this:

  • Replace the Phabricator logo with the Phorge logo.
  • Replace the G+ like Google logo with the current Google logo that was used since 2015.

Optional: Replace the Facebook logo with the new one that has a different blue.

This revision now requires changes to proceed.Oct 6 2022, 19:00

Could you do this:

  • Replace the Phabricator logo with the Phorge logo.
  • Replace the G+ like Google logo with the current Google logo that was used since 2015.

Optional: Replace the Facebook logo with the new one that has a different blue.

I am not intending to update actual individual icon files in this commit - only running the script to regenerate the spritesheets based on the individual image files. Actually changing images can be done in a different commit. Also Using outdated logos isn't a big deal. And hard to say if we really want to change the Phabricator logo, as that auth provider could still be used to connect to an old Phabricator instance.

MacFan4000 added a reviewer: Blessed Committers.
avivey added a subscriber: avivey.

We should connect this script (scripts/celerity/*.php) into bin/celerity.

This revision is now accepted and ready to land.Fri, Oct 28, 15:35

Could somebody please land this at some point t?

This revision was automatically updated to reflect the committed changes.