Change Details

Looks like this property was introduced in D3669. I understand fixing the width of the modals, but I think we can let the height expand to fit the content? It doesn't seem right to arbitrarily cap the possible height of a Modal at this level of abstraction? I understand there might be the need to cap it for specific Modals? For example, here's the `ThreadSettingsModal` before and after this diff: Before: {F49119} After (and before D3669): {F49120}

Looks like this property was introduced in D3669. I understand fixing the width of the modals, but I think we can let the height expand to fit the content? It doesn't seem right to arbitrarily cap the possible height of a Modal at this level of abstraction? I understand there might be the need to cap it for specific Modals? For example, here's the `ThreadSettingsModal` before and after this diff: Before: {F49120} After (and before D3669): {F49119}

Looks like this property was introduced in D3669. I understand fixing the width of the modals, but I think we can let the height expand to fit the content? It doesn't seem right to arbitrarily cap the possible height of a Modal at this level of abstraction? I understand there might be the need to cap it for specific Modals? For example, here's the `ThreadSettingsModal` before and after this diff: Before: {F49119}20} After (and before D3669): {F49120}19}