Skip to content

Commit b49800d

Browse files
committed
Merge pull request #37 from 10gen/toggle-type-minicharts
click type bars to toggle minicharts
2 parents 91f95ca + 5150b79 commit b49800d

File tree

3 files changed

+15
-1
lines changed

3 files changed

+15
-1
lines changed

scout-ui/src/field-list/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ var ViewSwitcher = require('ampersand-view-switcher');
66
var _ = require('lodash');
77

88
var BasicFieldView = View.extend({
9+
props: {
10+
minichartModel: 'state'
11+
},
912
bindings: {
1013
'model._id': [
1114
{
@@ -52,6 +55,7 @@ var BasicFieldView = View.extend({
5255
// @todo currently only support boolean, number, date, category
5356
if (['objectid', 'boolean', 'number', 'date', 'string'].indexOf(type) === -1) return;
5457

58+
this.minichartModel = typeModel;
5559
var miniview = new MinichartView({
5660
model: typeModel,
5761
});

scout-ui/src/field-list/type-list-item.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ module.exports = AmpersandView.extend({
3030
}
3131
]
3232
},
33+
events: {
34+
'click .schema-field-wrapper': 'typeClicked'
35+
},
3336
derived: {
3437
percent: {
3538
deps: ['model.probability'],
@@ -45,5 +48,11 @@ module.exports = AmpersandView.extend({
4548
});
4649
}.bind(this), 300));
4750
},
48-
template: require('./type-list-item.jade')
51+
template: require('./type-list-item.jade'),
52+
typeClicked: function(evt) {
53+
if (this.parent.parent.minichartModel.cid !== this.model.cid) {
54+
this.parent.parent.switchView(this.model);
55+
}
56+
}
57+
4958
});

scout-ui/src/home/index.less

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@
141141
.schema-field-wrapper {
142142
float: left;
143143
overflow: hidden;
144+
cursor: pointer;
144145
}
145146

146147
.schema-field-type {

0 commit comments

Comments
 (0)