Skip to content

Fix checkbox parameter display issues#48

Closed
cdr-robot wants to merge 1 commit into
mainfrom
fix/checkbox-parameter-display
Closed

Fix checkbox parameter display issues#48
cdr-robot wants to merge 1 commit into
mainfrom
fix/checkbox-parameter-display

Conversation

@cdr-robot
Copy link
Copy Markdown
Contributor

Summary

  • Always show parameter label at the top for all parameter types consistently
  • Add support for custom checkbox label text via styling.label property
  • Default checkbox label to \Enable\ for better UX when no custom label is provided

Fixes #25

Test plan

  1. Create a template with checkbox parameters using both approaches:
    • Standard checkbox parameter
    • Checkbox parameter with styling.label property

This PR addresses the issue where checkbox parameters show the display name twice in the UI, by:

  1. Always showing the parameter name/display name at the top (consistent with other parameters)
  2. Using a custom label or default \Enable\ text next to the checkbox
  3. Supporting the styling.label property as proposed in the original issue

Workspace URL: https://demo.coder.com/ben/issue-25

… custom label from styling

- Always show parameter label at the top for all parameter types
- Add support for custom checkbox label text via styling.label property
- Default checkbox label to \Enable\ for better UX

Fixes #25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

parameters: how can/should we support a different checkbox value from the title

1 participant