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

Dummy app winter cleanup #447

Merged
merged 2 commits into from
Dec 17, 2024
Merged

Dummy app winter cleanup #447

merged 2 commits into from
Dec 17, 2024

Conversation

nathalieArnoux
Copy link
Contributor

@nathalieArnoux nathalieArnoux commented Nov 8, 2024

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.

Screenshot from 2024-12-13 11-18-50
Screenshot from 2024-12-13 11-19-35
Screenshot from 2024-12-13 11-19-41
Screenshot from 2024-12-13 11-19-54
Screenshot from 2024-12-13 11-20-04
Screenshot from 2024-12-13 11-20-13

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

@nathalieArnoux nathalieArnoux self-assigned this Nov 8, 2024
Copy link

linear bot commented Nov 8, 2024

Copy link
Contributor

@JulienVannier66 JulienVannier66 left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

Comment on lines +70 to +71
padding: 12px;
margin: 18px auto 18px auto;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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">
Copy link
Contributor

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>
Copy link
Contributor

Choose a reason for hiding this comment

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

😆

@nathalieArnoux nathalieArnoux merged commit 190a5e9 into master Dec 17, 2024
3 checks passed
@nathalieArnoux nathalieArnoux deleted the na/VEL-3968 branch December 17, 2024 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants