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

pr #5

Merged
merged 5 commits into from
Feb 20, 2021
Merged

pr #5

merged 5 commits into from
Feb 20, 2021

Conversation

anuragxxd
Copy link
Owner

Summary

Provide a detailed summary of your PR. Explain how you arrived at your solution. If it includes changes to UI elements include a screenshot or gif.

Checklist

  • Check against all themes for compatibility in both light and dark modes
  • Checked in mobile
  • Checked in Chrome, Safari, Edge, and Firefox
  • Props have proper autodocs and playground toggles
  • Added documentation
  • Checked Code Sandbox works for the any docs examples
  • Added or updated jest tests
  • Checked for breaking changes and labeled appropriately
  • Checked for accessibility including keyboard-only and screenreader modes
  • A changelog entry exists and is marked appropriately

cchaos and others added 5 commits February 18, 2021 15:39
* Adding all the colors for default theme too
* Added `hasBorder` prop
* Increasing contrast of dark mode page background color
* Panel with border only for amsterdam theme
* Replace static Id with htmlIdGenerator

* Remove id from quick_select_popover.test.tsx.snap

* Remove static Id from quick_select_popover.tsx

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Greg Thompson <thompson.glowe@gmail.com>
* Update data_grid.tsx

support sever side rendering

* Update src/components/datagrid/data_grid.tsx

Co-authored-by: Chandler Prall <chandler.prall@gmail.com>

* add changelog

* revert wrong commit

* Revert "revert wrong commit"

This reverts commit 87bf299.

Co-authored-by: Chandler Prall <chandler.prall@gmail.com>
@anuragxxd anuragxxd merged commit 9f63594 into anuragxxd:master Feb 20, 2021
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.

6 participants