Skip to content

fix(#1897): docs microsite header feedback click handler#243

Merged
ArakTaiRoth merged 1 commit into
GovAlta:alphafrom
syedszeeshan:SZ-1897-docs
Sep 26, 2024
Merged

fix(#1897): docs microsite header feedback click handler#243
ArakTaiRoth merged 1 commit into
GovAlta:alphafrom
syedszeeshan:SZ-1897-docs

Conversation

@syedszeeshan
Copy link
Copy Markdown
Collaborator

-documentation updates for MicroSiteHeader
-new example for custom click event handler for the feedback

@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 25, 2024

Deploy Preview for abgov-ui-component-docs ready!

Name Link
🔨 Latest commit 110f6a7
🔍 Latest deploy log https://app.netlify.com/sites/abgov-ui-component-docs/deploys/66f5c56b7320040008b19f54
😎 Deploy Preview https://deploy-preview-243--abgov-ui-component-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 39
Accessibility: 100
Best Practices: 100
SEO: 77
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@syedszeeshan syedszeeshan force-pushed the SZ-1897-docs branch 2 times, most recently from 0d9a8e3 to 9c5c601 Compare September 25, 2024 04:07
@syedszeeshan syedszeeshan changed the title fix(#1897): docs microsite header feedback click handler NOT READY - fix(#1897): docs microsite header feedback click handler Sep 25, 2024
@syedszeeshan syedszeeshan changed the title NOT READY - fix(#1897): docs microsite header feedback click handler fix(#1897): docs microsite header feedback click handler Sep 25, 2024
@ArakTaiRoth ArakTaiRoth linked an issue Sep 25, 2024 that may be closed by this pull request
@lizhuomeng71 lizhuomeng71 marked this pull request as ready for review September 26, 2024 07:20
@lizhuomeng71
Copy link
Copy Markdown
Collaborator

issue is verified on react and angular

Comment thread src/routes/components/MicrositeHeader.tsx
tags="angular"
allowCopy={true}
code={`
import { Component } from "@angular/core";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For other code snippets in the design system website, we don't include imports just to make things easier for users to copy the code snippet and paste to their existing code base. I suggest we should remove the import part, and refer other code snippets.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for pointing that out, I have removed the imports now.

@ArakTaiRoth ArakTaiRoth merged commit 447d005 into GovAlta:alpha Sep 26, 2024
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.

Microsite Header: Feedback click event

4 participants