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

Refactor/demo package #1072

Merged
merged 5 commits into from
Sep 19, 2023
Merged

Refactor/demo package #1072

merged 5 commits into from
Sep 19, 2023

Conversation

literat
Copy link
Collaborator

@literat literat commented Sep 18, 2023

Description

Unify multiple demo.css files into one package that is used by other demo-related apps.

Additional context

Currently, most of the changes done by @pavelklibani in his PRs are broken due to visual changes and non-unified CSS across all demos. This should solve the problem.

Issue reference

No issue reference here.


Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Follow the PR Title/Commit Message Convention.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

@netlify
Copy link

netlify bot commented Sep 18, 2023

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

Name Link
🔨 Latest commit b24f5a3
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/650996ca7ee90700080c5803

@netlify
Copy link

netlify bot commented Sep 18, 2023

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

Name Link
🔨 Latest commit b24f5a3
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/650996caa356d80008581ccf

@github-actions github-actions bot added the refactoring A code change that neither fixes a bug nor adds a feature label Sep 18, 2023
@netlify
Copy link

netlify bot commented Sep 18, 2023

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

Name Link
🔨 Latest commit b24f5a3
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/650996caf1e5bc0007b67789

@netlify
Copy link

netlify bot commented Sep 18, 2023

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

Name Link
🔨 Latest commit b24f5a3
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/650996cab71b640008972ae0

@coveralls
Copy link

coveralls commented Sep 18, 2023

Coverage Status

coverage: 70.919% (-25.5%) from 96.46% when pulling b24f5a3 on refactor/demo-package into 4e1b5eb on main.

@literat literat marked this pull request as ready for review September 19, 2023 09:18
Copy link
Contributor

@adamkudrna adamkudrna left a comment

Choose a reason for hiding this comment

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

Cool! Thanks for that! 🙏🏻

apps/web-twig-demo/assets/styles/app.scss Outdated Show resolved Hide resolved
packages/form-validations/assets/index.scss Outdated Show resolved Hide resolved
packages/web-react/assets/index.scss Outdated Show resolved Hide resolved
packages/demo/css/demo.css Outdated Show resolved Hide resolved
@literat literat force-pushed the refactor/demo-package branch from 9124b21 to b2547ff Compare September 19, 2023 12:01
@literat literat force-pushed the refactor/demo-package branch from b2547ff to b24f5a3 Compare September 19, 2023 12:40
@literat literat merged commit b16595c into main Sep 19, 2023
@literat literat deleted the refactor/demo-package branch September 19, 2023 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring A code change that neither fixes a bug nor adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants