Page MenuHomePhorge

User Badges: better integrate with Removed Comments
ClosedPublic

Authored by valerio.bozzolan on Apr 6 2023, 18:57.
Tags
None
Referenced Files
F3351314: D25112.1743785439.diff
Thu, Apr 3, 16:50
F3351313: D25112.1743785437.diff
Thu, Apr 3, 16:50
F3351312: D25112.1743785435.diff
Thu, Apr 3, 16:50
F3341104: D25112.1743648621.diff
Wed, Apr 2, 02:50
F3334803: D25112.1743554080.diff
Tue, Apr 1, 00:34
F3333870: D25112.1743541192.diff
Mon, Mar 31, 20:59
F3331959: D25112.1743516678.diff
Mon, Mar 31, 14:11
F3331162: D25112.1743502168.diff
Mon, Mar 31, 10:09

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