Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Config Doc - Use tooltip for default enum value #42374

Closed
gsmet opened this issue Aug 7, 2024 · 1 comment · Fixed by #42455
Closed

Config Doc - Use tooltip for default enum value #42374

gsmet opened this issue Aug 7, 2024 · 1 comment · Fixed by #42455
Assignees
@gsmet
Labels
area/config
Milestone
3.14.0.CR1

Comments

@gsmet
Copy link
Member

gsmet commented Aug 7, 2024

At the moment, the tooltip is only present for the accepted values but we should also use it for the default one.

The code is ready, it just needs to be used.
@gsmet gsmet self-assigned this Aug 7, 2024
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Aug 7, 2024

/cc @radcortez (config)

gsmet added a commit to gsmet/quarkus that referenced this issue Aug 9, 2024
@gsmet
Config Doc - Use tooltip for default values
21f4edf 
Fixes quarkusio#42374
gsmet added a commit to gsmet/quarkus that referenced this issue Aug 9, 2024
@gsmet
Config Doc - Use enum tooltip also for default value
c434ce9 
This is an improvement over what we had.
Fixes quarkusio#42374
@gsmet gsmet mentioned this issue Aug 9, 2024
Merged
@gsmet gsmet closed this as completed in dc7ddb2 Aug 11, 2024
@quarkus-bot quarkus-bot bot added this to the 3.14 - main milestone Aug 11, 2024
frne pushed a commit to frne/quarkus that referenced this issue Aug 13, 2024
@gsmet @frne-trifork
Config Doc - Use enum tooltip also for default value
0a99ae8 
This is an improvement over what we had.
Fixes quarkusio#42374
danielsoro pushed a commit to danielsoro/quarkus that referenced this issue Sep 20, 2024
@gsmet @danielsoro
Config Doc - Use enum tooltip also for default value
bb0f040 
This is an improvement over what we had.
Fixes quarkusio#42374
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gsmet gsmet

Labels
area/config
Projects
None yet
Milestone
3.14.0.CR1
Development

Successfully merging a pull request may close this issue.

Various improvements for config doc generation gsmet/quarkus
1 participant
@gsmet