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

feat(styling): add styling methods #30

Merged
merged 9 commits into from
Jun 26, 2022
Merged

Conversation

niloysikdar
Copy link
Member

@niloysikdar niloysikdar commented Jun 24, 2022

This PR covers:

  1. Add “sx” method
  2. Implemented makeStyles() util which gives useStyles() hook
  3. Add tests for sx and makeStyles
  4. Add stories for sx and makeStyles

Capture12

resolves #20

add sx method to combine stylings

resolves #20

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
add makeStyles which exposes a useStyles hook with mergeDeep function

resolves #20

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
migrated the commitlint config file from javascript to typescript

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
@niloysikdar niloysikdar added the GSoC 2022 Issues and Pull Requests under Google Summer of Code 2022 label Jun 24, 2022
improved the mergeStyles function and made changes to the makeStyles accordingly

resolves #20

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
add the missing tests for makeStyles inside the utils

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
src/utils/stylings.ts Outdated Show resolved Hide resolved
src/utils/makeStyles.test.ts Outdated Show resolved Hide resolved
src/utils/makeStyles.ts Outdated Show resolved Hide resolved
fix broken types and code for sx, mergeStyles and makeStyles

BREAKING CHANGE: named argument pass for useStyles

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
add stories for makeStyles

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
add stories for sx

Signed-off-by: Niloy Sikdar <niloysikdar30@gmail.com>
@niloysikdar niloysikdar marked this pull request as ready for review June 26, 2022 10:54
@agentmilindu agentmilindu merged commit b47e913 into main Jun 26, 2022
@niloysikdar niloysikdar deleted the 20-adding-styling-methods branch June 26, 2022 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GSoC 2022 Issues and Pull Requests under Google Summer of Code 2022
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding styling methods
2 participants