Page MenuHomePhabricator

Fix comment box borders in timelines after Phriction commenting
ClosedPublic

Authored by epriestley on Sep 18 2018, 2:50 PM.
Tags
None
Referenced Files
F15447849: D19684.id47037.diff
Mar 28 2025, 1:55 AM
F15438205: D19684.id47051.diff
Mar 25 2025, 11:46 PM
F15432647: D19684.id.diff
Mar 24 2025, 7:07 PM
F15432363: D19684.id.diff
Mar 24 2025, 5:48 PM
F15428531: D19684.diff
Mar 23 2025, 8:59 PM
F15427689: D19684.id47037.diff
Mar 23 2025, 3:35 PM
F15425346: D19684.diff
Mar 23 2025, 4:16 AM
F15386242: D19684.id.diff
Mar 15 2025, 12:13 AM
Subscribers
None

Details

Summary

Ref T13202. In D19660, I added comments to Phriction and tweaked some CSS.

One of these tweaks was getting rid of an extra border which was rendering under the comment area. However, I took off too much and ended up removing borders from other applications.

I think we don't actually need this setNoBorder() stuff after all -- a later change was sufficient to stop the actual border I was trying to get rid of from rendering. So this mostly just reverts part of D19660.

This rendering still isn't perfect, but I'm fine leaving that for another day for now.

Test Plan
  • Viewed comment areas in Phriction. Saw correct number of borders (1).
  • Viewed comment areas in Maniphest. Saw correct number of borders (1).
  • Grepped for extraneous removed classs, no hits.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable