Page MenuHomePhorge

User Badges: better integrate with Removed Comments
ClosedPublic

Authored by valerio.bozzolan on Apr 6 2023, 18:57.
Tags
None
Referenced Files
F3313388: D25112.1743217318.diff
Fri, Mar 28, 03:01
F3300232: D25112.1743036279.diff
Wed, Mar 26, 00:44
F3300180: D25112.1743034756.diff
Wed, Mar 26, 00:19
F3299906: D25112.1743031092.diff
Tue, Mar 25, 23:18
F3296266: D25112.1742970502.diff
Tue, Mar 25, 06:28
F3224881: D25112.1742089485.diff
Sat, Mar 15, 01:44
F3220514: D25112.1741810324.diff
Tue, Mar 11, 20:12
F3220513: D25112.1741810322.diff
Tue, Mar 11, 20:12

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