Skip to content

Commit afb83cf

Browse files
committed
Improve UX for multiple “no longer verified” members of a group.
// FREEBIE
1 parent f1e5be4 commit afb83cf

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

Signal/src/ViewControllers/ShowGroupMembersViewController.m

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,20 +136,21 @@ - (void)updateTableContents
136136
@"Title for the 'no longer verified' section of the 'group members' view.");
137137
membersSection.headerTitle = NSLocalizedString(
138138
@"GROUP_MEMBERS_SECTION_TITLE_MEMBERS", @"Title for the 'members' section of the 'group members' view.");
139-
[self addMembers:noLongerVerifiedRecipientIds toSection:noLongerVerifiedSection];
139+
[self addMembers:noLongerVerifiedRecipientIds toSection:noLongerVerifiedSection useVerifyAction:YES];
140140
[contents addSection:noLongerVerifiedSection];
141141
}
142142

143143
NSMutableSet *memberRecipientIds = [self.memberRecipientIds mutableCopy];
144144
[memberRecipientIds removeObject:[helper localNumber]];
145-
[self addMembers:memberRecipientIds.allObjects toSection:membersSection];
145+
[self addMembers:memberRecipientIds.allObjects toSection:membersSection useVerifyAction:NO];
146146
[contents addSection:membersSection];
147147

148148
self.contents = contents;
149149
}
150150

151151
- (void)addMembers:(NSArray<NSString *> *)recipientIds
152-
toSection:(OWSTableSection *)section
152+
toSection:(OWSTableSection *)section
153+
useVerifyAction:(BOOL)useVerifyAction
153154
{
154155
OWSAssert(recipientIds);
155156
OWSAssert(section);
@@ -190,7 +191,11 @@ - (void)addMembers:(NSArray<NSString *> *)recipientIds
190191
}
191192
customRowHeight:[ContactTableViewCell rowHeight]
192193
actionBlock:^{
193-
[weakSelf didSelectRecipientId:recipientId];
194+
if (useVerifyAction) {
195+
[weakSelf verifySafetyNumber:recipientId];
196+
} else {
197+
[weakSelf didSelectRecipientId:recipientId];
198+
}
194199
}]];
195200
}
196201
}

0 commit comments

Comments
 (0)