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

[docs] More Table TypeScript demos #15086

Merged
merged 8 commits into from
Mar 31, 2019
Merged

[docs] More Table TypeScript demos #15086

merged 8 commits into from
Mar 31, 2019

Conversation

jasondashwang
Copy link
Contributor

@jasondashwang jasondashwang commented Mar 28, 2019

@mui-pr-bot
Copy link

mui-pr-bot commented Mar 28, 2019

No bundle size changes comparing 2dedcc1...602e53f

Generated by 🚫 dangerJS against 602e53f

@jasondashwang jasondashwang changed the title More Table TS Demos [Docs] More Table TS Demos Mar 28, 2019
@jasondashwang
Copy link
Contributor Author

jasondashwang commented Mar 28, 2019

I am having trouble with some of the VirtualizedTable demos - listed some questions below.

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation typescript labels Mar 28, 2019
@oliviertassinari oliviertassinari changed the title [Docs] More Table TS Demos [docs] More Table TS Demos Mar 28, 2019
@@ -118,6 +118,7 @@ function CustomPaginationActionsTable() {
createData('Oreo', 437, 18.0),
].sort((a, b) => (a.calories < b.calories ? -1 : 1)),
);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was auto generated by the yarn docs:typescript:formatted even though the TS version does not have these spaces

Copy link
Member

Choose a reason for hiding this comment

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

That's fine. This is a transpilation artifact. The whitespace is inserted because the types added additional whitespace in the TS version and babel tries to align the corresponding lines. Since JS naturally has less lines whitespace gets inserted

@jasondashwang
Copy link
Contributor Author

@joshwooding I don't understand this error. I did not change anything with GridList to my knowledge.

@joshwooding
Copy link
Member

@jasondashwang It’s a problem with our visual regression tests. Thanks for your hard work!

@@ -118,6 +118,7 @@ function CustomPaginationActionsTable() {
createData('Oreo', 437, 18.0),
].sort((a, b) => (a.calories < b.calories ? -1 : 1)),
);

Copy link
Member

Choose a reason for hiding this comment

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

That's fine. This is a transpilation artifact. The whitespace is inserted because the types added additional whitespace in the TS version and babel tries to align the corresponding lines. Since JS naturally has less lines whitespace gets inserted

@eps1lon eps1lon mentioned this pull request Mar 31, 2019
@eps1lon eps1lon changed the title [docs] More Table TS Demos [docs] More Table TypeScript demos Mar 31, 2019
@eps1lon eps1lon merged commit 837c8b8 into mui:next Mar 31, 2019
@eps1lon
Copy link
Member

eps1lon commented Mar 31, 2019

@jasondashwang Much appreciated. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants