nettika/geary
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Starring message causes attachment icon to disappear: Closes #6056

Browse source
This commit is contained in:
Tiago Quelhas 2013-01-16 14:27:01 -08:00 • committed by Jim Nelson
parent a300fb564d
commit c476669ff5
1 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/client/views/conversation-viewer.vala
View file

@ -303,7 +303,8 @@ public class ConversationViewer : Gtk.Box {
warning("Error setting HTML for message: %s", html_error.message); warning("Error setting HTML for message: %s", html_error.message);
} }
// Add the attachments container if we have any attachments. // Set attachment icon and add the attachments container if we have any attachments.
set_attachment_icon(div_message, email.attachments.size > 0);
if (email.attachments.size > 0) { if (email.attachments.size > 0) {
insert_attachments(div_message, email.attachments); insert_attachments(div_message, email.attachments);
} }
@ -386,6 +387,15 @@ public class ConversationViewer : Gtk.Box {
} }
} }
private void set_attachment_icon(WebKit.DOM.HTMLElement container, bool show) {
try {
WebKit.DOM.DOMTokenList class_list = container.get_class_list();
Util.DOM.toggle_class(class_list, "attachment", show);
} catch (Error e) {
warning("Failed to set attachment icon: %s", e.message);
}
}
public void update_flags(Geary.Email email) { public void update_flags(Geary.Email email) {
// Nothing to do if we aren't displaying this email. // Nothing to do if we aren't displaying this email.
if (!email_to_element.has_key(email.id)) { if (!email_to_element.has_key(email.id)) {
@ -408,7 +418,6 @@ public class ConversationViewer : Gtk.Box {
WebKit.DOM.DOMTokenList class_list = container.get_class_list(); WebKit.DOM.DOMTokenList class_list = container.get_class_list();
Util.DOM.toggle_class(class_list, "read", !flags.is_unread()); Util.DOM.toggle_class(class_list, "read", !flags.is_unread());
Util.DOM.toggle_class(class_list, "starred", flags.is_flagged()); Util.DOM.toggle_class(class_list, "starred", flags.is_flagged());
Util.DOM.toggle_class(class_list, "attachment", email.attachments.size > 0);
} catch (Error e) { } catch (Error e) {
warning("Failed to set classes on .email: %s", e.message); warning("Failed to set classes on .email: %s", e.message);
} }