grommet / hpe-design-system

HPE Design System
48 stars 24 forks source link

Template library - Tracking issue #3535

Open halocline opened 1 year ago

halocline commented 1 year ago

Template File Library


The scope of this work includes:

Identifying assets (Seamus + Kenny)

Within this first pass, let's aim to include simple and quick-to-make/refine assets

Creating a roadmap of future assets (Seamus + Kenny)

Creating a template file

Migrating + Creating assets (Asif + Kurt)

Once identified, old and new assets must be reviewed by 1 other designer and 1 other dev. For each item, make sure the items are:

Make note of any major issues, let's tackle those in separate individual tickets.

EXISTING ASSETS

NEW ASSETS

Announcement

KennyAtHPE commented 1 year ago

OLD DELIVERABLES FOR REFERENCE

Tracking issue for template work.

Desired Templates

Controls

Layouts

Typography

SeamusLeonardHPE commented 1 year ago

This ticket aims to deliver a first draft of a design pattern library before the end of sprint.

We are to start designing form smaller organisms and on a following sprint we will assemble those into larger templates.

Phase 1

Phase 2

For visibility of other patterns that have been suggested/built via PAL & Sandbox https://hpe-my.sharepoint.com/:x:/r/personal/seamus_leonard_hpe_com/Documents/Asset%20%26%20Template%20Review%20-%20PAL+SANDBOX.xlsx?d=w77af88ad44ea4da6a3787c5f27475ee1&csf=1&web=1&e=7zfk6V

SeamusLeonardHPE commented 1 year ago

@kurt-ph @KennyAtHPE

Review existing design templates & add comments that are in the “construction” file, start with Side drawer & Center layer.
 @SeamusLeonardHPE will make edits based on feedback & copy over to the “public” file when agreed upon.


Center Modal



Side Drawer

Reviews will consist of

  1. Fonts - all fonts use HPE styles (button medium is an exception)
  2. Color styles are use HPE styles
  3. Elevation uses HPE styles (where applicable, not all containers require eg form groups)
  4. Padding, Gap, Border radius - All should adhere to HPE t-shirt sizing - inspect containers & auto-layout groups for adherence.

  5. Do the templates adhere to guidance (e.g. button alignment, order, centre modal is for concise confirmations or dialogs etc)

  6. Use components - Is the template build using current components everywhere possible (note I have broken name value pair out of name value list)
  7. Deprecated - Comment on any “deprecated” components used, or nested inside published components.

Last check before adding to public library will include making sure auto-flow setting (fill/fixed/hug) are all correct & consistent .

Comments and discussion for furthering the project.


Complete the review & edits & announce the public library form Monday.


KennyAtHPE commented 1 year ago

Reviewed both the Side and Center layer pages

  1. Fonts - all fonts use HPE styles (button medium is an exception)
    • Some areas are using Normal/default rather than Paragraph/default. Everything else is fine.
      1. Color styles are use HPE styles
    • All colors are using DS color styles
      1. Elevation uses HPE styles (where applicable, not all containers require eg form groups)
    • Looked good to me. When the SelectMultiple component is opened then the dropdown elevation clashes with the layer but this is a Figma limitation. In code this would appear fine.
      1. Padding, Gap, Border radius - All should adhere to HPE t-shirt sizing - inspect containers & auto-layout groups for adherence.

    • Auto layout padding looks good.
      1. Do the templates adhere to guidance (e.g. button alignment, order, centre modal is for concise confirmations or dialogs etc)

    • Questionable for some layer-closing behaviors. Review comments in Figma for more info.
      1. Use components - Is the template build using current components everywhere possible (note I have broken name value pair out of name value list)
    • Yes, but we should not be breaking NameValuePair out of the NameValueList. Same as in Grommet, the NameValuePair component cannot appear outside of a NameValueList component.
      1. Deprecated - Comment on any “deprecated” components used, or nested inside published components.
    • Looks good but we are using the TextArea component in some areas which is fine but that component has not been (but needs to be) refactored.

General comments: