Skip to content
Snippets Groups Projects

Cleanup «New user/Edit user» forms

Merged Sascha Eggenberger requested to merge 361641-cleanup-new-user-form into master
All threads resolved!

What does this MR do and why?

Changes

  • Removes unnessessary dividers
  • Changes labels to be on top of fields
  • Removes background from form actions

Screenshots or screen recordings

Before After
New usernew_user_before New usernew_user_after
Edit useredit_user_before Edit useredit_user_after

How to set up and validate locally

Go to /admin/users/new

Related to #361641 (closed)

Edited by Sascha Eggenberger

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robert Hunt removed review request for @rob.hunt

    removed review request for @rob.hunt

  • requested review from @rob.hunt

  • added 1 commit

    Compare with previous version

  • Sascha Eggenberger added 1582 commits

    added 1582 commits

    Compare with previous version

  • Sascha Eggenberger resolved all threads

    resolved all threads

  • Sascha Eggenberger mentioned in merge request !89414 (merged)

    mentioned in merge request !89414 (merged)

  • Sascha Eggenberger mentioned in merge request !89417 (merged)

    mentioned in merge request !89417 (merged)

  • Sascha Eggenberger mentioned in merge request !89418 (merged)

    mentioned in merge request !89418 (merged)

  • Sascha Eggenberger mentioned in merge request !89422 (merged)

    mentioned in merge request !89422 (merged)

  • Robert Hunt approved this merge request

    approved this merge request

  • 👋 @rob.hunt, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Robert Hunt enabled an automatic merge when the pipeline for c72aa86f succeeds

    enabled an automatic merge when the pipeline for c72aa86f succeeds

  • merged

  • Robert Hunt mentioned in commit 996de451

    mentioned in commit 996de451

  • added workflowstaging label and removed workflowcanary label

  • mentioned in issue #361999 (closed)

  • mentioned in issue #366325 (closed)

  • Daniel Mora mentioned in issue #346945

    mentioned in issue #346945

  • Please register or sign in to reply
    Loading