Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDS-711: New component for Highlight and Quote #1060

Merged
merged 5 commits into from
Jul 17, 2023
Merged

Conversation

minnas
Copy link
Contributor

@minnas minnas commented Jul 3, 2023

Description

This PR adds component for Highlights and Quotes and needed documentation. More detailed description in the tickets below

Motivation and Context

HDS-711
HDS-712
HDS-1643

How Has This Been Tested?

Tested locally

Demos

React stories
Core stories
Documentation

@minnas minnas requested a review from a team July 3, 2023 08:35
Copy link
Contributor

@harriplappalainen harriplappalainen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought the previous PR was done reviewing but I still found some things to improve perhaps 😅 But this is nearly there! Great effort so far 💪

packages/react/DEVELOPMENT.md Outdated Show resolved Hide resolved

# Highlight

<StatusLabel type="alert" variant="rounded">Beta</StatusLabel>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is missing the Accessible StatusLabel and tooltip. Copy the implementation from another component's docs.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tooltip is missing too. If you look at for example site/src/docs/components/accordion/tabs.mdx, there's the StatuslabelTooltip component after these, and all of them are wrapped inside a <div class="status-label-description">.

site/src/docs/components/highlight/tabs.mdx Outdated Show resolved Hide resolved
site/src/docs/components/highlight/index.mdx Outdated Show resolved Hide resolved
site/src/docs/components/highlight/index.mdx Outdated Show resolved Hide resolved
site/src/docs/components/highlight/code.mdx Show resolved Hide resolved
@minnas minnas merged commit 0e91e8c into development Jul 17, 2023
5 checks passed
@minnas minnas deleted the hds-711-highlight branch July 17, 2023 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants