Skip to content
Snippets Groups Projects

Update vertical spacing of project settings

Merged Annabel Dunstone Gray requested to merge project-settings-spacing into master
All threads resolved!

What does this MR do and why?

  • Updates vertical spacing of project settings (reported in Slack)
  • Aligns toggle to top of container when it's next to a select so there is even spacing between every toggle setting + label/description
  • Indents Enable CVE ID requests... under Issues. None of the behavior has changed (ie it won't be automatically deselected/disabled if you deselect the first Issues toggle item, but I think this makes sense

Screenshots or screen recordings

Before After
Screenshot_2024-11-01_at_1.02.01_PM Screenshot_2024-11-01_at_12.24.19_PM
Screenshot_2024-11-01_at_1.02.18_PM Screenshot_2024-11-01_at_12.24.30_PM
Screenshot_2024-11-01_at_1.02.29_PM Screenshot_2024-11-01_at_12.24.39_PM

How to set up and validate locally

Project > settings > general > expand Visibility, project features, permissions

Links

Edited by Annabel Dunstone Gray

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
  • Chad Lavimoniere
  • requested review from @clavimoniere

  • added 1 commit

    • 5c7e6d12 - Update vertical spacing of project settings

    Compare with previous version

  • requested review from @ccharnolevsky

  • Chad Lavimoniere approved this merge request

    approved this merge request

  • added 1 commit

    • 50e53b4a - Update vertical spacing of project settings

    Compare with previous version

  • added 664 commits

    Compare with previous version

  • Annabel Dunstone Gray changed the description

    changed the description

  • added 206 commits

    Compare with previous version

  • Chen Charnolevsky approved this merge request

    approved this merge request

  • requested review from @lorenzvanherwaarden

  • Lorenz van Herwaarden approved this merge request

    approved this merge request

  • Lorenz van Herwaarden resolved all threads

    resolved all threads

  • Lorenz van Herwaarden enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • mentioned in commit 5cdb2de5

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading