[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Breeze-icons and Plasma Breeze-theme-icons
From: "kainz.a" <kainz.a () gmail ! com>
Date: 2015-08-28 12:47:44
Message-ID: CAFryEO4u+cV6-o72zdxrbTpgpiAxgHPADSTOa1qgwHBBZ8X-2g () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Thanks for the replay
2015-08-28 14:31 GMT+02:00 David Edmundson <david@davidedmundson.co.uk>:
>
>
> On Tue, Aug 25, 2015 at 10:24 AM, kainz.a <kainz.a@gmail.com> wrote:
>
>> Ok I would prefere to have the icons at one place because Uri and I do a
>> lot of work that the right icon in the right SIZE is available for the
>> application in the screenshot you see toolbar icons (22px) for the widgets
>> (64px) and this lack.
>>
>> if something is using the "wrong" icon, please open a bug /email pointing
> to the misuse and the name of what icon should be used instead, and that
> case will be fixed within hours.
>
>
I will make bug reports and I will learn to fix it myself and push it to
reviewboard. Sometimes we have to make new icons in the right category with
a new name so it could happen that you have to update kde:breeze where the
icons are to get the new icons. but before an release everything would be
fine.
> From our code we just write iconName = "activities" or something, and
> there's no indication of whether it's meant to be used for a toolbar or
> what without a visual check. There is a folder structure in the icon set,
> but when we write this we can't see it. Not helped that it seems to have
> changed over the last 5 years.
>
>
I was yesterday the first time in the source and saw that you don't get an
information about the icon "Type". The VDG discuss the plasmoid icons
yesterday and the result was to make colored app icons (64px) for all
plasmoids so an plasmoid is from our point of view the same than an
application. What we would suggest is that the plasmoid icons get an prefix
like plasmoid-xxx like the preferences-xxx icons for system settings.
>
>
>
>>
>> https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/PlasmaDark_BreezeLight.png
>>
>> In addition there is one problem. Cause you can use breeze-light icons in
>> breeze-dark plasma you can't see the icons because the light breeze icons
>> are not realy visible on the dark plasma theme. I don't have an solution
>> for that maybe the see also section in the system settings (future) can
>> help.
>>
>>
> So in the current state:
> - Plasma icons get recoloured
> - normal icons do not
>
> Our icons come from both places
>
> If a user has a light applications theme and a dark plasma theme, they get
> screwed.
>
> Our options are:
>
> We make the the icon theme a property of the Plasma theme?
> - Arguably within the colors file, so it can change in multiple
> colorscopes
>
> We make you guys put all icons ever possibly used by Plasma inside the
> plasma repo which does the colour hinting
> (and fix all apples to actually use Plasma.IconItem in all cases - never
> QIconItem, which some places currently do)
>
> Put the colour hinting into the Breeze icon set in breeze; and have a
> custom QIconLoader used by the Plasma case to do the hinting whilst loading
> it from the same directory as everything else.
>
>
most of the problems with breeze is that the icons are monochrome and the
user get in trouble when the contrast is bad. It would be really nice to
find an soulution that not only plasma can recolour the icons also the
application could. This would be a BIG visual benefit for the user. So if
it would be possible tell me what I have to do.
https://forum.kde.org/viewtopic.php?f=285&t=127663&start=60#p340602
http://wstaw.org/m/2015/08/28/colorize-icons.png
Thanks
Andreas
[Attachment #5 (text/html)]
<div dir="ltr">Thanks for the replay<br><div><div class="gmail_extra"><br><div \
class="gmail_quote">2015-08-28 14:31 GMT+02:00 David Edmundson <span dir="ltr"><<a \
href="mailto:david@davidedmundson.co.uk" \
target="_blank">david@davidedmundson.co.uk</a>></span>:<br><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br><div \
class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Aug 25, 2015 \
at 10:24 AM, kainz.a <span dir="ltr"><<a href="mailto:kainz.a@gmail.com" \
target="_blank">kainz.a@gmail.com</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Ok I would prefere to have the \
icons at one place because Uri and I do a lot of work that the right icon in the \
right SIZE is available for the application in the screenshot you see toolbar icons \
(22px) for the widgets (64px) and this \
lack.<br><br></div></div></blockquote></span><div>if something is using the \
"wrong" icon, please open a bug /email pointing to the misuse and the name \
of what icon should be used instead, and that case will be fixed within \
hours.<br><br></div></div></div></div></div></blockquote><div><br><span class="">I \
will make bug reports and I will learn to fix it myself and push it to reviewboard. \
Sometimes we have to make new icons in the right category with a new name so it could \
happen that you have to update kde:breeze where the icons are to get the new icons. \
but before an release everything would be fine.</span><br> </div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div class="gmail_extra"><div \
class="gmail_quote"><div></div><div>From our code we just write iconName = \
"activities" or something, and there's no indication of whether \
it's meant to be used for a toolbar or what without a visual check. There is a \
folder structure in the icon set, but when we write this we can't see it. Not \
helped that it seems to have changed over the last 5 years.<br></div><span \
class=""><div><br></div></span></div></div></div></div></blockquote><div><br></div><div>I \
was yesterday the first time in the source and saw that you don't get an \
information about the icon "Type". The VDG discuss the plasmoid icons \
yesterday and the result was to make colored app icons (64px) for all plasmoids so an \
plasmoid is from our point of view the same than an application. What we would \
suggest is that the plasmoid icons get an prefix like plasmoid-xxx like the \
preferences-xxx icons for system settings.<br> </div><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><br><div dir="ltr"><div><div \
class="gmail_extra"><div class="gmail_quote"><span class=""><div> </div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><a \
href="https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/PlasmaDark_BreezeLight.png" \
target="_blank">https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/PlasmaDark_BreezeLight.png</a><br><br></div><div>In \
addition there is one problem. Cause you can use breeze-light icons in breeze-dark \
plasma you can't see the icons because the light breeze icons are not realy \
visible on the dark plasma theme. I don't have an solution for that maybe the see \
also section in the system settings (future) can \
help.<br><br></div></div></blockquote><br></span></div><div class="gmail_quote">So in \
the current state:<br></div><div class="gmail_quote"> - Plasma icons get \
recoloured<br></div><div class="gmail_quote"> - normal icons do not \
</div><br></div><div class="gmail_extra">Our icons come from both \
places<br><br></div><div class="gmail_extra">If a user has a light applications theme \
and a dark plasma theme, they get screwed.<br></div><div \
class="gmail_extra"><br></div><div class="gmail_extra">Our options \
are:<br><br></div>We make the the icon theme a property of the Plasma theme?<br><div \
class="gmail_extra"> - Arguably within the colors file, so it can change in multiple \
colorscopes<br><br></div><div>We make you guys put all icons ever possibly used by \
Plasma inside the plasma repo which does the colour hinting<br></div><div> (and fix \
all apples to actually use Plasma.IconItem in all cases - never QIconItem, which some \
places currently do)<br></div><div><br></div><div> Put the colour hinting into the \
Breeze icon set in breeze; and have a custom QIconLoader used by the Plasma case to \
do the hinting whilst loading it from the same directory as everything else.<span \
class=""><font color="#888888"><br></font></span></div></div><span class=""><font \
color="#888888"><br></font></span></div></blockquote><div><br></div><div>most of the \
problems with breeze is that the icons are monochrome and the user get in trouble \
when the contrast is bad. It would be really nice to find an soulution that not only \
plasma can recolour the icons also the application could. This would be a BIG visual \
benefit for the user. So if it would be possible tell me what I have to do.<br><br><a \
href="https://forum.kde.org/viewtopic.php?f=285&t=127663&start=60#p340602">htt \
ps://forum.kde.org/viewtopic.php?f=285&t=127663&start=60#p340602</a><br><a \
href="http://wstaw.org/m/2015/08/28/colorize-icons.png">http://wstaw.org/m/2015/08/28/colorize-icons.png</a><br></div><div> \
</div>Thanks<br></div><div \
class="gmail_quote">Andreas<br></div><br></div></div></div>
[Attachment #6 (text/plain)]
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic