Page MenuHomePhabricator

[web] Fix modal styles
ClosedPublic

Authored by inka on Wed, Aug 28, 3:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 18, 4:38 AM
Unknown Object (File)
Sun, Sep 15, 10:32 PM
Unknown Object (File)
Sat, Sep 14, 1:07 PM
Unknown Object (File)
Fri, Sep 13, 1:02 PM
Unknown Object (File)
Thu, Sep 12, 12:23 AM
Unknown Object (File)
Mon, Sep 9, 12:57 AM
Unknown Object (File)
Sun, Sep 8, 9:17 AM
Unknown Object (File)
Fri, Sep 6, 2:43 PM
Subscribers
None

Details

Summary

issue: ENG-8679
The issue was that the pinned messages modal div was not scrollable, but fixing this revealed another issue: contents of Modal were wrapped by modalContentContainer which allowed its content to extend it.

Test Plan

Checked ALL places where Modal component is used. Checked that after these changes they all work correctly. Here are modals that got changed:

Here is before and after for the search component (using prod and dev):

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable