Skip to content

Conversation

istarkov
Copy link
Member

Description

  1. What is this PR about (link the issue and add a short description)

Steps for reproduction

  1. click button
  2. expect xyz

Code Review

  • hi @kof, I need you to do
    • conceptual review (architecture, feature-correctness)
    • detailed review (read every line)
    • test it on preview

Before requesting a review

  • made a self-review
  • added inline comments where things may be not obvious (the "why", not "what")

Before merging

  • tested locally and on preview environment (preview dev login: 0000)
  • updated test cases document
  • added tests
  • if any new env variables are added, added them to .env file

@istarkov istarkov force-pushed the animate branch 4 times, most recently from 6c278d7 to 14da057 Compare February 19, 2025 17:34
@istarkov istarkov requested review from kof and TrySound February 23, 2025 13:24
@istarkov istarkov marked this pull request as ready for review February 23, 2025 13:25
Animate 2

Animation UI - Type Select

Subject selection

Create new Animation

New animation

Add

Add sort/delete

subject

Add Ranges Select

Fix scroll

Ranges Ready

Fix comments

Allow negative values

Add storybook

Add storie

Add error

Add offset

Add anim

Edit keyframes

Add Fill Mode

Add easing

Rename

Fix errors

Hide animate children

Add hook

upd

Fix

Fix hook

Allow calc

Add ability to play localy

isPinned

Add mutations to track

Support composition

Switch auto on add

Add animation-composition

Use default composite

Fix tests

Allow select self

Add kebab

Change private detection

refactor: output hyphenated properties from css parser (#4900)

We are going to switch to hyphenated properties in styles. Here
refactored css parser to output hyphenated property instead of camel
case and added camelCaseProperty utility which does the opposite of
hyphenateProperty.

fix my env
@istarkov istarkov merged commit 62a6d93 into main Feb 24, 2025
17 checks passed
@istarkov istarkov deleted the animate branch February 24, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants