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!: CSS-in-JS for modifiers of Box #164

Merged
merged 17 commits into from
Mar 10, 2020
Merged

feat!: CSS-in-JS for modifiers of Box #164

merged 17 commits into from
Mar 10, 2020

Conversation

tassoevan
Copy link
Collaborator

@tassoevan tassoevan commented Mar 8, 2020

Since CSS is faster and more flexible than style prop/attribute, this PR aims to create CSS rules in runtime to handle dynamic styles. Also, it's expected a good reduction of bundle size.

@tassoevan tassoevan added ➕ Enhancement New feature or request 💅 Styles labels Mar 9, 2020
@tassoevan tassoevan marked this pull request as ready for review March 9, 2020 01:13
@tassoevan tassoevan changed the title feat: CSS-in-JS for modifiers of Box feat!: CSS-in-JS for modifiers of Box Mar 9, 2020
Copy link
Member

@gabriellsh gabriellsh left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@tassoevan tassoevan merged commit 53731c0 into develop Mar 10, 2020
@tassoevan tassoevan deleted the feat/css-in-js branch March 10, 2020 16:25
gabriellsh added a commit that referenced this pull request Mar 10, 2020
….Fuselage into new/focus-visible

* 'new/focus-visible' of github.com:RocketChat/Rocket.Chat.Fuselage:
  feat!: CSS-in-JS for modifiers of Box (#164)
gabriellsh added a commit that referenced this pull request Mar 11, 2020
…into feat/buttons

* 'develop' of github.com:RocketChat/Rocket.Chat.Fuselage: (26 commits)
  v0.4.0
  feat!: CSS-in-JS for modifiers of Box (#164)
  perf: Avoid re-renders due to PropsContext (#154)
  feat: useMutableCallback (#156)
  chore: Add bug report template (#158)
  feat: Added callback prop for scroll position (#155)
  fix: Display name and prop types of Throbber component (#148)
  fix: Changed accordion item title proptype (#151)
  v0.3.0
  fix: actionId on action hooks (#149)
  v0.2.0 (#150)
  chore: Tabs Layout (#125)
  feat: Loading Component (#147)
  v0.2.0
  v0.2.0-alpha.30
  fix: Safari not triggering buttons anchors (#146)
  v0.2.0-alpha.29
  bump version (#145)
  v0.2.0-alpha.28
  fix: placeholder transition (#144)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
➕ Enhancement New feature or request 💅 Styles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants