Page MenuHomePhabricator

[web] Change how margins / paddings are used in message result
ClosedPublic

Authored by inka on Jun 14 2023, 6:20 AM.
Tags
None
Referenced Files
F1669485: D8207.id28151.diff
Sat, Apr 27, 10:32 AM
F1669483: D8207.id27738.diff
Sat, Apr 27, 10:32 AM
F1669482: D8207.id27993.diff
Sat, Apr 27, 10:32 AM
F1669474: D8207.id.diff
Sat, Apr 27, 10:31 AM
F1669459: D8207.diff
Sat, Apr 27, 10:18 AM
Unknown Object (File)
Mar 28 2024, 6:17 AM
Unknown Object (File)
Mar 28 2024, 6:17 AM
Unknown Object (File)
Mar 28 2024, 6:17 AM
Subscribers

Details

Summary

I need to change how margins / paddings are used in message result, because I will be using this component with our SearchModal component that has some default margin. So I move the responsibility of setting the margins / paddings to the outer container.

Additionally the margin in TogglePinModal gets fixed: it seems to me from the designs that there should be a space between the message and the top text. @rohan please tell me if I'm misunderstanding

Test Plan

Before:

After:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka edited the test plan for this revision. (Show Details)
inka added 1 blocking reviewer(s): rohan.
inka requested review of this revision.Jun 14 2023, 6:49 AM

The code makes sense, but I can't actually see the screenshots since they seem to be restricted. Could you attach them please?

This revision is now accepted and ready to land.Jun 20 2023, 6:58 AM
inka edited the summary of this revision. (Show Details)

Rebase