doc/development/contributing/design.md
Follow these guidelines when contributing or reviewing design and user interface (UI) changes. Refer to our code review guide for broader advice and best practices for code review in general.
The basis for most of these guidelines is Pajamas, GitLab design system. We encourage you to contribute to Pajamas with additions and improvements.
As a merge request (MR) author, you must:
If you are a team member: We recommend assigning the Product Designer suggested by the Reviewer Roulette as reviewer. This helps us spread work evenly, improve communication, and make our UI more consistent. If you have a reason to choose a different reviewer, add a comment to mention you assigned it to a Product Designer of your choice.
If you are a community contributor: We favor choosing the Product Designer that is a domain expert in the area you are contributing, to regardless of the Reviewer Roulette.
Check these aspects both when designing and reviewing UI changes.
Check visual design properties using your browser's elements inspector (Chrome, Firefox).
Check states using your browser's styles inspector to toggle CSS pseudo-classes
like :hover and others (Chrome,
Firefox).
Check responsive behavior using your browser's responsive mode (Chrome, Firefox).
Check accessibility using your browser's accessibility inspector (Chrome, Firefox).
When the design is ready, before starting its implementation:
At any moment, but usually during or after the design's implementation:
~Deferred UX
label for intentional deviations from the agreed-upon UX requirements due to
time or feasibility challenges, linking back to the corresponding issues or
merge requests.