Skip to content

Update shared template UI: style, phrasing

Amy Qualls requested to merge 330161-aqualls-instance-templates into master

What does this MR do?

Revises the wording at Settings > General > Templates for tone and style.

Note to the FE reviewer: I kludged together a lot of things. I can't guarantee I got it right, though the page does build. I'd appreciate a real review here:

  • Fix the very weird label on the dropdown box
  • Add helptext under the dropdown box, plus a link to the docs. I tried what I thought might work, and … nope. Hopefully it's clear what I was trying to do.
  • While we're at it, can we make these strings translatable? I wasn't sure how to do it.

Screenshots (strongly suggested)

Before

before-intro

before-section

After

after-intro

after-section

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec

  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods

  • Security reports checked/validated by a reviewer from the AppSec team

  • Closes #330208 (closed)

  • Related to technical-writing#419 (closed)

Edited by Marcel Amirault

Merge request reports

Loading