Skip to content

Commit

Permalink
Merge branch 'main' into feat/1650-change-uppercase-titles-to-sentenc…
Browse files Browse the repository at this point in the history
…e-case
  • Loading branch information
ArathyKumar committed Feb 9, 2025
2 parents 86a40dd + 7ab9dc5 commit 9b38749
Show file tree
Hide file tree
Showing 21 changed files with 257 additions and 51 deletions.
5 changes: 5 additions & 0 deletions .changeset/warm-forks-switch.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@rhds/elements": patch
---

`<rh-avatar>`: corrected small viewport margin for `plain` variant avatars
2 changes: 1 addition & 1 deletion docs/accessibility/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
title: Fundamentals
tags:
- accessibility
order: 75
order: 1
---

<script type="module" data-helmet>
Expand Down
2 changes: 1 addition & 1 deletion elements/rh-avatar/rh-avatar.css
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
}

:host([layout='block']) :is(img, canvas, svg),
#container.mobile :is(img, canvas, svg) {
:host(:not([plain])) #container.mobile :is(img, canvas, svg) {
margin-block-end: var(--rh-space-lg, 16px);
}

Expand Down
186 changes: 137 additions & 49 deletions elements/rh-button/docs/20-guidelines.md
Original file line number Diff line number Diff line change
Expand Up @@ -245,58 +245,146 @@ buttons. If buttons are stacked, the spacing between each button should be

## Best practices

### More than one line of text

Buttons should never have more than one line of text.

<uxdot-example width-adjustment="80px" danger>
<img src="../button-best-practice-1.png"
alt="Image of a button with two lines of text which is incorrect usage"
width="80"
height="60">
</uxdot-example>

### Multiple buttons

Do not use multiple Danger or Primary buttons in the same area.

<uxdot-example width-adjustment="544px" danger>
<img src="../button-best-practice-2.png"
alt="Image of two danger and two primary button groups which is incorrect usage"
width="544"
height="36">
</uxdot-example>
### Lines of text

<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="97px" slot="image">
<img src="../button-best-practices-text-lines-do.svg"
alt="Button with 'Add item' text on one line"
width="97"
height="60">
</uxdot-example>
<p>Keep the button’s text on one line.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="64px" slot="image">
<img src="../button-best-practices-text-lines-dont.svg"
alt="Button with 'Add item' text breaking to two lines"
width="64"
height="60">
</uxdot-example>
<p>Do not use more than one line of text.</p>
</uxdot-best-practice>
</div>

### Variants in button groups

<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="482px" slot="image">
<img src="../button-best-practices-variants-in-groups-do.svg"
alt="Button group with a primary button and a secondary button and another button group with a danger button and a link button"
width="482"
height="104">
</uxdot-example>
<p>Use one primary or primary danger button per button group.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="240px" slot="image">
<img src="../button-best-practices-variants-in-groups-dont.svg"
alt="Button group with two primary buttons and a second button group with two primary danger buttons"
width="240"
height="104">
</uxdot-example>
<p>Do not use multiple primary or primary danger buttons in the same button group.</p>
</uxdot-best-practice>
</div>

### Text labels

Do not write button text labels that are expressive or ambiguous.

<uxdot-example width-adjustment="358px" danger>
<img src="../button-best-practice-3.png"
alt="Image of two buttons; one has expressive language and the other has ambiguous language which is incorrect usage"
width="358"
height="36">
</uxdot-example>
<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="84px" slot="image">
<img src="../button-best-practices-labels-do.svg"
alt="Secondary button with 'Edit' text and primary button with `Submit` text"
width="84"
height="104">
</uxdot-example>
<p>Label buttons clearly and succinctly.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="120px" slot="image">
<img src="../button-best-practices-labels-dont.svg"
alt="Secondary button with 'Click here' text and primary button with 'Submit now!' text"
width="120"
height="104">
</uxdot-example>
<p>Do not write button text labels that are expressive or ambiguous.</p>
</uxdot-best-practice>
</div>

### Danger button

Do not use a Danger button for non-destructive purposes.

<uxdot-example width-adjustment="680px" danger>
<img src="../button-best-practice-4.png"
alt="Image of a search bar using a danger button which is incorrect usage"
width="680"
height="36">
</uxdot-example>

### Button as a call to action

Do not use buttons as links or change the Primary button styling, use a link or
call to action instead.

<uxdot-example width-adjustment="433px" danger>
<img src="../button-best-practice-5.png"
alt="Image of text styles with a button underneath that resembles a call to action which is incorrect usage"
width="433"
height="269">
</uxdot-example>
<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="482px" slot="image">
<img src="../button-best-practices-danger-do.svg"
alt="Danger button labeld 'Reset layout' below explanatory copy and next to a 'Cancel' link button"
width="482"
height="143">
</uxdot-example>
<p>Use a danger button for destructive actions only.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="482px" slot="image">
<img src="../button-best-practices-danger-dont.svg"
alt="Danger button labeled 'Search' next to a text input field"
width="482"
height="143">
</uxdot-example>
<p>Do not use a danger button for non-destructive actions.</p>
</uxdot-best-practice>
</div>

### Buttons vs. calls to action

<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="482px" slot="image">
<img src="../button-best-practices-buttons-vs-ctas-do.svg"
alt="Blue primary button below supporting copy"
width="482"
height="197">
</uxdot-example>
<p>Use buttons for actions, and retain the primary button styling to make them look different from calls to action.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="482px" slot="image">
<img src="../button-best-practices-buttons-vs-ctas-dont.svg"
alt="Brand red primary button below supporting copy"
width="482"
height="197">
</uxdot-example>
<p>Do not use buttons as links or change the style of a primary button to look more like a call to action.</p>
</uxdot-best-practice>
</div>

### Button icons

<div class="grid sm-two-columns">
<uxdot-best-practice variant="do">
<uxdot-example width-adjustment="121x" slot="image">
<img src="../button-best-practices-icons-do.svg"
alt="Button with one icon to the left of the text"
width="121"
height="36">
</uxdot-example>
<p>Use only one icon in a button.</p>
</uxdot-best-practice>

<uxdot-best-practice variant="dont">
<uxdot-example width-adjustment="145px" slot="image">
<img src="../button-best-practices-icons-dont.svg"
alt="Button with icons to the left and right of the text"
width="145"
height="36">
</uxdot-example>
<p>Do not use more than one icon in a button.</p>
</uxdot-best-practice>
</div>
Binary file removed elements/rh-button/docs/button-best-practice-1.png
Binary file not shown.
Binary file removed elements/rh-button/docs/button-best-practice-2.png
Binary file not shown.
Binary file removed elements/rh-button/docs/button-best-practice-3.png
Binary file not shown.
Binary file removed elements/rh-button/docs/button-best-practice-4.png
Binary file not shown.
Binary file removed elements/rh-button/docs/button-best-practice-5.png
Binary file not shown.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 7 additions & 0 deletions elements/rh-button/docs/button-best-practices-danger-do.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading

0 comments on commit 9b38749

Please sign in to comment.