Skip to content
Snippets Groups Projects

UX guide restructure

Merged username-removed-711361 requested to merge ux-guide-restructure into master
All threads resolved!

The goal of this MR was to update the current UI guide to more of an UX guide (starting to detail when to use certain controls and patterns), and add structure to help organize the information. This initial version of the UX guide also lays down the groundwork for principles and personality. This guide is currently documenting the existing design, and will continue to be updated as the design evolves.

Some inspiration/references used:

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
  • Added 2 commits:

    • 7f5eaa12 - Add more details to buttons
    • be5bba5a - Add button images. Update link on development

    Compare with previous version

  • Dimitrie Hoekstra
  • Dimitrie Hoekstra
  • Added 1 commit:

    • 6ed37c75 - Renamed surfaces to templates. Add tooltip details

    Compare with previous version

  • Added 1 commit:

    • fee51ba6 - Update typography and icons on Basics page

    Compare with previous version

  • Added 1 commit:

    • 18c884bd - Add images for color. First draft of voice and tone

    Compare with previous version

  • Added 2 commits:

    • 43b7dd91 - Delete findings page
    • 0454e7b6 - Refine pages. Fill out Surfaces pages

    Compare with previous version

  • Added 1 commit:

    • 9175fb5d - Clean up layout on basics, surfaces, features. Add anchorlinks and counts to components

    Compare with previous version

  • Added 1 commit:

    Compare with previous version

  • username-removed-711361 Resolved all discussions

    Resolved all discussions

  • Added 2 commits:

    • 0cc55f2d - Add item title and system info block
    • b929fb3e - Fill out Features page

    Compare with previous version

  • Added 473 commits:

    Compare with previous version

  • username-removed-711361 Unmarked this merge request as a Work In Progress

    Unmarked this merge request as a Work In Progress

  • Assignee removed

  • Added ~15078 label

  • Chris Peressini
  • username-removed-711361 Resolved all discussions

    Resolved all discussions

  • Added 51 commits:

    Compare with previous version

  • Added 88 commits:

    Compare with previous version

  • Added 49 commits:

    Compare with previous version

  • @axil: Would you be able to review and merge this, or should I ask someone else? Thanks!

  • Reassigned to @axil

  • @awhildy yep, I'll take a look :)

  • Added 2 commits:

    • deef24a2 - Rename README.md to index.md
    • 406eda17 - Remove <br> and replace GFM blockquote with the Markdown general

    Compare with previous version

  • Victor Wu Added 1 commit:

    Added 1 commit:

    • 9d0dc681 - Update README.md to link to UX guide

    Compare with previous version

  • Awesome! I pushed some things :)

    • I renamed README.md to index.md (in order to render as /ux_guide/ in the docs portal and not having to type the ugly `ux_guide/README.html)
    • The
      are not really needed. Markdown adds its own padding :)
    • I replaced multi-blockquotes to single ones, as this is specific to GitLab and it won't render correctly in docs.gitlab.com
    Edited by Achilleas Pipinellis
  • Thanks @victorwu for the addition :) I'll merge now.

  • Achilleas Pipinellis Status changed to merged

    Status changed to merged

  • Mentioned in commit 8458f53c

  • @axil : If you renamed it index.md, then the file won't render in GitLab right? The convention is to use README.md as the intro page, and GitLab, GitHub, and most tools will render it?

    So right now, it won't be rendered here: https://gitlab.com/gitlab-org/gitlab-ce/tree/master/doc/development/ux_guide

  • @victorwu I totally see your point, but since we're moving forward to deprecating the /help page and focusing on https://docs.gitlab.com, having clean URLs instead of README.html makes much more sense. I'd expect people to navigate through the docs portal instead of the repository.

    Welcome to the ongoing discussion of having to maintain two sets of docs :) The one rendered under /help (GitLab's built-in docs) and https://docs.gitlab.com. Read the following for some context:

    If you have any thoughts on the matter please do share in the issues above :)

  • Achilleas Pipinellis Mentioned in merge request !7444 (merged)

    Mentioned in merge request !7444 (merged)

  • Mentioned in commit 64802597

  • Sounds good @axil. It looks like this has been discussed and is the intentional structure. Great! Thanks for the background.

    So for this page, will it be retired, is it already in the docs somewhere, or should there be a link it from one of the docs pages?

    https://gitlab.com/gitlab-org/gitlab-ce/blob/master/README.md

  • @victorwu no this is the project's page and is not rendered in the docs portal. Only what's inside doc/ is picked :) You can see what gets picked from our various products in https://gitlab.com/gitlab-com/gitlab-docs/blob/master/Rakefile.

  • @awhildy awesome work, thanks!

  • Please register or sign in to reply
    Loading