From 6d2df94810fe764dba16c8d0207f36d1226e1481 Mon Sep 17 00:00:00 2001 From: gobeumsu Date: Thu, 23 Apr 2026 02:37:52 +0900 Subject: [PATCH] Reduce frontmatter editor action density to match icon-first controls The frontmatter editor used a redundant labeled folder action in a crowded modal surface. This commit keeps the same browse behavior but switches the control to an icon-first affordance with accessible text. Constraint: The browse action must stay discoverable without changing selection behavior Rejected: Rework the options section layout | unnecessary for the targeted design fix Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep standard utility actions icon-first unless a documented exception applies Tested: pnpm run ci Not-tested: Manual modal usability review in Obsidian --- src/ui/settings/modals/FrontmatterEditorModal.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/ui/settings/modals/FrontmatterEditorModal.ts b/src/ui/settings/modals/FrontmatterEditorModal.ts index 0729c52..ec604d5 100644 --- a/src/ui/settings/modals/FrontmatterEditorModal.ts +++ b/src/ui/settings/modals/FrontmatterEditorModal.ts @@ -113,8 +113,8 @@ export class ConfigurableSettingModal extends Modal { .setDesc('Values that the AI can use as suggestions') .addButton((button) => { button - .setButtonText('Browse files') .setIcon('folder') + .setTooltip('Browse files') .onClick(() => { const wikiLinkSelector = new WikiLinkSelector(this.app); wikiLinkSelector.openFileSelector((selectedLink) => { @@ -125,6 +125,7 @@ export class ConfigurableSettingModal extends Modal { this.updateOptionsTextarea(); }); }); + button.buttonEl.setAttribute('aria-label', 'Browse files'); }); if (this.props.options.showTextArea) {