Paths

Table of Contentst

[web] First pass at styling `ConcurrentModificationModal`
ClosedPublic
Actions

Authored by atul on Apr 11 2022, 3:56 PM.

Details

Reviewers

tomek
• jacek

Commits

rCOMM7c0ad9be2cb9: [web] First pass at styling `ConcurrentModificationModal`

Summary

Started with styles borrowed from thread-settings-modal.css and made some tweaks so it looks decent.

Before:

After:

Test Plan

Replaced InvalidUploadModal in InputStateContainer with ConcurrentModificationModal to make it easy
Dragged SVG into message list
ConcurrentModificationModal looked as expected

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Apr 11 2022, 3:56 PM

Herald added subscribers: • abosh, • benschac, • karol and 2 others. · View Herald TranscriptApr 11 2022, 3:56 PM

atul edited the summary of this revision. (Show Details)Apr 11 2022, 3:57 PM

.ok_button => .refresh_button

atul published this revision for review.Apr 11 2022, 3:58 PM

Harbormaster completed remote builds in B8101: Diff 11328.Apr 11 2022, 4:01 PM

Harbormaster failed remote builds in B8102: Diff 11329!Apr 12 2022, 8:29 AM

Harbormaster completed remote builds in B8102: Diff 11329.Apr 12 2022, 8:38 AM

tomek accepted this revision.Apr 13 2022, 4:23 AM

This revision is now accepted and ready to land.Apr 13 2022, 4:23 AM

rebase before landing

Harbormaster completed remote builds in B8166: Diff 11431.Apr 13 2022, 10:32 AM

Closed by commit rCOMM7c0ad9be2cb9: [web] First pass at styling `ConcurrentModificationModal` (authored by atul). · Explain WhyApr 13 2022, 10:33 AM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM7c0ad9be2cb9: [web] First pass at styling `ConcurrentModificationModal`.

Revision Contents
Changeset List

Path

Size

web/

modals/

concurrent-modification-modal.css

14 lines

concurrent-modification-modal.react.js

16 lines

Diff 11436

View Options

web/modals/concurrent-modification-modal.css