From 7e45474735ccbbcfa92713236e9ce8185507f751 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micka=C3=ABl=20Canouil?= <8896044+mcanouil@users.noreply.github.com> Date: Mon, 20 Jan 2025 22:23:45 +0100 Subject: [PATCH 1/8] fix: use text box instead of input for alt-text --- apps/vscode/CHANGELOG.md | 2 ++ packages/editor-ui/src/dialogs/edit-image.tsx | 11 +++++------ 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/apps/vscode/CHANGELOG.md b/apps/vscode/CHANGELOG.md index caa9c8cb..21348483 100644 --- a/apps/vscode/CHANGELOG.md +++ b/apps/vscode/CHANGELOG.md @@ -2,6 +2,8 @@ ## 1.119.0 (unreleased) +- Visual Editor: "Inset image" command now uses a text box for the alternative text. () + ## 1.118.0 (Release on 2024-11-26) - Provide F1 help at cursor in Positron () diff --git a/packages/editor-ui/src/dialogs/edit-image.tsx b/packages/editor-ui/src/dialogs/edit-image.tsx index cb405455..3d95260e 100644 --- a/packages/editor-ui/src/dialogs/edit-image.tsx +++ b/packages/editor-ui/src/dialogs/edit-image.tsx @@ -28,7 +28,8 @@ import { useId, RadioGroup, Label, - Radio + Radio, + Textarea } from "@fluentui/react-components"; import { ModalDialog, ModalDialogTabList, showValueEditorDialog} from "ui-widgets"; @@ -162,10 +163,11 @@ const imagePanel = {alt !== undefined ? - setAlt(data.value)} placeholder={t("(Optional)")} + style={{ height: "100px" }} /> : null @@ -288,6 +290,3 @@ const useStyles = makeStyles({ gridRowGap: tokens.spacingVerticalS, } }) - - - From 615dc1ee4726a7be91400729cf55b7f10fcb4479 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micka=C3=ABl=20Canouil?= <8896044+mcanouil@users.noreply.github.com> Date: Mon, 20 Jan 2025 22:57:25 +0100 Subject: [PATCH 2/8] fix: use resize instead of setting height --- packages/editor-ui/src/dialogs/edit-image.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/editor-ui/src/dialogs/edit-image.tsx b/packages/editor-ui/src/dialogs/edit-image.tsx index 3d95260e..4d2e5476 100644 --- a/packages/editor-ui/src/dialogs/edit-image.tsx +++ b/packages/editor-ui/src/dialogs/edit-image.tsx @@ -167,7 +167,7 @@ const imagePanel = value={alt} onChange={(_ev, data) => setAlt(data.value)} placeholder={t("(Optional)")} - style={{ height: "100px" }} + resize="vertical" /> : null From 1a828859de2406468baa4bf438c4e54bfd529c7e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micka=C3=ABl=20Canouil?= <8896044+mcanouil@users.noreply.github.com> Date: Tue, 21 Jan 2025 00:03:34 +0100 Subject: [PATCH 3/8] refactor: use text area for caption --- packages/editor-ui/src/dialogs/edit-image.tsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/editor-ui/src/dialogs/edit-image.tsx b/packages/editor-ui/src/dialogs/edit-image.tsx index 4d2e5476..1a9afd89 100644 --- a/packages/editor-ui/src/dialogs/edit-image.tsx +++ b/packages/editor-ui/src/dialogs/edit-image.tsx @@ -154,10 +154,11 @@ const imagePanel = : null } - setCaption(data.value)} placeholder={t("(Optional)")} + resize="vertical" /> From f4bf877096b84b4e6318d2733e2bacf0beb9d0c5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micka=C3=ABl=20Canouil?= <8896044+mcanouil@users.noreply.github.com> Date: Tue, 21 Jan 2025 00:07:12 +0100 Subject: [PATCH 4/8] refactor: text area for table caption --- packages/editor-ui/src/dialogs/insert-table.tsx | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/editor-ui/src/dialogs/insert-table.tsx b/packages/editor-ui/src/dialogs/insert-table.tsx index 27f3f2e7..593422db 100644 --- a/packages/editor-ui/src/dialogs/insert-table.tsx +++ b/packages/editor-ui/src/dialogs/insert-table.tsx @@ -91,7 +91,12 @@ const InsertTableDialog: React.FC<{ {props.options.captions ? ( - setCaption(data.value)} placeholder={t("(Optional)")}/> +