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

update demo page to use gcds-heading component #334

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

melaniebmn
Copy link
Collaborator

Summary | Résumé

Update the demo page to use the gcds-heading component instead of plain heading tags.

Copy link
Collaborator

@ethanWallace ethanWallace left a comment

Choose a reason for hiding this comment

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

LGTM

@melaniebmn melaniebmn merged commit 12283ab into develop Nov 14, 2023
3 checks passed
@melaniebmn melaniebmn deleted the update-demo-headings branch November 14, 2023 17:04
melaniebmn added a commit that referenced this pull request Nov 15, 2023
* chore: synced file(s) with cds-snc/site-reliability-engineering (#323)

chore: synced local '.github/workflows/backstage-catalog-helper.yml' with remote 'tools/sre_file_sync/backstage-catalog-helper.yml'

Co-authored-by: sre-read-write[bot] <92993749+sre-read-write[bot]@users.noreply.github.com>

* Add heading component (#306)

* wip: add heading component

* add heading component

* add prettier formatting for storybook overview files + add missing resources links for signature

* add figma link to heading resources

* add default margins to heading component

* add default value in events & properties table for heading story

* fix: Change slot CSS in gcds-breadcrumbs to display text underline (#325)

Change slot CSS in gcds-breadcrumbs to display text underline

* Prep for v 0.16.0 (#326)

* update EN changelog

* create version v 0.16.0

* update EN changelog

* chore: synced file(s) with cds-snc/site-reliability-engineering (#324)

chore: synced local '.github/workflows/backstage-catalog-helper.yml' with remote 'tools/sre_file_sync/backstage-catalog-helper.yml'

Co-authored-by: sre-read-write[bot] <92993749+sre-read-write[bot]@users.noreply.github.com>

* fix: give `npm publish` time to complete (#329)

fix: give npm publish time to complete

Update the CDN publish step to sleep for `60s` so that the npm publish
has time to complete.

* chore(deps): lock file maintenance (#327)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update js-devtools/npm-publish digest to 8d0621f (#330)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Add catalog-info.yaml (#332)

Co-authored-by: melaniebmn <melaniebmn@users.noreply.github.com>

* feat: Link component (#305)

* feat: init link component files (copied button component)

* feat: wip link component

* chore: updated component and tests

* wip: component changes and stories

* chore: add referrerpolicy inherited attribute

* chore: update link component to implementation requirements
also adds icons for email, download and phone numbers

* chore: update gcds-tokens package version

* fix: failing tests on introduction of non nullable attribute

* chore: replace Tag placeholder with a tag, add a11y considerations

* chore: remove unused css conditions for button text

* chore: update css and update tags on index.html

* chore: update spec tests

* chore: refactor / moved function

* chore: remove unnecessary slot

* chore: updates (noreferrer, required props, validation thoughts)

* chore: remove click handler

* chore: update link storybook

* chore: update default value for display storybook link component

* chore: remove "inherit" from display

* feat: add inherit as an option on size property

* chore: add todos for PR review questions

* chore: update todos, add variant tests for sizes, change prop target to string

* fix: event emmitters and cleanup

* chore: update build

* test: add more tests, and some cleanup

* chore: update stories

* chore: update package for gcds-tokens to v1.10.0

* chore: add back button text only styling

* chore: revert button styling

* chore: css updates

* chore: update default display settings

* chore: mark optional values, fix case for icon style

* chore: lint fix for button css

* chore: lint fixes

* chore: update i18n labels [en]

* chore: update i18n

* chore: update focus style, stories

* chore: storybook updates

* chore: story updates

* chore: story updates

* chore: update overview page

* chore: remove small button

* chore: cleanup

* chore: change size to default to inherit

* chore: update tests, add test for regular sized links

* chore: updates a11y test

* chore: update css

* chore: remove display option inline-block

* chore: change slot display for a11y fix and display settings

* chore: update target for external story

* chore: update story with link sizes

* chore: updated text and i18n

* chore: update link story

* chore: update component build

* chore: update tokens package version

* chore: css and story updates

* update demo page to use gcds-heading component (#334)

* add default value for margin top + bottom for text component (#333)

* fix gcds-link external icon spacing (#337)

* Improve storybook a11y (#338)

* storybook: replace boolean + radio controls in stories with select to improve support for assistive technologies

* storybook: improved focus styling for control form elements

* change focus colour for strybook control elements

* revert gcds-error-summary story update

* Update version + changelog for v 0.17.0 (#339)

* update changelog

* update version

---------

Co-authored-by: sre-read-write[bot] <92993749+sre-read-write[bot]@users.noreply.github.com>
Co-authored-by: Ethan Wallace <ethan.wallace.91@gmail.com>
Co-authored-by: Pat Heard <patrick.heard@cds-snc.ca>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: melaniebmn <melaniebmn@users.noreply.github.com>
Co-authored-by: Daine Trinidad <daine.trinidad@gmail.com>
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