Skip to content

Conversation

@starryeyez024
Copy link
Member

@starryeyez024 starryeyez024 commented Sep 16, 2021

Pull request description

Fixes names of CSS variables in demo page

Related issues

Preview

https://deploy-preview-1782--patternfly-elements.netlify.app/elements/pfe-styles/demo/typography-mobile.html

http://localhost:8080/elements/pfe-styles/demo/typography-mobile.html

What has changed and why

CSS variables had too many dashes, so they weren't working. Now you can see the demo of responsive typography in action.

Testing instructions

Merging

Please squash when merging and ensure your commit message uses conventional commit formatting.

Be sure to share your updates with the patternfly-elements-contribute@redhat.com mailing list!

@github-actions github-actions bot added the demo Updating demo pages label Sep 16, 2021
@netlify
Copy link

netlify bot commented Sep 16, 2021

✔️ Deploy Preview for patternfly-elements ready!

🔨 Explore the source changes: 175504d

🔍 Inspect the deploy log: https://app.netlify.com/sites/patternfly-elements/deploys/6143ba5471740f00079cae0a

😎 Browse the preview: https://deploy-preview-1782--patternfly-elements.netlify.app

@github-actions github-actions bot added the AT passed Automated testing has passed label Sep 16, 2021
Copy link
Contributor

@heyMP heyMP left a comment

Choose a reason for hiding this comment

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

LGTM!

@heyMP heyMP merged commit ac2af83 into master Sep 16, 2021
@heyMP heyMP deleted the issue-1781--typography-mobile-demo branch September 16, 2021 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AT passed Automated testing has passed demo Updating demo pages ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Typography demo page has incorrect variables

3 participants