Re: Review Request 125104: When using scalable icons use still only use sizes defined by index metadata

2015-09-08 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125104/#review85013 --- Ship it! Ship It! - andreas kainz On Sept. 8, 2015, 2:27

Re: Review Request 128065: move glass theme to kde-look.

2016-06-10 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128065/#review96339 --- Ship it! Ship It! - Andreas Kainz On Mai 31, 2016, 8

Re: Review Request 126740: Add a script for optimizing svgs

2016-01-14 Thread andreas kainz
the icons use therefore an script (check if the stylesheets fit, compress, make png files, ) - andreas kainz On Jan. 14, 2016, 3 vorm., Aleix Pol Gonzalez wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread andreas kainz
> On Feb. 1, 2016, 8:12 p.m., andreas kainz wrote: > > Hi, I know I say I will keep oxygen-icons5 up to date, and I hope I get the > > time I need the plasma oxygen theme too to give the user the one single > > click feature (look & feel package). Is there an

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread andreas kainz
> On Feb. 1, 2016, 8:12 p.m., andreas kainz wrote: > > Hi, I know I say I will keep oxygen-icons5 up to date, and I hope I get the > > time I need the plasma oxygen theme too to give the user the one single > > click feature (look & feel package). Is there an

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread andreas kainz
oxygen is also really sexy in addition a lot of uers work with kde4 (which is unmainted too) and update to kf5 with Kubuntu 16.04 LTS. It would be bad to support oxygen for an additional LTS cycle, but for the switch it would be nice to give the user the posibility for an soft switch. - andreas

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-01 Thread andreas kainz
> On Feb. 1, 2016, 8:12 p.m., andreas kainz wrote: > > Hi, I know I say I will keep oxygen-icons5 up to date, and I hope I get the > > time I need the plasma oxygen theme too to give the user the one single > > click feature (look & feel package). Is there an

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-02 Thread andreas kainz
> On Feb. 1, 2016, 8:12 nachm., andreas kainz wrote: > > Hi, I know I say I will keep oxygen-icons5 up to date, and I hope I get the > > time I need the plasma oxygen theme too to give the user the one single > > click feature (look & feel package). Is there an

Re: Review Request 126949: Remove Air and Oxygen themes

2016-02-02 Thread andreas kainz
> On Feb. 2, 2016, 9:17 vorm., Marco Martin wrote: > > maybe not oxygen, but i want to point out that i *do* maintain the Air theme > > Martin Klapetek wrote: > Then we got some fixing to do as this is how Air looks like here: > http://paste.opensuse.org/view/raw/6071be9f (also notice the

Re: Review Request 126523: Use a KMessageWidget in KNewPasswordDialog

2016-02-29 Thread Andreas Kainz
https://bugs.kde.org/show_bug.cgi?id=357210 but I am not sure that knewpassworddialog use an icon cause there is no x icon. is this icon used from KNewPasswordDialog? can it be colored in white #f2f2f2. thanks for the feedback. maybe you can answer on the bug report. - Andreas Kainz On Feb. 1

Re: Review Request 126523: Use a KMessageWidget in KNewPasswordDialog

2016-02-29 Thread Andreas Kainz
> On Feb. 29, 2016, 10:23 p.m., Andreas Kainz wrote: > > hi, heiko fill a bug report cause you cant see the x icon > > https://bugs.kde.org/show_bug.cgi?id=357210 but I am not sure that > > knewpassworddialog use an icon cause there is no x icon. is this icon used > &

Re: Review Request 126523: Use a KMessageWidget in KNewPasswordDialog

2016-02-29 Thread Andreas Kainz
> On Feb. 29, 2016, 10:23 p.m., Andreas Kainz wrote: > > hi, heiko fill a bug report cause you cant see the x icon > > https://bugs.kde.org/show_bug.cgi?id=357210 but I am not sure that > > knewpassworddialog use an icon cause there is no x icon. is this icon used > &

Re: Review Request 126740: Add a script for optimizing svgs

2016-02-23 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126740/#review92666 --- Ship it! Ship It! - Andreas Kainz On Feb. 16, 2016, 2

Re: Review Request 126738: Remove external dependencies from svgs

2016-01-21 Thread andreas kainz
-- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126738/ > --- > > (Updated Jan. 14, 2016, 1:18 vorm.) > > > Review request for KDE Frameworks, andreas kainz and Ur

Re: Review Request 126740: Add a script for optimizing svgs

2016-01-19 Thread andreas kainz
to svgz and run the svg optimizing script ones in future by each change the dev can say run script if wanted. BUT as we already have this stylesheet script (marco) please make one script and say me how it works. - andreas kainz On Jan. 19, 2016, 10:54 p.m., Aleix Pol Gonzalez wrote

Re: Review Request 127875: Selected state concept for icons

2016-05-09 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127875/#review95309 --- Ship it! Ship It! - Andreas Kainz On Mai 9, 2016, 3:26

Re: Review Request 127875: Selected state concept for icons

2016-05-09 Thread Andreas Kainz
> On Mai 9, 2016, 10:21 nachm., Andreas Kainz wrote: > > Ship It! I can't review the code but this is one of the best features in the past 2 years - Andreas --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 127878: use the "selected" icon mode in file open dialog sidebar

2016-05-09 Thread Andreas Kainz
of the best features in the past 2 years - Andreas Kainz On Mai 9, 2016, 4:24 nachm., Marco Martin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 127876: use the "selected" state for icons in selected menu items and selected buttons

2016-05-09 Thread Andreas Kainz
of the best features in the past 2 years +1 from the vdg - Andreas Kainz On Mai 9, 2016, 3:32 nachm., Marco Martin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 127632: Prioritize correct extension per theme

2016-04-26 Thread Andreas Kainz
don't know what this is doing. sorry. - Andreas Kainz On April 26, 2016, 7:12 nachm., Aleix Pol Gonzalez wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 126738: Remove external dependencies from svgs

2016-07-05 Thread Andreas Kainz
without troubles if you find some please let me know. otherwise you can close the request. thanks Andreas - Andreas Kainz On Jan. 14, 2016, 1:18 vorm., Dirk Hohndel wrote: > > --- > This is an automatically generated e-mail.

[Differential] [Commented On] D4495: Remove monochrome application icons

2017-02-08 Thread Andreas Kainz
andreaska added a comment. when was the last release for ktorrent? REVISION DETAIL https://phabricator.kde.org/D4495 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: guoyunhe, #breeze Cc: andreaska, #frameworks

[Differential] [Commented On] D4495: Remove monochrome application icons

2017-02-08 Thread Andreas Kainz
andreaska added a comment. please have a look at applications like amarok or ktorrent some icons are needed for this applications. REVISION DETAIL https://phabricator.kde.org/D4495 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: guoyunhe, #breeze Cc:

[Differential] [Accepted] D4251: Redesign aptana icon

2017-01-23 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. andreaska added a comment. This revision is now accepted and ready to land. submitted REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4251 EMAIL PREFERENCES

[Differential] [Accepted] D4252: fix brackets icon

2017-01-23 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4252 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

[Differential] [Commented On] D4252: fix brackets icon

2017-01-23 Thread Andreas Kainz
andreaska added a comment. I submitt the icon thanks REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4252 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, #breeze, andreaska Cc: guoyunhe, #frameworks

[Differential] [Commented On] D4253: redesign gwenview icon

2017-01-23 Thread Andreas Kainz
andreaska added a comment. first of all, thanks for starting the discussion I think also that the gwenview icon could get an update, but I miss something in the update in addition the dark eye background look to hard. REPOSITORY R266 Breeze Icons REVISION DETAIL

[Differential] [Commented On] D4258: redesign gimp icon

2017-01-24 Thread Andreas Kainz
andreaska added a comment. How does it look without pencil? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4258 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, andreask, #breeze Cc: andreaska, alex-l, #frameworks

[Differential] [Commented On] D4262: redesign android-studio icon

2017-01-24 Thread Andreas Kainz
andreaska added a comment. oh schit didn't no this. thanks but I thought only the circle is in the icon now nearly the full height and in your design only 2/3 of the icon height. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4262 EMAIL PREFERENCES

[Differential] [Accepted] D4278: fix icons bittorrent-sync, codeblocks, darktable, ffmulticonverter

2017-01-26 Thread Andreas Kainz
andreaska accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4278 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, alex-l, andreaska, #breeze Cc:

[Differential] [Commented On] D4253: redesign gwenview icon

2017-01-28 Thread Andreas Kainz
andreaska added a comment. Varlesh you do awesome icon work, but gwenview was changed some time ago so no icon redesign needed sorry. Please work on other icons. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4253 EMAIL PREFERENCES

[Differential] [Accepted] D4264: redesign android-file-transfer icon

2017-01-25 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. andreaska added a comment. This revision is now accepted and ready to land. In https://phabricator.kde.org/D4264#79949, @varlesh wrote: > @alex-l reverted gradient: > F2164722: Снимок экрана от 2017-01-25

[Differential] [Commented On] D4289: Add VLC tray icon

2017-01-26 Thread Andreas Kainz
andreaska added a comment. can you upload the svgz file cause I can't download it here. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4289 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: guoyunhe,

[Differential] [Updated] D4278: fix icons bittorrent-sync, codeblocks, darktable, ffmulticonverter

2017-01-26 Thread Andreas Kainz
andreaska added a comment. +1 bittorent sync +1 darktable +1 ffmulticonverter codeblocks is better but the hammer had to much different colors I think REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4278 EMAIL PREFERENCES

[Differential] [Accepted] D4649: add a preview for the disk quota plasmoid

2017-02-17 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. andreaska added a comment. This revision is now accepted and ready to land. thanks look good. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D4649 EMAIL PREFERENCES

[Differential] [Accepted] D4626: Renamed icons for encrypted and decripted folders

2017-02-15 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a comment. This revision is now accepted and ready to land. good point. If you want to use this icons I'll make them for the other places size. REPOSITORY R266 Breeze Icons BRANCH ivan/rename-icons REVISION DETAIL

[Differential] [Commented On] D4262: redesign android-studio icon

2017-01-24 Thread Andreas Kainz
andreaska added a comment. why is the circle now so small? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4262 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, andreask, #breeze Cc: andreaska, alex-l, #frameworks

[Differential] [Accepted] D4261: redesign atom icon

2017-01-24 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4261 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

[Differential] [Accepted] D4260: fix bitcoin128 icon

2017-01-24 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4260 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

[Differential] [Commented On] D4264: redesign android-file-transfer icon

2017-01-24 Thread Andreas Kainz
andreaska added a comment. where is this icon used? is it an application or an mimetype? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4264 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, andreask, #breeze Cc:

[Differential] [Accepted] D4267: redesign bluefish icon

2017-01-24 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. andreaska added a comment. This revision is now accepted and ready to land. ship it REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4267 EMAIL PREFERENCES

[Differential] [Commented On] D4253: redesign gwenview icon

2017-01-24 Thread Andreas Kainz
andreaska added a comment. I also think that we shouldn't update the icon. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4253 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: varlesh, #breeze, andreask Cc: alex-l,

[Differential] [Accepted] D4259: fix alienarena icon

2017-01-24 Thread Andreas Kainz
andreaska accepted this revision. andreaska added a reviewer: andreaska. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D4259 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To:

Re: Review Request 128694: Fix all edit-clear-location-ltr icons

2016-09-03 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128694/#review98836 --- the patch isn't needed any more - Andreas Kainz On Aug

Re: Review Request 128694: Fix all edit-clear-location-ltr icons

2016-09-03 Thread Andreas Kainz
) the bug. - Andreas Kainz On Aug. 15, 2016, 1:57 p.m., David Edmundson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.

Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-10 Thread Andreas Kainz
--- > > (Updated Sept. 1, 2016, 4:45 nachm.) > > > Review request for KDE Frameworks, KDE Usability and Andreas Kainz. > > > Bugs: 97420 > https://bugs.kde.org/show_bug.cgi?id=97420 > > > Repository: kwidgetsaddons > > > Description > ---

Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-13 Thread Andreas Kainz
> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote: > > Andreas, any idea which icon the action could use? > > Andreas Kainz wrote: > I would use visibility and hint to show/hide the font's that are not > available in the selected font. will the font change to

Re: Review Request 128680: [KCharSelect] Add ToolButton for disabling characters not available in selected font

2016-09-13 Thread Andreas Kainz
> On Sept. 4, 2016, 2:34 vorm., Christoph Feck wrote: > > Andreas, any idea which icon the action could use? > > Andreas Kainz wrote: > I would use visibility and hint to show/hide the font's that are not > available in the selected font. will the font change to

Re: Review Request 129106: Give a different name to Breeze Dark icon theme

2016-10-07 Thread Andreas Kainz
, please have a look if it now work propper. the problem is that it is to late for the 5.27 release. - Andreas Kainz On Okt. 7, 2016, 12:16 vorm., Aleix Pol Gonzalez wrote: > > --- > This is an automatically generated e-mail. To rep

D4917: [Breeze Plasma Theme] Improve action-overlay icons

2017-03-06 Thread Andreas Kainz
andreaska added a comment. F2753390: action-overlays.svgz only a draft (not finished, only that you can review something) F2753393: overlayScreenshot.png REPOSITORY R242 Plasma Framework (Library) REVISION

D7446: Add a Recent Documents places item to Dolphin and file pickers by default

2017-10-12 Thread Andreas Kainz
andreaska added a comment. works for the VDG really good job. well done. I like it. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7446 To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp Cc: andreaska, gregormi, markg, alexeymin,

D7520: Fix icon of KStandardAction::MoveToTrash

2017-10-12 Thread Andreas Kainz
andreaska added a comment. +1 REPOSITORY R265 KConfigWidgets REVISION DETAIL https://phabricator.kde.org/D7520 To: elvisangelaccio, #frameworks Cc: andreaska, broulik

D8034: add overflow-menu icon

2017-09-28 Thread Andreas Kainz
andreaska accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D8034 To: mart, andreask, andreaska Cc: #frameworks

D11874: change 24px view-media-artist icon

2018-06-10 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH media-artist REVISION DETAIL https://phabricator.kde.org/D11874 To: astippich, #vdg, andreask Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13137: Updated handles and overflow-menu-*

2018-05-28 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D13137 To: alex-l, mart, andreask, andreaska Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D11863: Add virt-manager icons

2018-04-29 Thread Andreas Kainz
andreask closed this revision. andreask added a comment. Thanks for the commit. I checked the icons and optimize the icon size. virt-manager is now in master. https://cgit.kde.org/breeze-icons.git/commit/?id=63450fec7aa14ba3a32972a434064c499ea048a9 REPOSITORY R266 Breeze Icons

D9815: [KFileItem] Use "emblem-shared" for shared files

2018-01-12 Thread Andreas Kainz
andreaska accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9815 To: broulik, #frameworks, dfaure, andreask, andreaska Cc: apol

D9815: [KFileItem] Use "emblem-shared" for shared files

2018-01-12 Thread Andreas Kainz
andreask added a comment. add an real emblem-shared icon cause the existing one was a symbolic one for gnome. should work now so +1 REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9815 To: broulik, #frameworks, dfaure, andreask Cc: apol

D10212: New virtualbox icon

2018-02-01 Thread Andreas Kainz
andreaska accepted this revision. REVISION DETAIL https://phabricator.kde.org/D10212 To: lshoravi, #breeze, ngraham, #vdg, andreaska Cc: ltoscano, davidc, #frameworks, michaelh, ngraham

D10211: Suggestion for emacs icon

2018-02-01 Thread Andreas Kainz
andreaska accepted this revision. REVISION DETAIL https://phabricator.kde.org/D10211 To: lshoravi, #breeze, #vdg, ngraham, andreaska Cc: davidc, michaelh, abetts, ngraham, #frameworks

D10243: Disconnected wi-fi icons are almost invisible BUG:384018

2018-02-01 Thread Andreas Kainz
andreask created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. andreask requested review of this revision. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL

D10243: Disconnected wi-fi icons are almost invisible BUG:384018

2018-02-01 Thread Andreas Kainz
andreask edited the summary of this revision. andreask added reviewers: Plasma, Plasma Accessibility, sebas, ngraham. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL

D10243: Improve visibility of disconnected Wi-Fi icon

2018-02-02 Thread Andreas Kainz
andreask retitled this revision from "Disconnected wi-fi icons are almost invisible BUG:384018" to "Improve visibility of disconnected Wi-Fi icon". andreask edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D10243

D10243: Improve visibility of disconnected Wi-Fi icons

2018-02-02 Thread Andreas Kainz
andreask retitled this revision from "Improve visibility of disconnected Wi-Fi icon" to "Improve visibility of disconnected Wi-Fi icons". REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D10243 To: andreask, #plasma, #plasma_accessibility, sebas,

D10243: Improve visibility of disconnected Wi-Fi icons

2018-02-02 Thread Andreas Kainz
andreask added a comment. F5688471: wifi icon.png top after - breeze(light) - breeze-dark bottom before - breeze(light) REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D10243 To: andreask,

D10365: New icon for Elisa music player

2018-02-07 Thread Andreas Kainz
andreask added a comment. Elisa is a new music player app (yes another one) cassettes are old-school yes, but the benefit is that it look like an old (stable well developed) app in addition it's way more unique than other music player apps. REPOSITORY R266 Breeze Icons REVISION DETAIL

D10365: New icon for Elisa music player

2018-02-13 Thread Andreas Kainz
andreask added a comment. can we get it now to master? @paullesur is hopefully interested in other icon's too. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D10365 To: paullesur, #breeze, #vdg, #elisa, andreaska, mgallien Cc: mmustac,

D10492: add 64px media icons for elisa

2018-02-13 Thread Andreas Kainz
andreask created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. andreask requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D10492 AFFECTED

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. andreask requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D10494 AFFECTED

D10492: add 64px media icons for elisa

2018-02-13 Thread Andreas Kainz
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R266:6800b88f1819: add 64px media icons for elisa (authored by andreask). REPOSITORY R266 Breeze Icons CHANGES SINCE

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask added reviewers: alex-l, mart. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10494 To: andreask, alex-l, mart Cc: #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask added a comment. the old icon with 12 dots are a bit of a overkill now with three dot's compare to three lines. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10494 To: andreask, alex-l, mart Cc: #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask added a comment. F5710538: photo5773697838952328398.jpg left after right before REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10494 To: andreask, alex-l, mart Cc: ngraham, #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask added a comment. F5710546: Screenshot_20180213_230517.png sorry wrong screenshot left new right old REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10494 To: andreask, alex-l, mart Cc: ngraham,

D10492: add 64px media icons for elisa

2018-02-13 Thread Andreas Kainz
andreask added a comment. cause I changed my workflow and start use arc but I wasn't interested into a review. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10492 To: andreask Cc: ngraham, #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-02-13 Thread Andreas Kainz
andreask added a comment. it doesn't matter where the element in kirigami swap it's for settings and have an dotted icon would only increase publicity like here in discover have three dot's in each line will give you the information that you can do something, but the focus is more on the

D10494: update handle- icons for kirigami

2018-02-15 Thread Andreas Kainz
andreask added a comment. submitted REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10494 To: andreask, alex-l, mart Cc: matheusm, mmustac, ngraham, #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-02-15 Thread Andreas Kainz
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R266:55b56d236e9b: update handle- icons for kirigami (authored by andreask). REPOSITORY R266 Breeze Icons CHANGES SINCE

D10279: add 24px media-album-cover icon

2018-02-19 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH media-album-cover REVISION DETAIL https://phabricator.kde.org/D10279 To: astippich, #breeze, #vdg, andreaska, andreask Cc: ngraham, #frameworks, michaelh

D10293: remove view-media-playlist from preferences icons

2018-02-19 Thread Andreas Kainz
andreask accepted this revision. andreask added a comment. This revision is now accepted and ready to land. I added the colored icon for elisa but I understood if you don't like it. personally I'm not 100% happy with large monochrome icons. REPOSITORY R266 Breeze Icons BRANCH

D10292: change 32px icons for playlist shuffle and repeat

2018-02-19 Thread Andreas Kainz
andreask requested changes to this revision. andreask added a comment. This revision now requires changes to proceed. only in 16px there is no circle and in general I'm not really happy to have no circle around the icon. maybe for repeat and shuffle it will work but play/pause are more

D14218: Introduce a test to make sure we're not offering duplicated icons

2018-07-23 Thread Andreas Kainz
andreaska accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL https://phabricator.kde.org/D14218 To: apol, andreaska, mart, vkrause, #breeze, #frameworks Cc: ngraham, andreask, kde-frameworks-devel, michaelh,

D14123: Add `drive-optical` icon

2018-07-16 Thread Andreas Kainz
andreaska added a comment. Thanks for the patch. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D14123 To: ngraham, broulik, andreask, #breeze, #vdg Cc: andreaska, kde-frameworks-devel, michaelh, ngraham, bruns

D11049: Remove frame around media action icons

2018-03-06 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH reworkMediaActions REVISION DETAIL https://phabricator.kde.org/D11049 To: romangg, #breeze, #vdg, andreask Cc: mart, #frameworks, michaelh

D10494: update handle- icons for kirigami

2018-04-17 Thread Andreas Kainz
andreaska added a comment. In D10494#220993 , @mart wrote: > As discussed on the channel, the icon should be one of these forms, for me are both ok > F5744901: handle-right.png I am for this

D11137: overflow-menu as view-more-symbolic, and horizontal

2018-04-17 Thread Andreas Kainz
andreaska accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH phab/overflow REVISION DETAIL https://phabricator.kde.org/D11137 To: mart, #vdg, andreaska Cc: Fuchs, #frameworks, michaelh, ngraham, bruns

D11461: rename 64 px icons added for elisa

2018-03-28 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH icons_move REVISION DETAIL https://phabricator.kde.org/D11461 To: astippich, #breeze, #vdg, andreask Cc: #frameworks, michaelh, ngraham

D10292: change 32px icons for playlist shuffle and repeat

2018-03-28 Thread Andreas Kainz
andreask accepted this revision. andreask added a comment. This revision is now accepted and ready to land. sorry for the late approve. REPOSITORY R266 Breeze Icons BRANCH playlist_shuffle_repeat REVISION DETAIL https://phabricator.kde.org/D10292 To: astippich, #breeze, #vdg,

D11863: Add virt-manager icons

2018-04-03 Thread Andreas Kainz
andreask accepted this revision. REPOSITORY R266 Breeze Icons BRANCH virt-manager_icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D11863 To: ndavis, #vdg, ngraham, andreaska, andreask Cc: ngraham, andreaska, #frameworks, michaelh

D11390: new icons for disconnected/disabled devices

2018-03-16 Thread Andreas Kainz
andreask accepted this revision. andreask added a comment. This revision is now accepted and ready to land. look good to me thanks REPOSITORY R242 Plasma Framework (Library) BRANCH phab/network REVISION DETAIL https://phabricator.kde.org/D11390 To: mart, #plasma, #vdg, andreask Cc:

D11054: Add media status icons

2018-03-06 Thread Andreas Kainz
andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH mediaStatus REVISION DETAIL https://phabricator.kde.org/D11054 To: romangg, #plasma, #frameworks, #vdg, andreask Cc: michaelh

D7446: [Places panel] Add a Recently Used item by default

2018-11-07 Thread Andreas Kainz
andreask added a comment. I would suggest to have by default detail view instead of icon view cause with detail view it's easier to read the paths. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D7446 To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg,

D15931: [Places panel] use more appropriate network icon

2018-10-04 Thread Andreas Kainz
andreaska accepted this revision. REPOSITORY R241 KIO BRANCH better-network-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D15931 To: ngraham, #vdg, nicolasfella, andreaska Cc: broulik, acrouthamel, kde-frameworks-devel, michaelh, ngraham, bruns

D18282: document-new icon follow ColorScheme-Text

2019-01-15 Thread Andreas Kainz
This revision was automatically updated to reflect the committed changes. Closed by commit R266:e3804e633e67: document-new icon follow ColorScheme-Text (authored by andreask). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18282?vs=49583=49585 REVISION

D18282: document-new icon follow ColorScheme-Text

2019-01-15 Thread Andreas Kainz
andreask created this revision. andreask added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. andreask requested review of this revision. REVISION SUMMARY icon follow now ColorScheme-Text color TEST PLAN change to breeze-dark and open

D17033: Change primary monochrome icon colors to match new HIG colors

2018-11-19 Thread Andreas Kainz
andreask added a comment. Thanks for the update, I will work with the new colors also in LibO. We have there an issue on OS-X cause the contrast was not good enough and the new darker color will give an better contrast. REPOSITORY R266 Breeze Icons REVISION DETAIL

D19512: KateStatusBar: Show lock icon on modified label when in read-only mode

2019-03-31 Thread Andreas Kainz
andreask added a comment. I had choose also an tooltip like most other apps do and there it work very well. F6740005: libreoffice-password-content.png REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D19512 To: loh.tar,