Skip to content

Fix #1638: Add missing text prompt suffix#2102

Merged
patriksvensson merged 1 commit into
spectreconsole:mainfrom
merklegroot:text-prompt-colon-consistency
Apr 25, 2026
Merged

Fix #1638: Add missing text prompt suffix#2102
patriksvensson merged 1 commit into
spectreconsole:mainfrom
merklegroot:text-prompt-colon-consistency

Conversation

@merklegroot

Copy link
Copy Markdown
Contributor

Fixes #1638

Checklist

  • I have read the Contribution Guidelines
  • I have checked that there isn't already another pull request that solves the above issue
  • All newly added code is adequately covered by tests
  • All existing tests are still running without errors

Changes

  • Fixed TextPrompt so the suffix (default :) is consistently shown, even when no default value is provided.
  • This resolves the visual inconsistency reported in Prompt suffix inconsistency #1638.

Tested on .NET 10.0 — relevant tests pass.

AI Disclosure

I used Cursor and Grok to help generate the initial fix, then reviewed and made several manual tweaks.

@merklegroot

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@patriksvensson patriksvensson merged commit 9aeb9e7 into spectreconsole:main Apr 25, 2026
3 checks passed
@patriksvensson

Copy link
Copy Markdown
Contributor

Merged! Thank you for your contribution. Much appreciated! 👍

This was referenced Jun 11, 2026
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.

Prompt suffix inconsistency

2 participants