shadowfacts
/
Tusker
2
10
Fork 2
Code Issues 78 Pull Requests Releases Wiki Activity

Fix requires attachment descriptions preference not working

This commit is contained in:
Shadowfacts 2020-03-15 11:39:35 -04:00
parent c5d2e9af68
commit 18e91feb00
Signed by: shadowfacts
GPG Key ID: 94A5AB95422746E5
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Tusker/Screens/Compose/Attachments/ComposeAttachmentTableViewCell.swift
View File

@ -82,8 +82,8 @@ class ComposeAttachmentTableViewCell: UITableViewCell {
extension ComposeAttachmentTableViewCell: UITextViewDelegate { extension ComposeAttachmentTableViewCell: UITextViewDelegate {
func textViewDidChange(_ textView: UITextView) { func textViewDidChange(_ textView: UITextView) {
delegate?.attachmentDescriptionChanged(self)
attachment.attachmentDescription = textView.text attachment.attachmentDescription = textView.text
updateDescriptionPlaceholderLabel() updateDescriptionPlaceholderLabel()
delegate?.attachmentDescriptionChanged(self)
} }
} }

3
Tusker/Screens/Compose/Attachments/ComposeAttachmentsViewController.swift
View File

@ -25,13 +25,14 @@ class ComposeAttachmentsViewController: UITableViewController {
var attachments: [CompositionAttachment] = [] { var attachments: [CompositionAttachment] = [] {
didSet { didSet {
delegate?.composeSelectedAttachmentsDidChange() delegate?.composeSelectedAttachmentsDidChange()
delegate?.composeRequiresAttachmentDescriptionsDidChange()
updateAddAttachmentsButtonEnabled() updateAddAttachmentsButtonEnabled()
} }
} }
var requiresAttachmentDescriptions: Bool { var requiresAttachmentDescriptions: Bool {
if Preferences.shared.requireAttachmentDescriptions { if Preferences.shared.requireAttachmentDescriptions {
return !attachments.allSatisfy { $0.description.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty } return attachments.contains { $0.attachmentDescription.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty }
} else { } else {
return false return false
} }