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

Add compressed props on basic/memory tables #687

Merged
merged 2 commits into from
Apr 18, 2018

Conversation

snide
Copy link
Contributor

@snide snide commented Apr 18, 2018

Fixed #460

Adds in the ability to use the compressed prop on EuiBasicTable and EuiInMemoryTable

@snide
Copy link
Contributor Author

snide commented Apr 18, 2018

cc @peteharverson

@snide snide requested a review from cchaos April 18, 2018 20:50
@snide
Copy link
Contributor Author

snide commented Apr 18, 2018

@cchaos This is now rebased against your PR. I've noted the compressed prop in the propdocs but not the display docs. That page is getting super heavy now and I don't want to distract from the content.

Copy link
Contributor

@cchaos cchaos left a comment

Choose a reason for hiding this comment

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

Works.

Quick question, should we be changing the font-size of the header and pagination as well?

screen shot 2018-04-18 at 18 50 29 pm

I just realized that I changed the font size on the header at too low of a level:

screen shot 2018-04-18 at 18 48 14 pm

@snide
Copy link
Contributor Author

snide commented Apr 18, 2018

I don't think we want the font bumping down bellow 12px/.75rem. At that point it becomes hard to read for people with not perfect eyes.

I think the header actually looks fine in yours. I like it a little smaller, helps it stand out a bit 👍

@snide snide merged commit 8e5cedd into elastic:master Apr 18, 2018
@snide snide deleted the table/compressed branch April 18, 2018 23:22
@timroes
Copy link
Contributor

timroes commented May 4, 2018

@snide I think this PR forgot to add the compressed prop to the type definition of the EuiInMemoryTable?

snide added a commit to snide/eui that referenced this pull request May 11, 2018
snide added a commit that referenced this pull request May 14, 2018
* pagination labeling fixes #735
* fixes #621 adds menubar roles to side nav
* fixes #734, adds more descriptive aria labeling for pagination
* fixes #729, apply aria-live to popovers
* fixes #723, provides screen reader notice for bottom bar appearance
* fix ownFocus needing to be true for non-context menu sitations. apply aria labels depending upon ownFocus state
* fixes #616 adds home page label
* fixes #687, documentation button labeling
* fixes #617 applies aria label to theme selector
* fixes #746 applies proper label to select all checkboxes in table
* clone element and id no longer needed for popover
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