[evolution/gnome-3-36] em-formatter: Remove unneeded attachment-expander button's 'data' attribute
- From: Milan Crha <mcrha src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution/gnome-3-36] em-formatter: Remove unneeded attachment-expander button's 'data' attribute
- Date: Thu, 9 Apr 2020 10:17:37 +0000 (UTC)
commit c3253db11b1b50a438af265d122a200c56b6cfe6
Author: Milan Crha <mcrha redhat com>
Date: Thu Apr 9 12:14:38 2020 +0200
em-formatter: Remove unneeded attachment-expander button's 'data' attribute
The 'data' attribute contained HTML code, which had been used a bit later
in the HTML structure and, more importantly, it was not accessed anywhere,
thus it only made the HTML code larger with no gain. Also, the attribute
value could cause HTML structure breakage, when the 'data' attribute
contained double quotes.
src/em-format/e-mail-formatter-attachment.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/src/em-format/e-mail-formatter-attachment.c b/src/em-format/e-mail-formatter-attachment.c
index 9a186618d7..a91871475a 100644
--- a/src/em-format/e-mail-formatter-attachment.c
+++ b/src/em-format/e-mail-formatter-attachment.c
@@ -260,7 +260,7 @@ emfe_attachment_format (EMailFormatterExtension *extension,
"<table width=\"100%%\" border=\"0\" style=\"border-spacing: 0px\">"
"<tr valign=\"middle\">"
"<td align=\"left\" width=\"1px\" style=\"white-space:pre;\">"
- "<button type=\"button\" class=\"attachment-expander\" id=\"%s\" value=\"%p\" data=\"%s\"
style=\"vertical-align:middle; margin:0px;\">"
+ "<button type=\"button\" class=\"attachment-expander\" id=\"%s\" value=\"%p\"
style=\"vertical-align:middle; margin:0px;\">"
"<img id=\"attachment-expander-img-%p\" src=\"gtk-stock://%s?size=%d\" width=\"%dpx\"
height=\"%dpx\" style=\"vertical-align:middle;\">"
"<img src=\"%s\" width=\"%dpx\" height=\"%dpx\" style=\"vertical-align:middle;\">"
"</button>"
@@ -268,7 +268,7 @@ emfe_attachment_format (EMailFormatterExtension *extension,
"<img src=\"gtk-stock://x-evolution-arrow-down?size=%d\" width=\"%dpx\" height=\"%dpx\"
style=\"vertical-align:middle;\">"
"</button>"
"</td><td align=\"left\">%s</td></tr>",
- part_id, attachment_ptr, html, attachment_ptr,
+ part_id, attachment_ptr, attachment_ptr,
e_mail_part_should_show_inline (part) ? "go-down" : e_mail_part_attachment_get_expandable
(empa) ? "go-next" : "go-top",
GTK_ICON_SIZE_BUTTON, icon_width, icon_height,
icon_uri, icon_width, icon_height,
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]