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

[Table] Add truncation controls to example page #1253

Merged
merged 17 commits into from
Jun 20, 2017

Conversation

cmslewis
Copy link
Contributor

@cmslewis cmslewis commented Jun 19, 2017

Changes proposed in this pull request:

On the table playground page:

  • Add switch to enable/disable truncation
  • Add child menu to choose the truncation popover mode (ALWAYS, NEVER, WHEN_TRUNCATED).

Reviewers should focus on:

Behold how slow and laggy truncation makes the table feel. :)

I periodically see Maximum call stack size exceeded errors in the console when I change the truncation popover mode. Not sure why. Help me figure it out if you see it?

Screenshot

menus

@blueprint-bot
Copy link

Merge branch 'cl/table-example-truncation' of github.com:palantir/blueprint into cl/table-example-truncation

Preview: documentation | table
Coverage: core | datetime

@blueprint-bot
Copy link

Show disabled child elements in sidebar, use Classes everywhere

Preview: documentation | table
Coverage: core | datetime

@cmslewis cmslewis force-pushed the cl/table-example-truncation branch from fda14f1 to 424cde4 Compare June 20, 2017 19:28
@cmslewis
Copy link
Contributor Author

cmslewis commented Jun 20, 2017

@llorca @tgreenwatts I updated the code and the GIF above. See how the new enable/disable toggling feels.

EDIT: Well, wait till the latest build finishes.

@blueprint-bot
Copy link

Show disabled child elements in sidebar, use Classes everywhere

Preview: documentation | table
Coverage: core | datetime

@cmslewis cmslewis merged commit 383e032 into master Jun 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants