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

Invalid filters #375

Merged
merged 15 commits into from
Jan 14, 2019
Merged

Invalid filters #375

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6ba022a
Moved filter validation into a separate function
Ro4052 Jan 4, 2019
0c1c93b
Fixed expected/received value order
Ro4052 Jan 7, 2019
a7dcb9f
Separated data and valid filter checks
Ro4052 Jan 7, 2019
780f769
Viewer now highlights invalid filters
Ro4052 Jan 7, 2019
cfe06cb
Slight re-structure to viewer filter validation
Ro4052 Jan 8, 2019
5b40c00
Slight refactor to perspective filter validation
Ro4052 Jan 8, 2019
de8e9fb
Updated styling for invalid filters
Ro4052 Jan 9, 2019
b3b5220
Re-named filter validation function
Ro4052 Jan 9, 2019
520850b
Added some commenting for newly exposed table method
Ro4052 Jan 9, 2019
5dba849
Added tool tip to invalid filters
Ro4052 Jan 9, 2019
f90cbd1
Can no longer filter by an empty string
Ro4052 Jan 9, 2019
c300a6d
Slight refactor of filter validation function
Ro4052 Jan 9, 2019
67131a5
Added screenshot test for invalid filter highlighting
Ro4052 Jan 10, 2019
e261e5f
Added test that empty string filters are not used
Ro4052 Jan 10, 2019
ecb7d44
Added tests for is_valid_filter function
Ro4052 Jan 10, 2019
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
Updated styling for invalid filters
  • Loading branch information
@Ro4052
Ro4052 committed Jan 9, 2019
commit de8e9fbaebfa3ec43af1ad990d14b282d320f672
1 change: 1 addition & 0 deletions packages/perspective-viewer/src/html/row.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
<select id="column_aggregate" class="string"></select>
<select id="filter_operator"></select>
<input id="filter_operand" placeholder="Value" />
<span id="row_exclamation" hidden>&#x26A0;</span>
<span id='row_close'>&#x2715;</span>
</div>
</div>
Expand Down
8 changes: 6 additions & 2 deletions packages/perspective-viewer/src/js/viewer/perspective_element.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -216,14 +216,18 @@ export class PerspectiveElement extends StateElement {
const filterNodes = this._get_view_filter_nodes();
for (let i = 0; i < filterNodes.length; i++) {
const node = filterNodes[i];
const operandNode = node.shadowRoot.getElementById("filter_operand");
const exclamation = node.shadowRoot.getElementById("row_exclamation");
const name = node.getAttribute("name");
const {operator, operand} = JSON.parse(node.getAttribute("filter"));
const filter = [name, operator, operand];
if (await this._table.valid_filter(filter)) {
filters.push(filter);
node.className = "";
operandNode.style.borderColor = "";
exclamation.hidden = true;
} else {
node.className = "invalid-filter";
operandNode.style.borderColor = "red";
exclamation.hidden = false;
}
}

Expand Down
6 changes: 0 additions & 6 deletions packages/perspective-viewer/src/less/column_labels.less
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,6 @@
margin-top: 23px;
}

.invalid-filter {
border-style: dashed;
border-width: 0.1em;
border-color: red;
}

:host {
#app #top_panel {
#row_pivots label:before {
Expand Down
4 changes: 4 additions & 0 deletions packages/perspective-viewer/src/less/row.less
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,10 @@
padding: var(--sort_order-padding, 0);
}

#row_exclamation {
color: red;
}

#row_close {
color: #999;
font-family: Arial;
Expand Down