-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dummy app winter cleanup #447
Conversation
a0a1d11
to
6c8c969
Compare
12f383a
to
9ea5367
Compare
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.
Otherwise LGTM
padding: 12px; | ||
margin: 18px auto 18px auto; |
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.
padding: 12px; | |
margin: 18px auto 18px auto; | |
padding: var(--spacing-px-12); | |
margin: var(--spacing-px-18) auto var(--spacing-px-18) auto; |
|
||
</div> | ||
|
||
{{!-- <div class="fx-col fx-gap-px-6 padding-px-18"> |
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.
Do you want to keep all this part commented?
<div class="page-title-container"> | ||
<span class="font-size-h2 font-weight-semibold">Overlay</span> | ||
<span class="font-color-gray-500">Components used to display information on the foreground</span> | ||
<span class="font-color-gray-500 font-size-xs text-style-italic">(c'est qui qu'est over laid ?)</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.
😆
What does this PR do?
Related to: #VEL-3968
Complete reorganization of the OSS Components Dummy App into different pages for better organization and navigation.
Information on the repartition of the components is available in the spreadsheet here.
Good PR checklist