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

Import animation styles separate from general styles. #2576

Merged
merged 30 commits into from Nov 6, 2023

Conversation

AnthonyLedesma
Copy link
Member

@AnthonyLedesma AnthonyLedesma commented Nov 1, 2023

Description

Moved Animation Styles. New Tests. New Workflow to test only on changed files. Bump to WP-next version.

Screenshots

animationOnCoreBlocks

Types of changes

Splitting styles apart into new Webpack compile.
Importing new styles always.
New tests to check for the condition of always-loaded styles.
New workflow for testing velocity.

How has this been tested?

Tested manually as well as new tests in CICD

Acceptance criteria

Should allow for animation to properly function on the Editor and Front end for all blocks, CoBlocks and Core.

Checklist:

  • My code is tested
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@AnthonyLedesma AnthonyLedesma self-assigned this Nov 3, 2023
@AnthonyLedesma AnthonyLedesma added the [Type] Bug Something that is not working as expected label Nov 3, 2023
Copy link
Contributor

@mtashjianjr-godaddy mtashjianjr-godaddy left a comment

Choose a reason for hiding this comment

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

Overall this looks great! Really great job putting some efficiency into our pipeline by only testing what changed!

includes/class-coblocks-block-assets.php Show resolved Hide resolved
.github/workflows/test-e2e-cypress.yml Outdated Show resolved Hide resolved
.github/workflows/test-e2e-cypress.yml Show resolved Hide resolved
@EvanHerman
Copy link
Contributor

EvanHerman commented Nov 6, 2023

PR changes are working perfectly on my end, great work on that! Just some minor changes above and I think it's good to go.

AnthonyLedesma and others added 8 commits November 6, 2023 10:41
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
Co-authored-by: Evan Herman <evan.m.herman@gmail.com>
@EvanHerman EvanHerman self-requested a review November 6, 2023 17:51
@AnthonyLedesma AnthonyLedesma dismissed mtashjianjr-godaddy’s stale review November 6, 2023 18:12

Attempted optimize but reverted as good enough for now.

@AnthonyLedesma AnthonyLedesma merged commit f67902e into master Nov 6, 2023
61 checks passed
@AnthonyLedesma AnthonyLedesma deleted the enqueue_animation_css_always branch November 6, 2023 18:23
@AnthonyLedesma AnthonyLedesma added this to the Next Release milestone Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug Something that is not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants