Skip to content

Fix truncated key purpose dropdown labels#6

Merged
PastaPastaPasta merged 1 commit into
PastaPastaPasta:mainfrom
thephez:fix-truncation
Jan 27, 2026
Merged

Fix truncated key purpose dropdown labels#6
PastaPastaPasta merged 1 commit into
PastaPastaPasta:mainfrom
thephez:fix-truncation

Conversation

@thephez
Copy link
Copy Markdown
Contributor

@thephez thephez commented Jan 27, 2026

Remove substring truncation and widen dropdowns to display full key purpose names like AUTHENTICATION and ENCRYPTION.

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Summary by CodeRabbit

Release Notes

  • UI/UX Improvements
    • Key purpose names in configuration and management dropdowns now display in full instead of truncated versions.
    • Select dropdown controls have been resized for improved visual layout and consistency.

✏️ Tip: You can customize this high-level summary in your review settings.

Remove substring truncation and widen dropdowns to display full
key purpose names like AUTHENTICATION and ENCRYPTION.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

This PR updates UI presentation for key purpose selectors by enforcing minimum widths via CSS and extending option labels to display full purpose names instead of truncated versions across multiple select controls.

Changes

Cohort / File(s) Summary
CSS Styling Updates
index.html
Added two min-width: 155px constraints to .key-config .key-purpose-select and .add-key-config .manage-key-purpose-select to prevent select elements from collapsing.
Component Rendering Updates
src/ui/components.ts
Replaced truncated purpose names (p.substring(0, 6)) with full purpose strings (${p}) in key and manage key option rendering across renderConfigureKeysStep and renderManageViewKeysStep functions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A rabbit's widened view,
Selects grow plump with rightful might,
No more truncated names in sight,
Full purpose strings now take their flight,
CSS and code align just right!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing substring truncation and widening dropdowns to display full key purpose names in dropdown labels.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@PastaPastaPasta PastaPastaPasta merged commit ca93dd0 into PastaPastaPasta:main Jan 27, 2026
1 check passed
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