Skip to content

fix(DATAGO-133564): Fix 'Stop' button cutoff in narrow chat window#1455

Merged
amir-ghasemi merged 6 commits intomainfrom
amir/fix-ui
Apr 27, 2026
Merged

fix(DATAGO-133564): Fix 'Stop' button cutoff in narrow chat window#1455
amir-ghasemi merged 6 commits intomainfrom
amir/fix-ui

Conversation

@amir-ghasemi
Copy link
Copy Markdown
Collaborator

@amir-ghasemi amir-ghasemi commented Apr 25, 2026

Also addresses: DATAGO-133557
Fix broken icons in the Plan section

This pull request improves the display and handling of GitHub-flavored Markdown (GFM) task lists in the chat UI, enhances responsive design for chat controls, and updates agent templates to encourage proper Markdown checklist syntax. The most significant changes are grouped below:

Markdown Task List Rendering:

  • The MarkdownHTMLConverter component now detects and renders GFM task list checkboxes with custom-styled spans instead of native inputs, ensuring correct appearance and accessibility. List and list item styling is also adjusted to visually match GitHub's task lists. [1] [2]

Agent and Template Instructions:

  • Agent templates and example gateway YAML files now explicitly instruct agents to use GFM task list syntax (- [ ], - [x]) for checklists, and to avoid using emoji squares or other symbols. This ensures consistent rendering in the frontend. [1] [2] [3]

Responsive and Accessibility Improvements:

  • The chat input area and context usage indicator now use responsive CSS classes to hide or show labels and controls based on the viewport width, improving usability on smaller screens. [1] [2] [3]

@amir-ghasemi amir-ghasemi changed the title Amir/fix UI fix(DATAGO-133564): Fix 'Stop' button cutoff in narrow chat window Apr 25, 2026
@amir-ghasemi amir-ghasemi requested a review from lgh-solace April 25, 2026 00:52
@amir-ghasemi amir-ghasemi enabled auto-merge (squash) April 25, 2026 00:52
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

✅ FOSSA Guard: Licensing (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (1.20.4) • 0 new, 9 total (9 in base)

Scan Report | View Details in FOSSA

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 25, 2026

✅ FOSSA Guard: Vulnerability (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (1.20.4) • 0 new, 8 total (8 in base)

Scan Report | View Details in FOSSA

@amir-ghasemi amir-ghasemi changed the title fix(DATAGO-133564): Fix 'Stop' button cutoff in narrow chat window fix(DATAGO-133564): Fix 'Stop' button being cutoff in narrow chat window Apr 25, 2026
@amir-ghasemi amir-ghasemi changed the title fix(DATAGO-133564): Fix 'Stop' button being cutoff in narrow chat window fix(DATAGO-133564): Fix 'Stop' button cutoff in narrow chat window Apr 25, 2026
</Button>

<div>Agent: </div>
<div className="hidden @[480px]:block">Agent: </div>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we extract this hidden @[480px]:block and hidden @[480px]:inline to sharable constants?

borderRadius: "2px",
backgroundColor: isChecked ? "var(--primary-wMain)" : "transparent",
backgroundImage: isChecked
? "url(\"data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 16 16' fill='white'%3E%3Cpath d='M13.78 4.22a.75.75 0 0 1 0 1.06l-7 7a.75.75 0 0 1-1.06 0l-3-3a.75.75 0 1 1 1.06-1.06L6.25 10.69l6.47-6.47a.75.75 0 0 1 1.06 0Z'/%3E%3C/svg%3E\")"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a "white" in here - does it look ok in both themes?

But, this does seem kind of fragile and there are some more standard ways? Could use lucide icons? Or could just style the input (add a domNode.attribs.class = "size-3.5..." or something)? There's also a checkbox component, but that might be heavy for this usage

Could you explore the second suggestion? Accepting the input and just rendering it slightly differently?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.
image
image

@sonarqube-solacecloud
Copy link
Copy Markdown

@amir-ghasemi amir-ghasemi merged commit eb84685 into main Apr 27, 2026
27 of 29 checks passed
@amir-ghasemi amir-ghasemi deleted the amir/fix-ui branch April 27, 2026 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants