Skip to content

Commit 41bc572

Browse files
review comments
1 parent 693d470 commit 41bc572

File tree

3 files changed

+18
-13
lines changed

3 files changed

+18
-13
lines changed

frontend/src/components/Dialog.module.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,6 @@
4343
}
4444

4545
.scroll {
46-
padding: 10px;
46+
padding: 20px;
4747
overflow-y: auto;
4848
}

frontend/src/components/Feedback.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import Alert from "@/components/Alert";
1111
import Button from "@/components/Button";
1212
import Dialog from "@/components/Dialog";
1313
import Flex from "@/components/Flex";
14+
import Help from "@/components/Help";
1415
import Link from "@/components/Link";
1516
import TextBox from "@/components/TextBox";
1617
import { userAgent } from "@/util/browser";
@@ -174,17 +175,19 @@ const Feedback = () => {
174175
type="submit"
175176
/>
176177
)}
178+
<div />
177179
<Button
178180
text="Screenshot"
179181
icon={<FaDownload />}
180182
design="hollow"
181-
tooltip="Download a screenshot of the current page. Attach to your created discussion to help us debug."
183+
tooltip="Download a screenshot of the current page"
182184
onClick={async () => {
183185
close();
184186
await downloadJpg(document.body, ["screenshot.jpg"]);
185187
open();
186188
}}
187189
/>
190+
<Help tooltip="A screenshot of the current page can help us troubleshoot issues. Currently, we can't automatically attach a screenshot with your feedback, so you'll have to download it and attach/send it to us manually." />
188191
</Flex>
189192
</form>
190193
)}

frontend/src/components/TextBox.tsx

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -71,17 +71,19 @@ const TextBox = ({
7171
if (text || value)
7272
sideElement = (
7373
<div ref={sideRef} className={classes.side}>
74-
<button
75-
className={classes["side-button"]}
76-
type="button"
77-
onClick={async () => {
78-
await window.navigator.clipboard.writeText(text);
79-
toast("Copied text", "success");
80-
}}
81-
aria-label="Copy text"
82-
>
83-
<FaRegCopy />
84-
</button>
74+
{multi && (
75+
<button
76+
className={classes["side-button"]}
77+
type="button"
78+
onClick={async () => {
79+
await window.navigator.clipboard.writeText(text);
80+
toast("Copied text", "success");
81+
}}
82+
aria-label="Copy text"
83+
>
84+
<FaRegCopy />
85+
</button>
86+
)}
8587
<button
8688
className={classes["side-button"]}
8789
type="button"

0 commit comments

Comments
 (0)