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.

Sign up for GitHub

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

<perspective-viewer> UI cleanup #905

Merged
merged 10 commits into from
Feb 5, 2020
Merged

<perspective-viewer> UI cleanup #905

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
77781f8
Fixes hypergrid console spamming when removed from DOM.
texodus Feb 2, 2020
b1cf3a8
Pre-size plugin on responsive column check.
texodus Feb 3, 2020
2fd8fe1
Fixes over-aggressive attribute reflection in `perspective-viewer`, a…
texodus Feb 3, 2020
b6bae10
Fixes #878
texodus Feb 3, 2020
a3dd975
Adds support for `Promise<Table>` to `PerspectiveViewerHTMLElement.lo…
texodus Feb 3, 2020
84d741a
Style improvements, move commas to end of items in a list.
texodus Feb 4, 2020
718f172
Better tab styling & other visual fixes.
texodus Feb 3, 2020
3eba691
Fix bug in `perspective-toggle-config` event dispatch order.
texodus Feb 3, 2020
b8015e9
Fixed `perspective-viewer` computed column UX overdraw
texodus Feb 4, 2020
2ed7814
Fixed tests
texodus Feb 4, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixes #878
  • Loading branch information
@texodus
texodus committed Feb 3, 2020
commit b6bae10fde46bc92d148b18264faa70f72c75f18
4 changes: 2 additions & 2 deletions packages/perspective-viewer/src/js/viewer.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -675,14 +675,14 @@ class PerspectiveViewer extends ActionElement {
* Clears the rows in the current {@link table}.
*/
clear() {
this._table.clear();
this._table?.clear();
}

/**
* Replaces all rows in the current {@link table}.
*/
replace(data) {
this._table.replace(data);
this._table ? this._table.replace(data) : this._load(data);
}

/**
Expand Down