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

Fix #1084 #1085

Merged
merged 4 commits into from
Apr 10, 2019
Merged

Fix #1084 #1085

merged 4 commits into from
Apr 10, 2019

Conversation

HenriBeck
Copy link
Member

What would you like to add/fix?
When the props have the same length while sorting in jss-plugin-props-sort, we check if the first prop is bigger than the second one and return 1 then. That way we should have a consistent output of orders even across different node versions

Corresponding issue (if exists): #1084

Henri Beck added 2 commits April 9, 2019 21:10
When the props have the same length while sorting in jss-plugin-props-sort, we check if the first prop is bigger than the second one and return 1 then. That way we should have a consistent output of orders even across different node versions
* master:
  Update size snapshots

# Conflicts:
#	packages/jss-preset-default/.size-snapshot.json
#	packages/react-jss/.size-snapshot.json
@HenriBeck HenriBeck requested a review from kof April 9, 2019 19:25
@HenriBeck
Copy link
Member Author

I will validate this fix tomorrow morning

@HenriBeck
Copy link
Member Author

@kof, please review

Copy link
Member

@kof kof left a comment

Choose a reason for hiding this comment

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

Nice one

@HenriBeck HenriBeck merged commit 49be31e into master Apr 10, 2019
@HenriBeck HenriBeck deleted the bugfix/1084 branch April 10, 2019 15:44
HenriBeck pushed a commit that referenced this pull request Apr 14, 2019
* master: (24 commits)
  Fix #1084 (#1085)
  Update size snapshots
  [jss-plugin-expand]Added the ability to expand the attributes for border-bottom (#1083)
  Update react-jss.md (#1078)
  Fix typspec for default unit plugin (#1077)
  Delete CNAME
  Create CNAME
  v10.0.0-alpha.16
  fix versions manually
  update changelog, make build
  Add local yarn version (#1071)
  update snapshots
  Update size-snapshots
  v10.0.0-alpha.15
  update changelog, make build
  Update to css-vendor@2.0.1 (#1068)
  Improve IE 11 support (#1065)
  fix changelog
  v10.0.0-alpha.14
  [jss] Add support for multiple referenced keyframes (#1063)
  ...

# Conflicts:
#	packages/jss-plugin-default-unit/.size-snapshot.json
#	packages/jss-preset-default/.size-snapshot.json
#	packages/jss-starter-kit/.size-snapshot.json
#	packages/jss/.size-snapshot.json
#	packages/react-jss/.size-snapshot.json
#	packages/react-jss/src/types.js
#	packages/react-jss/src/withStyles.js
bhupinderbola pushed a commit to bhupinderbola/jss that referenced this pull request Sep 17, 2019
* Fix cssinjs#1084

When the props have the same length while sorting in jss-plugin-props-sort, we check if the first prop is bigger than the second one and return 1 then. That way we should have a consistent output of orders even across different node versions

* Fix sorting

* Update changelog
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.

2 participants