mirror of
https://github.com/windmill-labs/windmill.git
synced 2026-09-06 00:02:13 +00:00
fix(frontend): address review nits on the markdown prose fix
- default `GfmMarkdown` to the `sm` stack rather than `xs`: the AI-agent tool Message pane takes the default and has no ancestor font size, so `xs` left it smaller than its own label. The group note, whose wrapper is `text-2xs`, opts down explicitly. - regenerate `static/tailwind_full.css`, which raw apps are served and which still carried the deleted list rules. - correct the marker-color rationale: the typography config already maps markers to tertiary, so the rule steps them up rather than rescuing them. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01S6G5gDXJnm6uqch4uCPkPE
This commit is contained in:
co-authored by
Claude Opus 5
parent
fe36aa34f0
commit
ee235507c7
@@ -216,10 +216,9 @@
|
||||
list-style-type: '- ';
|
||||
}
|
||||
|
||||
/* The list markers, horizontal rules and blockquote bars otherwise fall
|
||||
through to Tailwind Typography's default bullet/border colors, which are
|
||||
nearly invisible on dark backgrounds (e.g. the AI chat). Use theme-aware
|
||||
tokens so they stay readable in both light and dark mode. */
|
||||
/* List markers, horizontal rules and blockquote bars take the tertiary/light
|
||||
tokens the typography config maps them to, which is too faint to read on
|
||||
the denser markdown surfaces (e.g. the AI chat). Step them up one. */
|
||||
.prose :is(ul, ol) > li::marker {
|
||||
color: rgb(var(--color-text-secondary));
|
||||
}
|
||||
|
||||
@@ -10,7 +10,7 @@
|
||||
prose?: MarkdownProseSize
|
||||
}
|
||||
|
||||
let { md, noPadding, prose = 'xs' }: Props = $props()
|
||||
let { md, noPadding, prose = 'sm' }: Props = $props()
|
||||
|
||||
// Rendering markdown turns `` into a real `<img>`, i.e. a request. On the
|
||||
// public replay page the source is a recording from an arbitrary origin and the
|
||||
|
||||
@@ -133,7 +133,7 @@
|
||||
ondblclick={editMode ? stopPropagation(preventDefault(handleDoubleClick)) : undefined}
|
||||
onpointerdown={editMode ? stopPropagation(() => {}) : undefined}
|
||||
>
|
||||
<GfmMarkdown md={note} noPadding />
|
||||
<GfmMarkdown md={note} prose="xs" noPadding />
|
||||
</div>
|
||||
{:else}
|
||||
<!-- svelte-ignore a11y_no_static_element_interactions -->
|
||||
|
||||
@@ -299,7 +299,7 @@
|
||||
NOTE_TEXT_COLOR_OVERRIDE
|
||||
)}
|
||||
>
|
||||
<GfmMarkdown md={textForDisplay} prose="sm" noPadding />
|
||||
<GfmMarkdown md={textForDisplay} noPadding />
|
||||
</div>
|
||||
{:else}
|
||||
<div class={twMerge('text-xs italic opacity-60', colorConfig.text)}>
|
||||
|
||||
@@ -666,7 +666,6 @@
|
||||
<div class="p-4 rounded-md bg-surface-secondary">
|
||||
<GfmMarkdown
|
||||
md={defaultIfEmptyString(flow?.description, 'No description')}
|
||||
prose="sm"
|
||||
noPadding
|
||||
/>
|
||||
</div>
|
||||
|
||||
@@ -888,7 +888,6 @@
|
||||
<div class="p-4 rounded-md bg-surface-secondary">
|
||||
<GfmMarkdown
|
||||
md={defaultIfEmptyString(script?.description, 'No description')}
|
||||
prose="sm"
|
||||
noPadding
|
||||
/>
|
||||
</div>
|
||||
|
||||
File diff suppressed because one or more lines are too long
Reference in New Issue
Block a user