mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Show call to action on group member list when verifying

Browse source 
FREEBIE
This commit is contained in:
Scott Nonnenberg 2017-06-16 16:48:06 -07:00
parent 31bf05e14a
commit 16433d661a
5 changed files with 29 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

11
js/views/conversation_view.js
View file

@ -208,8 +208,7 @@
return;
}
// TODO: need to be able to specify string to override group list header
this.showMembers(null, unverified);
this.showMembers(null, unverified, {needVerify: true});
},
onVerifiedChange: function() {
@ -578,13 +577,17 @@
}
},
showMembers: function(e, members) {
showMembers: function(e, members, options) {
options = options || {};
_.defaults(options, {needVerify: false});
members = members || this.model.contactCollection;
var view = new Whisper.GroupMemberList({
model: members,
// we pass this in to allow nested panels
listenBack: this.listenBack.bind(this)
listenBack: this.listenBack.bind(this),
needVerify: options.needVerify
});
this.listenBack(view);

14
js/views/group_member_list_view.js
View file

@ -10,6 +10,8 @@
className: 'group-member-list panel',
templateName: 'group-member-list',
initialize: function(options) {
this.needVerify = options.needVerify;
this.render();
this.member_list_view = new Whisper.ContactListView({
@ -23,8 +25,16 @@
this.$('.container').append(this.member_list_view.el);
},
render_attributes: {
members: i18n('groupMembers')
render_attributes: function() {
var summary;
if (this.needVerify) {
summary = i18n('membersNeedingVerification');
}
return {
members: i18n('groupMembers'),
summary: summary
};
}
});
})();