Skip to content

Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.#10378

Merged
cjcenizal merged 2 commits into
elastic:masterfrom
cjcenizal:bug/fixed-table-column-widths
Feb 16, 2017
Merged

Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.#10378
cjcenizal merged 2 commits into
elastic:masterfrom
cjcenizal:bug/fixed-table-column-widths

Conversation

@cjcenizal
Copy link
Copy Markdown
Contributor

@cjcenizal cjcenizal commented Feb 15, 2017

  • Fix ellipsis-truncation of text in IE11 and Edge.
  • Update Visualize landing table markup.

Partially addresses #10283.

Sorting the different columns change their size - causes the columns to "jump around"

sort-column-jump

Browsers

Edge

image

IE11

image

OS X, Chrome

image

OS X, Safari

image

OS X, Firefox

image

@cjcenizal cjcenizal force-pushed the bug/fixed-table-column-widths branch 2 times, most recently from 0e3b0bc to 3ea9cd4 Compare February 15, 2017 19:50
…ths, regardless of cell content.

- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.
@cjcenizal cjcenizal force-pushed the bug/fixed-table-column-widths branch from 3ea9cd4 to e423918 Compare February 15, 2017 19:52
@cjcenizal cjcenizal added Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. bug Fixes for quality problems that affect the customer experience v5.3.0 v5.4.0 v6.0.0 labels Feb 15, 2017
@cjcenizal cjcenizal requested a review from BigFunger February 16, 2017 01:03
@epixa
Copy link
Copy Markdown
Contributor

epixa commented Feb 16, 2017

CSS is literally the most awful thing ever created by humans.

LGTM

@epixa
Copy link
Copy Markdown
Contributor

epixa commented Feb 16, 2017

Actually, do you need to adjust the dashboard styles as well? The checkbox alignment looks good in visualize (and in the users/roles pr), but dashboard looks like this:

screen shot 2017-02-16 at 9 42 07 am

@cjcenizal
Copy link
Copy Markdown
Contributor Author

@epixa Great catch. Looks like I forgot to push a commit.

@epixa
Copy link
Copy Markdown
Contributor

epixa commented Feb 16, 2017

LGTM for real this time

@cjcenizal cjcenizal merged commit bfadc5e into elastic:master Feb 16, 2017
@cjcenizal cjcenizal deleted the bug/fixed-table-column-widths branch February 16, 2017 16:45
elastic-jasper added a commit that referenced this pull request Feb 16, 2017
…ths, regardless of cell content.

Backports PR #10378

**Commit 1:**
Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.
- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.

* Original sha: e423918
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:45:49Z

**Commit 2:**
Update Dashboard landing table and Saved Objects table with cell liner markup.

* Original sha: 4e0f431
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:55:33Z
elastic-jasper added a commit that referenced this pull request Feb 16, 2017
…ths, regardless of cell content.

Backports PR #10378

**Commit 1:**
Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.
- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.

* Original sha: e423918
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:45:49Z

**Commit 2:**
Update Dashboard landing table and Saved Objects table with cell liner markup.

* Original sha: 4e0f431
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:55:33Z
cjcenizal pushed a commit that referenced this pull request Feb 16, 2017
…ths, regardless of cell content. (#10401)

Backports PR #10378

**Commit 1:**
Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.
- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.

* Original sha: e423918
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:45:49Z

**Commit 2:**
Update Dashboard landing table and Saved Objects table with cell liner markup.

* Original sha: 4e0f431
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:55:33Z
cjcenizal pushed a commit that referenced this pull request Feb 16, 2017
…ths, regardless of cell content. (#10400)

Backports PR #10378

**Commit 1:**
Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.
- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.

* Original sha: e423918
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:45:49Z

**Commit 2:**
Update Dashboard landing table and Saved Objects table with cell liner markup.

* Original sha: 4e0f431
* Authored by CJ Cenizal <cj@cenizal.com> on 2017-02-15T19:55:33Z
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
…ths, regardless of cell content. (elastic#10378)

* Adjust Table component styles and markup to maintain fixed column widths, regardless of cell content.
- Fix ellipsis-truncation of text in IE11 and Edge.
- Update Visualize landing table markup.

* Update Dashboard landing table and Saved Objects table with cell liner markup.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. v5.3.0 v5.4.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants