[web] Add option to change search modal size
ClosedPublic
Actions

Authored by tomek on Apr 7 2022, 3:25 AM.

Details

Reviewers

• benschac
atul
• jacek

Commits

rCOMM291a16c110a5: [web] Add option to change search modal size

Summary

It should be possible to use small and large modals.

Test Plan

Check existing usages - they should look the same. Render a large modal and check if its size is increased.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Apr 7 2022, 3:25 AM

Herald added subscribers: • abosh, • karol, • adrian, ashoat. · View Herald TranscriptApr 7 2022, 3:25 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 7 2022, 3:36 AM

Harbormaster failed remote builds in B7958: Diff 11162!

Harbormaster completed remote builds in B8027: Diff 11234.Apr 8 2022, 7:15 AM

tomek requested review of this revision.Apr 8 2022, 7:15 AM

tomek added a child revision: D3674: [web] Introduce user list component.Apr 8 2022, 7:17 AM

• benschac accepted this revision.Apr 8 2022, 8:06 AM

This revision is now accepted and ready to land.Apr 8 2022, 8:06 AM

atul accepted this revision.Apr 8 2022, 12:23 PM

tomek mentioned this in D3670: [web] Allow passing `Modal` parameters into `SearchModal`.Apr 11 2022, 6:50 AM

Merge imports

This revision was landed with ongoing or failed builds.Apr 15 2022, 2:16 AM

Closed by commit rCOMM291a16c110a5: [web] Add option to change search modal size (authored by tomek). · Explain Why

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM291a16c110a5: [web] Add option to change search modal size.

Harbormaster completed remote builds in B8223: Diff 11497.Apr 15 2022, 8:11 AM

Revision Contents
Changeset List

Path

Size

web/

modals/

search-modal.react.js

7 lines

Diff 11499

View Options

web/modals/search-modal.react.js

[web] Add option to change search modal sizeClosedPublicActions