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

Redesign the Handlebars demo app (Web, React, Form Validations) #DS-1479 #1666

Conversation

adamkudrna
Copy link
Contributor

@adamkudrna adamkudrna commented Sep 26, 2024

Description

Include header and footer on all demo pages for easier navigation.

Additional context

obrazek

Issue reference

https://jira.almacareer.tech/browse/DS-1479

Copy link

netlify bot commented Sep 26, 2024

Deploy Preview for spirit-design-system ready!

Name Link
🔨 Latest commit 6ebc564
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system/deploys/66fe572d75dc4a0008c8d402
😎 Deploy Preview https://deploy-preview-1666--spirit-design-system.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: 96 (no change from production)
Accessibility: 100 (🟢 up 7 from production)
Best Practices: 100 (no change from production)
SEO: 83 (🟢 up 1 from production)
PWA: -
View the detailed breakdown and full score reports

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

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 26, 2024
Copy link

netlify bot commented Sep 26, 2024

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit 6ebc564
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/66fe572d70b51b00087ab8d4

@crishpeen
Copy link
Member

Screenshot_20240926-221524.png

Very nice, looking forward 😅😅😅

@literat
Copy link
Collaborator

literat commented Sep 30, 2024

Do not forget to update the snapshot tests in web-twig.

Base automatically changed from bc/header-v3-design-tokens to integration/BC-v3-design-tokens October 1, 2024 09:32
@adamkudrna adamkudrna force-pushed the docs/redesign-with-theme-switcher branch from c982a49 to fd3e6f0 Compare October 1, 2024 09:51
@coveralls
Copy link

coveralls commented Oct 1, 2024

Coverage Status

coverage: 78.791%. remained the same
when pulling 6ebc564 on docs/redesign-with-theme-switcher
into bc650cf on integration/BC-v3-design-tokens.

@adamkudrna adamkudrna force-pushed the docs/redesign-with-theme-switcher branch from fd3e6f0 to 9bfd318 Compare October 1, 2024 11:07
@crishpeen crishpeen force-pushed the integration/BC-v3-design-tokens branch from 60a10d7 to 429c9e1 Compare October 1, 2024 11:27
@adamkudrna adamkudrna force-pushed the integration/BC-v3-design-tokens branch from 429c9e1 to ec062d6 Compare October 1, 2024 11:34
@crishpeen crishpeen force-pushed the integration/BC-v3-design-tokens branch from ec062d6 to a4baaa7 Compare October 1, 2024 11:38
@adamkudrna adamkudrna force-pushed the docs/redesign-with-theme-switcher branch 4 times, most recently from b0b34a4 to a86bccf Compare October 1, 2024 14:17
@adamkudrna
Copy link
Contributor Author

Screenshot_20240926-221524.png

Very nice, looking forward 😅😅😅

I reduced the scope of this PR to the Handlebars demo only, without the theme switcher. More PRs are on their way! 🎉

obrazek

@adamkudrna adamkudrna changed the title WIP Demo redesign with theme switcher Redesign the Handlebars demo app (Web, React, Form Validations) #DS-1479 Oct 1, 2024
@adamkudrna adamkudrna marked this pull request as ready for review October 1, 2024 14:28
@adamkudrna adamkudrna force-pushed the docs/redesign-with-theme-switcher branch from a86bccf to c09a825 Compare October 1, 2024 15:38
@adamkudrna
Copy link
Contributor Author

adamkudrna commented Oct 1, 2024

The original draft PR was split into #1666 (this PR), #1675 (Twig), #1676 (theme switcher), #1677 (visual tests update).

Copy link
Collaborator

@literat literat left a comment

Choose a reason for hiding this comment

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

Nice work. I am looking forward to some future demo updates. 👍

apps/demo/partials/cover.hbs Outdated Show resolved Hide resolved
apps/demo/partials/cover.hbs Outdated Show resolved Hide resolved
apps/demo/partials/header.hbs Show resolved Hide resolved
apps/demo/partials/web/icons.hbs Outdated Show resolved Hide resolved
packages/demo/scss/index.scss Outdated Show resolved Hide resolved
Include header and footer on all demo pages for easier navigation.
@adamkudrna adamkudrna force-pushed the docs/redesign-with-theme-switcher branch from 824634c to 6ebc564 Compare October 3, 2024 08:34
@adamkudrna adamkudrna merged commit 7acd02d into integration/BC-v3-design-tokens Oct 3, 2024
17 checks passed
@adamkudrna adamkudrna deleted the docs/redesign-with-theme-switcher branch October 3, 2024 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants