openproject icon indicating copy to clipboard operation
openproject copied to clipboard

[#69702] Primerize API settings form

Open myabc opened this issue 3 months ago • 1 comments

⚠️ This PR is based on #21601. Please view/merge that PR first.

Ticket

https://community.openproject.org/wp/69702

What are you trying to accomplish?

Primerize the API Settings form. Uses the fieldset_group input group introduced in #21601.

Screenshots

image

What approach did you choose and why?

A best effort.

The "warning text" styling still needs to be discuss.

Merge checklist

  • [X] Added/updated tests
  • [X] Added/updated documentation in Lookbook (patterns, previews, etc)
  • [X] Tested major browsers (Chrome, Firefox, Edge, ...)

myabc avatar Dec 06 '25 22:12 myabc

The changes itself look good. I am personally not a big fan of the name InlineMessage

It was previously named WarningText. However i've changed it as it should now behave exactly like a Primer InlineMessage.

myabc avatar Jan 13 '26 09:01 myabc

The changes itself look good. I am personally not a big fan of the name InlineMessage

It was previously named WarningText. However i've changed it as it should now behave exactly like a Primer InlineMessage.

Okay, so it belongs to the PVC repo after all 👍

HDinger avatar Jan 14 '26 06:01 HDinger