Page MenuHomePhorge

User Badges: better integrate with Removed Comments
ClosedPublic

Authored by valerio.bozzolan on Apr 6 2023, 18:57.
Tags
None
Referenced Files
F3552664: D25112.1744905225.diff
Wed, Apr 16, 15:53
F3528149: D25112.1744848131.diff
Wed, Apr 16, 00:02
F3489782: D25112.1744782468.diff
Tue, Apr 15, 05:47
F3406826: D25112.1744586125.diff
Sat, Apr 12, 23:15
F3404747: D25112.1744577068.diff
Sat, Apr 12, 20:44
F3396414: D25112.1744526311.diff
Sat, Apr 12, 06:38
F3391337: D25112.1744477789.diff
Fri, Apr 11, 17:09
F3390346: D25112.1744465785.diff
Fri, Apr 11, 13:49

Details

Summary

The left part of a Removed Comment can contains User Badges that,
after this change, are more consistent with the new general softness.

BeforeProposed
Phorge Removed Comments User Badges after.png (169×377 px, 24 KB)
Phorge Removed Comments User Badges before.png (169×377 px, 24 KB)

Probably five people in a million will notice this change, BUT,
these people will have a huge, deep, breath of relief, knowing that
Phorge takes care about their obsessive-compulsive impulses, to
have everything nice and consistent and neat. You are welcome!

Closes T15235

Test Plan
  • do something to deserve a Badge
  • share a Comment somewhere (Maniphest?)
  • delete that comment
  • check that the badge is slightly softer

Diff Detail

Repository
rP Phorge
Branch
removed-comments-badges
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 226
Build 226: arc lint + arc unit