Skip to content

Resolve "Minor visual bugs for mobile view"

Hiru Fernando requested to merge 97-minor-visual-bugs-for-mobile-view into main

What does this MR do and why?

This MR takes care of the issues mentioned here #97 (closed). Most issues are relevant to mobile screens.

Can be reviewed using MR review app: https://new.docs.gitlab.com/review-mr-252/

Screenshots, screen recordings, or links to review app

  1. Reference architectures in landing page needed padding at the top in mobile version
BEFORE AFTER
image image
  1. When there's a large table, the other content is not centered
BEFORE AFTER
image image
  1. Footer on main content - mobile version was looking weird. Looks better stacked (after image)
BEFORE AFTER
image image
  1. Links are awkwardly broken up in "Install on Kubernetes" on landing page.
BEFORE AFTER
image image
  1. Nav toggle is repeating on mobile (used Xcode to debug - cannot replicate in normal browser responsive states)
BEFORE AFTER
image image
  1. Help and Feedback section - should stack on small window and display: flex on larger screen
BEFORE AFTER
image image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Configure a local GitLab Docs environment: https://gitlab.com/gitlab-org/technical-writing-group/gitlab-docs-hugo/-/blob/main/doc/setup.md.
  2. Check padding on "Reference Architectures" section around the table. Please check both mobile and normal website.
  3. Check main-content text is more centered and fills screen when there's a large table - https://new.docs.gitlab.com/review-mr-252/administration/reference_architectures/10k_users/#configure-praefect-postgresql on both mobile and website.
  4. Check footer on main-content on this page as well https://new.docs.gitlab.com/review-mr-252/administration/reference_architectures/10k_users/#configure-praefect-postgresql on both mobile and website. This should now be stacked at small width or mobile version instead of awkwardly wrapping in multiple lines.
  5. Check the links are stacked on "Install on Kubernetes" section on landing page on https://new.docs.gitlab.com/review-mr-252/ on mobile version and small screen on website.
  6. Check nav toggle (three dots at the top right) on header is not repeating on https://new.docs.gitlab.com/review-mr-252/ on mobile version.
  7. Check Help & Feedback on any of the main content pages https://new.docs.gitlab.com/review-mr-252/ on mobile and website.

Merge request acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Closes #97 (closed)

Edited by Hiru Fernando

Merge request reports

Loading