Page MenuHomePhabricator
Files F3507827

D10344.id34688.diff
No OneTemporary
Actions

D10344.id34688.diff
View Options

diff --git a/web/modals/chat/message-results-modal.css b/web/modals/chat/message-results-modal.css
--- a/web/modals/chat/message-results-modal.css
+++ b/web/modals/chat/message-results-modal.css
@@ -1,3 +1,7 @@
+.container {
+ height: 80vh;
+}
+
hr.separator {
border: 0;
margin: 20px 0 0 0;
@@ -8,12 +12,10 @@
}
.messageResultsContainer {
+ display: flex;
+ flex-direction: column;
overflow-y: scroll;
- padding: 0 32px 8px 32px;
-}
-
-.messageResultsContainer > * {
- margin-bottom: 16px;
+ row-gap: 16px;
}
.loadingIndicator {
@@ -21,7 +23,7 @@
}
.topSpace {
- height: 48px;
+ height: 8px;
align-items: center;
justify-content: center;
display: flex;
diff --git a/web/modals/chat/message-results-modal.react.js b/web/modals/chat/message-results-modal.react.js
--- a/web/modals/chat/message-results-modal.react.js
+++ b/web/modals/chat/message-results-modal.react.js
@@ -148,12 +148,20 @@
return null;
}, [loadingStatus]);
+ const subheader = React.useMemo(() => <hr className={css.separator} />, []);
+
return (
- <Modal name={modalName} onClose={popModal} size="large">
- <hr className={css.separator} />
- <div className={css.topSpace}>{loadingIndicator}</div>
- <div className={css.messageResultsContainer}>
- {messageResultsToDisplay}
+ <Modal
+ name={modalName}
+ onClose={popModal}
+ size="large"
+ subheader={subheader}
+ >
+ <div className={css.container}>
+ <div className={css.topSpace}>{loadingIndicator}</div>
+ <div className={css.messageResultsContainer}>
+ {messageResultsToDisplay}
+ </div>
</div>
</Modal>
);

File Metadata

Mime Type
text/plain
Expires
Sat, Dec 21, 9:37 PM (20 h, 21 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2689281
Default Alt Text
D10344.id34688.diff (1 KB)

Event Timeline