Page MenuHomePhorge

User Badges: better integrate with Removed Comments
ClosedPublic

Authored by valerio.bozzolan on Apr 6 2023, 18:57.
Tags
None
Referenced Files
F2937171: D25112.1737948524.diff
Sun, Jan 26, 03:28
F2933299: D25112.1737760337.diff
Thu, Jan 23, 23:12
F2933282: D25112.1737758793.diff
Thu, Jan 23, 22:46
F2932991: D25112.1737750169.diff
Thu, Jan 23, 20:22
F2932990: D25112.1737750164.diff
Thu, Jan 23, 20:22
F2932981: D25112.1737749572.diff
Thu, Jan 23, 20:12
F2932951: D25112.1737747547.diff
Thu, Jan 23, 19:39
F2929259: D25112.1737654635.diff
Wed, Jan 22, 17:50

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