Bug 364449 - Regression: after updating frameworks and plasma kmail is missing the "sent" icon
Summary: Regression: after updating frameworks and plasma kmail is missing the "sent" ...
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: 5.6.5
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-18 10:33 UTC by Christian (Fuchs)
Modified: 2016-06-26 18:53 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Lacking icons (right side of the image, in the list) (7.23 KB, image/png)
2016-06-18 10:34 UTC, Christian (Fuchs)
Details
old icon (4.45 KB, image/png)
2016-06-24 23:05 UTC, Christian (Fuchs)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christian (Fuchs) 2016-06-18 10:33:21 UTC
After updating to plasma 5.6.5.1 and frameworks 5.23  kmail is missing the icon for "sent" e-mails. 

See the attached screenshot for details. 

Before the update the old (coloured) icon was used.

Reproducible: Always

Steps to Reproduce:
1. Update to the mentioned frameworks and plasma versions
2. Start kmail
3. Open the sent folder

Actual Results:  
Icons are missing

Expected Results:  
Icons are there

kmail is from kdepim 16.04.2
Comment 1 Christian (Fuchs) 2016-06-18 10:34:18 UTC
Created attachment 99562 [details]
Lacking icons (right side of the image, in the list)
Comment 2 andreas 2016-06-19 20:25:55 UTC
Hi Fuchs,

thanks for the bug report, cause I don't use kmail it's always good to get feedback. 

In this bug I think it's not a bug, cause when you select an item (blue selection) the icon color change to the color of the text (white). This is new (and awesome).
Comment 3 Christian (Fuchs) 2016-06-19 20:28:59 UTC
(In reply to andreas from comment #2)
> Hi Fuchs,

Hi Andreas.

> In this bug I think it's not a bug, cause when you select an item (blue
> selection) the icon color change to the color of the text (white). This is
> new (and awesome).

Please re-read the bug report and attachment description. 
It is a bug, the icon that is lacking is on the _right_ side, in the list of e-mails.
 
Thanks, kind regards, 

Christian
Comment 4 andreas 2016-06-20 05:42:09 UTC
ah sorry don't saw it.
Comment 5 Christian (Fuchs) 2016-06-20 05:42:54 UTC
No worries, thanks for having a look and fixing :)
Comment 6 andreas 2016-06-24 22:41:56 UTC
I think it's a kmail bug
Comment 7 Christian (Fuchs) 2016-06-24 22:49:14 UTC
debatable, given it wasn't kmail I updated when the bug occurred and it working in older versions. I recommend at least for now fixing it where it broke.
Comment 8 andreas 2016-06-24 23:02:10 UTC
I use kde neon dev unstable and there I cant see the text. I saw some white icons but it didn't look like in your screen shot and in addition I can't remember that I had removed one of those icons in the past. So I can't reproduce to bug. please stay also in contact with the kmail team, maybe they can say me which icon was used there.
Comment 9 Christian (Fuchs) 2016-06-24 23:04:59 UTC
I have an older kubuntu at paw, I'll give you another screenshot of how it used to look when it worked.
Comment 10 Christian (Fuchs) 2016-06-24 23:05:32 UTC
Created attachment 99681 [details]
old icon
Comment 11 Thomas Pfeiffer 2016-06-25 07:58:21 UTC
I can tell which icon it is: It's 16x16/status/mail-sent in Oxygen, which indeed does not exist in Breeze.

It has nothing to do with the icon coloring, it really is a missing icon which is used to indicate sent mails (although not always that one is used, not sure why that is, but that's unrelated to this bug)
Comment 12 andreas 2016-06-26 18:52:34 UTC
Git commit 0da410b517e21407afd839e5e59f956c24f6ac01 by andreas kainz.
Committed on 26/06/2016 at 18:51.
Pushed by andreask into branch 'master'.

fix bug 364449 thanks for reporting it

A  +1    -0    icons-dark/actions/16/mail-read.svg
A  +1    -0    icons-dark/actions/16/mail-sent.svg
A  +1    -0    icons-dark/actions/16/mail-task.svg
A  +1    -0    icons-dark/actions/22/mail-sent.svg
A  +1    -0    icons-dark/actions/22/mail-task.svg
A  +1    -0    icons/actions/16/mail-read.svg
A  +1    -0    icons/actions/16/mail-sent.svg
A  +1    -0    icons/actions/16/mail-task.svg
A  +1    -0    icons/actions/22/mail-sent.svg
A  +1    -0    icons/actions/22/mail-task.svg

http://commits.kde.org/breeze-icons/0da410b517e21407afd839e5e59f956c24f6ac01
Comment 13 andreas 2016-06-26 18:53:37 UTC
thank's for reporting this bug, yes this icons were still missing (also in the past) as I don't use kmail thanks for the feedback.