Skip to content

docs: document withSpring options#3328

Merged
piaskowyk merged 1 commit intomainfrom
@kacperkapusciak/document-withSpring-options
Jun 28, 2022
Merged

docs: document withSpring options#3328
piaskowyk merged 1 commit intomainfrom
@kacperkapusciak/document-withSpring-options

Conversation

@kacperkapusciak
Copy link
Copy Markdown
Member

Description

This PR adds descriptions to options of the withSpring animation.

https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withSpring

Screenshots / GIFs

Before

image

After

Sorry for the dark theme 🙈

image

Test code and steps to reproduce

cd docs/
yarn
yarn start

and navigate to http://localhost:3000/react-native-reanimated/docs/api/animations/withSpring

Checklist

  • Updated documentation

@tomekzaw
Copy link
Copy Markdown
Member

Finally we know what these numbers really do. Thanks for your contribution!

Copy link
Copy Markdown
Member

@piaskowyk piaskowyk left a comment

Choose a reason for hiding this comment

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

🚀 ❤️

@piaskowyk piaskowyk merged commit 57b949d into main Jun 28, 2022
@piaskowyk piaskowyk deleted the @kacperkapusciak/document-withSpring-options branch June 28, 2022 12:54
piaskowyk pushed a commit that referenced this pull request Jun 28, 2022
## Description

This PR adds descriptions to options of the `withSpring` animation.

https://docs.swmansion.com/react-native-reanimated/docs/api/animations/withSpring


## Screenshots / GIFs

### Before

![image](https://user-images.githubusercontent.com/39658211/175949620-091a7b24-8df1-421f-a822-4f3507872a46.png)

### After

Sorry for the dark theme 🙈

![image](https://user-images.githubusercontent.com/39658211/175949320-e9a4cc64-d2de-4595-84fa-e5c8d4d3531e.png)


## Test code and steps to reproduce

```sh
cd docs/
yarn
yarn start
```

and navigate to http://localhost:3000/react-native-reanimated/docs/api/animations/withSpring

## Checklist

- [x] Updated documentation
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