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

Allow users to override default table row styles #1175

Merged
merged 1 commit into from
Oct 3, 2018
Merged

Allow users to override default table row styles #1175

merged 1 commit into from
Oct 3, 2018

Conversation

emroussel
Copy link
Contributor

@emroussel emroussel commented Jul 17, 2018

This allows users to override default row styles on tables with a style object.

Since the object was spread before the defaults, there was no way to override them, and users had to use a class.

Fixes #1171

I looked if this change should be made in other places, and didn't see anything, but please let me know if I missed something!

Copy link
Contributor

@wuweiweiwu wuweiweiwu left a comment

Choose a reason for hiding this comment

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

LTGM. Thanks for contributing!

@emroussel
Copy link
Contributor Author

@wuweiweiwu any chance you get this merged and released soon?

@wuweiweiwu wuweiweiwu merged commit 6739027 into bvaughn:master Oct 3, 2018
@wuweiweiwu
Copy link
Contributor

I'll push out a minor release soon! @emroussel

@emroussel
Copy link
Contributor Author

Thanks!

errendir added a commit to IdeaFlowCo/react-virtualized that referenced this pull request Oct 24, 2018
* bvaughn/master: (54 commits)
  Update version and changelog for 9.21.0 release (bvaughn#1252)
  chore: update lockfile
  Update ci badge (bvaughn#1227)
  Allow users to override default table row styles (bvaughn#1175)
  Add onColumnClick to Table (bvaughn#1207)
  remove unused variable in Masonry.example.js (bvaughn#1218)
  Fix Table aria attributes (bvaughn#1208)
  Fix typo in CellMeasurer.DynamicHeightTableColumn.example.js (bvaughn#1190)
  Update usingAutoSizer.md (bvaughn#1186)
  Add an extra check for an e.target.className.indexOf function (bvaughn#1210)
  Fix broken Slack badge image (bvaughn#1205)
  docs(CellMeasurer): fix `import` statement (bvaughn#1187)
  Added new friend (bvaughn#1197)
  Fix createMultiSort bug (bvaughn#1051)
  adding new usecase example and fix some typos (bvaughn#1168)
  Updating version to 9.20.1
  Update changelog for the 9.20.1 release (bvaughn#1167)
  Prevent early debounceScrollEndedCallback when there is a slow render (bvaughn#1141)
  removing sideEffects (bvaughn#1163)
  fix for bvaughn#998 with test cases (bvaughn#1154)
  ...
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