yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-11-18 22:04:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

adds feature to show all potential member data fields in readonly mode, filters to contained member data fields by default.

Browse Source
This commit is contained in:
Seyi Adebajo 2017-08-20 18:53:01 -07:00
parent ef1c1df63b
commit 9bbf77fe99
2 changed files with 63 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
wherehows-web/app/components/dataset-compliance.js
View File

@ -293,6 +293,12 @@ export default Component.extend({
.every(({ value }) => [true, false].includes(value)); .every(({ value }) => [true, false].includes(value));
}), }),
/**
* Determines if all member data fields should be shown in the member data table i.e. show only fields contained in
* this dataset or otherwise
*/
isShowingAllMemberData: computed.or('showAllDatasetMemberData', 'isEditing'),
/** /**
* Determines if the save feature is allowed for the current dataset, otherwise e.g. interface should be disabled * Determines if the save feature is allowed for the current dataset, otherwise e.g. interface should be disabled
* @type {Ember.computed} * @type {Ember.computed}
@ -650,6 +656,13 @@ export default Component.extend({
); );
}, },
/**
* Sets the flag to show all member potential member data fields that may be contained in this dataset
*/
onShowAllDatasetMemberData() {
return set(this, 'showAllDatasetMemberData', true);
},
/** /**
* Handle the user intent to place this compliance component in edit mode * Handle the user intent to place this compliance component in edit mode
*/ */

54
wherehows-web/app/templates/components/dataset-compliance.hbs
View File

@ -101,19 +101,22 @@
</header> </header>
</section> </section>
<table class="nacho-table nacho-table--bordered nacho-table--stripped"> {{#dataset-table
<thead> class="nacho-table nacho-table--bordered nacho-table--stripped"
<tr> fields=datasetClassification as |table|}}
<th>Types of member data</th>
<th class="dataset-field-content__header"> {{#table.head as |head|}}
{{#head.column}}Types of member data{{/head.column}}
{{#head.column class="dataset-field-content__header"}}
<span class="dataset-field-value">Is this type of member data contained in this dataset?</span> <span class="dataset-field-value">Is this type of member data contained in this dataset?</span>
</th> {{/head.column}}
</tr> {{/table.head}}
</thead>
<tbody> {{#table.body as |body|}}
{{#each datasetClassification as |classification|}} {{#each (if isShowingAllMemberData table.data (filter-by "value" true table.data)) as |classification|}}
<tr> {{#body.row as |row|}}
<td class="dataset-field-content__prompt"> {{#row.cell class="dataset-field-content__prompt"}}
<span class="dataset-tag-container"> <span class="dataset-tag-container">
{{#if (eq classification.value true)}} {{#if (eq classification.value true)}}
@ -143,8 +146,9 @@
{{classification.label}} {{classification.label}}
</span> </span>
</td> {{/row.cell}}
<td class="dataset-field-content__input">
{{#row.cell class="dataset-field-content__input"}}
<span class="dataset-field-value"> <span class="dataset-field-value">
{{#radio-button-composer {{#radio-button-composer
value=true value=true
@ -160,12 +164,24 @@
changed=(action "onChangeDatasetClassification")}} changed=(action "onChangeDatasetClassification")}}
No No
{{/radio-button-composer}} {{/radio-button-composer}}
</span> </span>{{/row.cell}}
</td> {{/body.row}}
</tr>
{{/each}} {{/each}}
</tbody> {{/table.body}}
</table>
{{#unless isShowingAllMemberData}}
{{#table.foot}}
<td colspan="2" class="text-center">
<button
{{action "onShowAllDatasetMemberData"}}
class="nacho-button--large nacho-button--tertiary">
See More <span class="caret" aria-label="See More Dataset Member Data"></span>
</button>
</td>
{{/table.foot}}
{{/unless}}
{{/dataset-table}}
<section class="metadata-prompt"> <section class="metadata-prompt">
<header class="metadata-prompt__header"> <header class="metadata-prompt__header">