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(react-grid): Put the dx-react-grid dependencies in order #148

Merged
merged 17 commits into from
Jun 20, 2017

Conversation

dxbykov
Copy link
Contributor

@dxbykov dxbykov commented Jun 16, 2017

No description provided.

@dxbykov dxbykov requested a review from viterobk June 16, 2017 08:32
@dxbykov dxbykov changed the title fix(react-grid): Move core packages to the grid peer dependencies fix(react-grid): Put the dx-react-grid dependencies in order Jun 19, 2017
@dxbykov dxbykov requested review from kvet and viterobk June 19, 2017 11:13
@@ -12,21 +12,27 @@ The DevExtreme React Grid component allows you to display table data with a set
npm i @devexpress/dx-react-grid --save
```

This package does not contain any visual components, so all examples provided below use the DevExtreme React Grid Bootstrap3 package to use Bootstrap rendering for the Grid visual components.
This package does not contain any visual components; all examples below use the DevExtreme React Grid Bootstrap3 package to use Bootstrap rendering for the Grid visual components.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"all examples below use the DevExtreme React Grid Bootstrap3 package to use Bootstrap rendering for the Grid visual components" is a confusing sentence. Consider rewriting it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can omit the second part of the sentence since it's obvious that if we use the BS3 package then BS3 will be used for rendering the Grid.

Copy link
Contributor

Choose a reason for hiding this comment

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

all examples below import modules from the DevExtreme React Grid Bootstrap 3 package to use Bootstrap rendering for the Grid visual components

@dxbykov dxbykov merged commit fe60801 into DevExpress:master Jun 20, 2017
@dxbykov dxbykov deleted the fix-dependencies branch June 20, 2017 08:55
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.

4 participants