Skip to content

Conversation

@RomneyDa
Copy link
Collaborator

@RomneyDa RomneyDa commented Nov 20, 2025

Description

Before:
image

After:
small button styling with default theme border


Summary by cubic

Fixed the Thinking block toggle button styling so colors and borders match the default theme. Switched to the shared Button component and tightened spacing for a smoother expand/collapse.

  • Bug Fixes
    • Replaced native button with ui Button (outline) for consistent theming and border.
    • Simplified container and removed extra margins to reduce visual jitter when toggling.

Written for commit e856e36. Summary will update automatically on new commits.

@RomneyDa RomneyDa requested a review from a team as a code owner November 20, 2025 03:01
@RomneyDa RomneyDa requested review from tingwai and removed request for a team November 20, 2025 03:01
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 20, 2025
@RomneyDa RomneyDa changed the title fix: thinking block colors fix: HOTFIX thinking block colors Nov 20, 2025
@github-actions
Copy link

github-actions bot commented Nov 20, 2025

✅ Review Complete

Code Review Summary

⚠️ Continue configuration error. Please verify that the assistant exists in Continue Hub.


Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file

Prompt for AI agents (all 1 issues)

Understand the root cause of the following 1 issues and fix them.


<file name="gui/src/components/mainInput/belowMainInput/ThinkingBlockPeek.tsx">

<violation number="1" location="gui/src/components/mainInput/belowMainInput/ThinkingBlockPeek.tsx:62">
Preserve the previous non‑submitting behavior by passing `type=&quot;button&quot;` to the shared `Button`; otherwise the toggle will default to a submit button inside forms.</violation>
</file>

Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR

@RomneyDa RomneyDa requested review from Patrick-Erichsen and removed request for tingwai November 20, 2025 09:03
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Nov 20, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 20, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit 74b1f73 into main Nov 20, 2025
58 of 60 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 20, 2025
@Patrick-Erichsen Patrick-Erichsen deleted the dallin/thinking-colors branch November 20, 2025 17:51
@github-actions github-actions bot locked and limited conversation to collaborators Nov 20, 2025
@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.35.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.31.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants