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

Pals Knapsack: Contribute Back to Hyku #2193

Closed

Conversation

ShanaLMoore
Copy link
Collaborator

@ShanaLMoore ShanaLMoore commented Apr 22, 2024

dupe of: #2178 , sha 62b7d9 due to newly introduced build issues

This is a first pass of the PALs work.  There was a lot of head
scratching and copying things into Hyku Prime.  Together this is a
rough process.

Related to:

- scientist-softserv/palni_palci_knapsack@3595d6f
flex-direction: column;

&.active {
display: flex !important;

Choose a reason for hiding this comment

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

!important should not be used

@keyframes spin {
0% { transform: rotate(0deg); }
100% { transform: rotate(360deg); }
}

#splash {
display: none;

Choose a reason for hiding this comment

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

Properties should be ordered align-items, background, display, flex-direction, height, justify-content, left, position, top, width, z-index

@keyframes spin {
0% { transform: rotate(0deg); }
100% { transform: rotate(360deg); }
}

#splash {

Choose a reason for hiding this comment

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

Avoid using id selectors
Rule set contains (11/10) properties

width: 120px;
height: 120px;
-webkit-animation: spin 2s linear infinite; /* Safari */

Choose a reason for hiding this comment

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

Use // comments everywhere

@ShanaLMoore ShanaLMoore added the major-ver for release notes label Apr 22, 2024
@orangewolf orangewolf deleted the 62b7d9c5df28e1a576a399e0ebc505e56533fcf4-knapsack branch August 26, 2024 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants