Skip to content

Cosmetics in Java properties table#2672

Open
chgeo wants to merge 3 commits into
mainfrom
prop-table-cosmetics
Open

Cosmetics in Java properties table#2672
chgeo wants to merge 3 commits into
mainfrom
prop-table-cosmetics

Conversation

@chgeo

@chgeo chgeo commented Jun 25, 2026

Copy link
Copy Markdown
Member

Better column balance in Java properties table

  • Give Description column more room.
  • Make long property names wrap.
  • Reduce 'Default' column width.

Render property name with config inspector.

Also make the table header sticky when scrolling.


Before:
image


After:
image

- Give Description column more room.
- Make long property names wrap.
- Reduce 'Default' column width.
@chgeo chgeo requested a review from beckermarc June 25, 2026 13:26
@chgeo

chgeo commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

@beckermarc beside this, the description of cds.security.authentication.mode is a bit off:
image

That's the doc string from the json:
Determines the mode that is used when authenticating endpoints of protocol-adapters.<br>Possible values are: - never: No endpoint requires authentication - model-relaxed:<br>Authentication is derived from @requires/@restrict, defaults to public endpoints. -<br>model-strict: Authentication is derived from @requires/@restrict, defaults to authenticated<br>endpoints. - always: All endpoints require authentication

Maybe you can improve that.

@chgeo chgeo changed the title Cosmetics in Java properties table. Cosmetics in Java properties table Jun 25, 2026
@cap-bots cap-bots requested a review from swaldmann as a code owner June 25, 2026 13:45
@cap-bots cap-bots force-pushed the prop-table-cosmetics branch from a20fd1e to 3dffb16 Compare June 25, 2026 13:47
swaldmann
swaldmann previously approved these changes Jun 25, 2026
@chgeo chgeo marked this pull request as draft June 25, 2026 14:32
@chgeo chgeo marked this pull request as ready for review June 25, 2026 14:41
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