-
Notifications
You must be signed in to change notification settings - Fork 60
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
docs: remove SCSS from the landing page, update library name #429
Conversation
0f2f91e
to
de6469e
Compare
Deploy preview for fundamental-styles ready! Built with commit 0f2f91e |
Deploy preview for fundamental-styles ready! Built with commit daf265b |
date picker & multiimput are 404/ navigating to wrong locations in patterns section: https://deploy-preview-429--fundamental-styles.netlify.com/patterns/multi-input.html |
- title: Combobox Input | ||
url: /patterns/combobox-input.html | ||
- title: Date Picker | ||
url: /patterns/multi-input.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is the cause of the 404
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
<span class="fd-toggle fd-toggle--xs"> | ||
<input class="toggle-bg fd-toggle__input" type="checkbox" name="" value="" id="{{ randomNumber }}"> | ||
<span class="fd-toggle__switch" role="presentation"></span> | ||
</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we getting rid of the background toggle?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the actual value of this?
de6469e
to
925b21a
Compare
config/postcss.config.js
Outdated
postcssAddFallback({importFrom: 'dist/root.css'}), | ||
postcssAddFallback({importFrom: 'node_modules/@sap-theming/theming-base-content/content/Base/baseLib/sap_fiori_3/css_variables.css'}), | ||
postcssAddFallback({ importFrom: 'dist/root.css' }), | ||
postcssAddFallback({ importFrom: 'node_modules/@sap-theming/theming-base-content/content/Base/baseLib/sap_fiori_3/css_variables.css' }), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you revert the re-formatting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reverted
5d39458
to
48d92c8
Compare
48d92c8
to
daf265b
Compare
fixes: #427