D20623: Update audio, drive, edit, go, list, media, plasmavault icons to match breeze-icons

2019-04-16 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REPOSITORY R242 Plasma Framework (Library) BRANCH update-icons (branched from master) REVISION DETAIL

D20597: Align z's to pixel grid in system.svg

2019-04-16 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R242:0ff81fe4c728: Align zs to pixel grid in system.svg (authored by ndavis). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20597?vs=56356=56365

D20597: Align z's to pixel grid in system.svg

2019-04-16 Thread Noah Davis
ndavis edited the summary of this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D20597 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D20597: Align z's to pixel grid in system.svg

2019-04-16 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY This does the same thing as D20586 , but for the

D20586: Sharpen Z's in system-suspend* icons

2019-04-16 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:db9fdceac1ac: Sharpen Zs in system-suspend* icons (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20586?vs=56331=56355 REVISION DETAIL

D20586: Sharpen Z's in system-suspend* icons

2019-04-15 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20586 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D20586: Sharpen Z's in system-suspend* icons

2019-04-15 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20586 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D20586: Sharpen Z's in system-suspend* icons

2019-04-15 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY They weren't aligned to the pixel grid before. REPOSITORY R266 Breeze Icons BRANCH

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-04-15 Thread Noah Davis
ndavis added a comment. In D20140#450349 , @GB_2 wrote: > Ping Sorry, I'm just busy right now and I'm not that familiar with QML, so it won't be quick and easy for me. REPOSITORY R296 KDeclarative REVISION DETAIL

D20426: Improve "widget-alternatives" icon

2019-04-13 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. This revision is now accepted and ready to land. I can't think of any more suggestions that would be an obvious improvement, so I'll accept this. REPOSITORY R266 Breeze Icons BRANCH master REVISION DETAIL

D20426: Improve "widget-alternatives" icon

2019-04-13 Thread Noah Davis
ndavis added a comment. Can you flip the direction of the 16px version? I know the icon you're copying the design of is also like this, but I think it needs to be fixed. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20426 To: GB_2, #vdg Cc: ndavis,

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:aaa6451d308e: Add go-up/down/next/previous-skip (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20435?vs=55938=55939 REVISION DETAIL

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis edited the summary of this revision. REPOSITORY R266 Breeze Icons BRANCH more-go (branched from master) REVISION DETAIL https://phabricator.kde.org/D20435 To: ndavis, #vdg, #atelier, ngraham Cc: abetts, ngraham, laysrodrigues, kde-frameworks-devel, michaelh, bruns

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis updated this revision to Diff 55938. ndavis edited the summary of this revision. ndavis added a comment. - Turn window-keep-* into symlinks to go-up/down-skip REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20435?vs=55899=55938 BRANCH more-go

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis added a comment. Now I'll need to turn `window-keep-above` and `window-keep-below` into symlinks to `go-up-skip` and `go-down-skip`. `arc patch` hates it when files or symlinks are converted to each other, so I'll hold off doing that until this gets accepted. REPOSITORY R266

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis edited the summary of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20435 To: ndavis, #vdg Cc: laysrodrigues, kde-frameworks-devel, michaelh, ngraham, bruns

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis edited the summary of this revision. ndavis edited the test plan for this revision. ndavis added a subscriber: laysrodrigues. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20435 To: ndavis, #vdg Cc: laysrodrigues, kde-frameworks-devel, michaelh, ngraham,

D20435: Add go-up/down/next/previous-skip

2019-04-10 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY These are meant to be used when moving by the normal amount (go-up/down/next/previous) is

D19736: Add view-orientation-auto/landscape/portrait

2019-04-09 Thread Noah Davis
ndavis retitled this revision from "Add screen-rotate-auto-on/off" to "Add view-orientation-auto/landscape/portrait". ndavis edited the summary of this revision. ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19736

D19736: Add screen-rotate-auto-on/off

2019-04-09 Thread Noah Davis
ndavis updated this revision to Diff 55804. ndavis added a comment. - Change to 3 icons REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19736?vs=53826=55804 BRANCH rotate (branched from master) REVISION DETAIL https://phabricator.kde.org/D19736

D20226: Add "notifications" and "notifications-disabled" icons

2019-04-04 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. Those curves are perfect  REPOSITORY R266 Breeze Icons BRANCH add-notifications-and-notifications-disabled-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D20226 To: GB_2, #plasma, #vdg, ngraham, ndavis

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-04-03 Thread Noah Davis
ndavis commandeered this revision. ndavis edited reviewers, added: arvidhansson; removed: ndavis. ndavis added a comment. Alright. I'll finish it when I have time. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19952 To: ndavis, #vdg, arvidhansson Cc: ngraham,

D20226: Add "notifications" and "notifications-disabled" icons

2019-04-03 Thread Noah Davis
ndavis added a comment. In D20226#442934 , @bruns wrote: > Diagonal or round shapes are **not** required to be pixel grid aligned. Actually, it is not even possible, as there is **always** some antialiasing. It even helps to make round shapes a

D20226: Add "notifications" and "notifications-disabled" icons

2019-04-03 Thread Noah Davis
ndavis added a comment. In D20226#442906 , @GB_2 wrote: > In D20226#442882 , @ngraham wrote: > > > Also, could we create a 22x22 version too? > > > How can you efficiently do that without

D20226: Add "notifications" and "notifications-disabled" icons

2019-04-03 Thread Noah Davis
ndavis added a comment. The proportions aren't quite right. F6749114: Screenshot_20190403_112343.png This is one way to make a perfectly smooth bell. Subtract the red shapes from the black shapes. F6749117: Screenshot_20190403_112550.png

D20166: Keep desktoptheme SVG files uncompressed in repo, install svgz

2019-04-01 Thread Noah Davis
ndavis added a comment. In D20166#441863 , @kossebau wrote: > IMHO the SVG files in the repo may contain comments and metadatas, for further work on them. Like source code also has all the oomments and metadata. > Only the deployed graphics

D20166: Keep desktoptheme SVG files uncompressed in repo, install (stripped) svgz

2019-04-01 Thread Noah Davis
ndavis added a comment. In D20166#441807 , @ngraham wrote: > Strictly speaking, there's no need to optimize the SVGs as a part of this, so maybe, yeah, let's split that out into a separate feature (and one that the Breeze Icons repo would

D20166: Keep desktoptheme SVG files uncompressed in repo, install (stripped) svgz

2019-04-01 Thread Noah Davis
ndavis added a comment. Be careful that you are not stripping stylesheets when you use SVG Cleaner. `scour` is less effective than SVG Cleaner, but it doesn't strip stylesheets and it's available in more distros. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL

D20147: Improve executable dialog buttons' icons

2019-03-31 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. Yes, this is much better. Wow, that `application-x-executable` icon is really blurry. REPOSITORY R241 KIO BRANCH improve-execute-dialog-icons (branched from master) REVISION DETAIL https://phabricator.kde.org/D20147 To: ngraham,

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-31 Thread Noah Davis
ndavis added a comment. Another alternative would be to turn the thumbnail outline into a real outline instead of a rounded rectangle behind the thumbnailArea. Then it could go on top of the thumbnailArea and the thumbnail highlight could fit perfectly inside of it. This would give previews

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-31 Thread Noah Davis
ndavis added a comment. Here is how no radius for the thumbnailArea and thumbnail highlight looks: F6739553: Screenshot_20190331_131729.png REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D20140 To: ndavis, #plasma, #vdg

D20063: Sublime Merge Icon.

2019-03-31 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:69d7182daffc: Sublime Merge Icon. (authored by otavva, committed by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20063?vs=54981=55140 REVISION

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-31 Thread Noah Davis
ndavis added a comment. In D20140#440989 , @ngraham wrote: > It's probably going to be more noticeable for High DPI users. It seems like there must be a way to fix this rather than just removing the thumbnail radius. I could remove the

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-30 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D20140 To: ndavis, #plasma, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-30 Thread Noah Davis
ndavis updated this revision to Diff 55106. ndavis added a comment. - Remove radius from thumbnailArea highlight REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20140?vs=55105=55106 BRANCH grid-delegate-rectangle (branched from master) REVISION

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-30 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D20140 To: ndavis, #plasma, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D20140: [GridDelegate] Fix gaps in corners of thumbnailArea highlight

2019-03-30 Thread Noah Davis
ndavis created this revision. ndavis added reviewers: Plasma, VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY There was a small gap in the corners of the thumbnailArea highlight rectangle that

D20063: Sublime Merge Icon.

2019-03-29 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. @otavva I need your email and real name to land your patch with the correct author. If you use the Arcanist tool for patches in the future, you will need to make sure that your real name is set as your Git user name. REPOSITORY R266

D20063: Sublime Merge Icon.

2019-03-27 Thread Noah Davis
ndavis added a comment. Oh wait no, it's not there yet, but it still looks pretty good. However, I did notice this on a closer inspection of the shapes: F6726111: Screenshot_20190327_182820.png @otavva, could you make the top shape more like

D20063: Sublime Merge Icon.

2019-03-27 Thread Noah Davis
ndavis added a comment. In D20063#439465 , @ngraham wrote: > I don't that version is actually in the patch though. @otavva can you update this patch to have that version of the icon? Then we can get it landed for you. No, it's there,

D20063: Sublime Merge Icon.

2019-03-27 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. This revision is now accepted and ready to land. This is very nice. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D20063 To: otavva, #vdg, ndavis Cc: trickyricky26, filipf, ngraham, #vdg,

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-26 Thread Noah Davis
ndavis added a comment. In D19952#439031 , @arvidhansson wrote: > In D19952#438965 , @ndavis wrote: > > > The rounded corners and bold 1 are not very breeze like, so I don't think so, unless he

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-26 Thread Noah Davis
ndavis added a comment. The rounded corners and bold 1 are not very breeze like, so I don't think so, unless he changes his design. Unless you still plan to do the icon yourself, you should probably hand off responsibility for the patch to someone else. REPOSITORY R266 Breeze Icons

D12327: Show Detailed Tree View by default

2019-03-26 Thread Noah Davis
ndavis added a comment. +1 Navigation is so much faster in tree view. I think navigation speed is what matters most of the time when people are selecting files or locations. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12327 To: ngraham, #frameworks, #vdg, apol

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-25 Thread Noah Davis
ndavis added a comment. In D19952#438130 , @arvidhansson wrote: > In D19952#437726 , @ndavis wrote: > > > In D19952#437670 , @arvidhansson wrote: > >

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-24 Thread Noah Davis
ndavis added a comment. In D19952#437670 , @arvidhansson wrote: > In D19952#437419 , @ndavis wrote: > > > This can be done in the system tray widget instead of making a separate icon for off. Ever

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-24 Thread Noah Davis
ndavis added a comment. In D19952#436853 , @arvidhansson wrote: > In D19952#436834 , @taskf wrote: > > > There is one issue I'm not sure about. The user might feel pressured to check something if

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-22 Thread Noah Davis
ndavis added a comment. Here's my attempt: F6711669: montage-breeze-Background.png F6711673: input-numlock-on.16.svg F6711674: input-numlock-on.22.svg REPOSITORY

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-22 Thread Noah Davis
ndavis added a comment. In D19952#436425 , @taskf wrote: > Hi, I created some icons a week ago, just didn't post them as @arvidhansson was already on the task. > > They are not polished as you can see from the screenshot. The left version

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-21 Thread Noah Davis
ndavis added a comment. You need to make the icons on 16x16px and 22x22px canvases. If you use a grid with 1 line every pixel, it will be easier to make icons look sharp. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19952 To: arvidhansson, #vdg, ndavis Cc:

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-21 Thread Noah Davis
ndavis added a reviewer: ndavis. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19952 To: arvidhansson, #vdg, ndavis Cc: ndavis, kde-frameworks-devel, michaelh, ngraham, bruns

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-21 Thread Noah Davis
ndavis added a comment. That "1" is too blurry. Did you use the font tool or did you draw it with the pen or rectangle tools? Since this is "numlock" Wouldn't it make more sense to put a "1" inside of a lock? REPOSITORY R266 Breeze Icons REVISION DETAIL

D19952: Added 2 icons (1 16px and 1 22px) named "input-num-on" that shows a num-lock key

2019-03-21 Thread Noah Davis
ndavis added a reviewer: VDG. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19952 To: arvidhansson, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D19736: Add screen-rotate-auto-on/off

2019-03-19 Thread Noah Davis
ndavis added a comment. In D19736#434274 , @ngraham wrote: > Maybe the rectangle itself doesn't need to be so big? Or maybe it could be even bigger and have the arrows inside? I can make the rectangle slightly smaller. I can't make the

D19736: Add screen-rotate-auto-on/off

2019-03-19 Thread Noah Davis
ndavis added a comment. In D19736#434616 , @apol wrote: > How about having 3 icons? autorotate, locked portrait and locked landscape? > > This way we don't need to have a weird red strike-through. That's what I asked about the first

D19736: Add screen-rotate-auto-on/off

2019-03-18 Thread Noah Davis
ndavis added a comment. This is what the current 32px version with arrowheads added looks like, but it can't be done like that with 16px or 22px. F6701152: montage.png REPOSITORY R266 Breeze Icons REVISION DETAIL

D19736: Add screen-rotate-auto-on/off

2019-03-18 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19736 To: ndavis, #vdg, apol Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D19736: Add screen-rotate-auto-on/off

2019-03-18 Thread Noah Davis
ndavis added a comment. @ngraham Here's something similar to what you asked for: F6701127: Screenshot_20190318_235338.png This is 22px. I couldn't get it to work for 16px. REPOSITORY R266 Breeze Icons REVISION DETAIL

D19812: Add a web page to view and compare icons of different sizes

2019-03-18 Thread Noah Davis
ndavis added a comment. In D19812#433941 , @guoyunhe wrote: > In D19812#433916 , @ndavis wrote: > > > Could you use `#eff0f1` and `#31363b` for the light and dark backgrounds since those are the

D19812: Add a web page to view and compare icons of different sizes

2019-03-18 Thread Noah Davis
ndavis added a comment. Could you use `#eff0f1` and `#31363b` for the light and dark backgrounds since those are the colors we normally use for window backgrounds? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19812 To: guoyunhe, ngraham, #vdg, ndavis,

D17469: Give applications-games and input-gaming more contrast with Breeze Dark

2019-03-16 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:559782c5f5b4: Give applications-games and input-gaming more contrast with Breeze Dark (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowdecorations icons

2019-03-15 Thread Noah Davis
ndavis accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH arcpatch-D19733 REVISION DETAIL https://phabricator.kde.org/D19733 To: GB_2, #vdg, ndavis Cc: ngraham, ndavis, kde-frameworks-devel, #vdg, michaelh, bruns

D17469: Give applications-games and input-gaming more contrast with Breeze Dark

2019-03-14 Thread Noah Davis
ndavis retitled this revision from "Give applications-games more contrast with Breeze Dark" to "Give applications-games and input-gaming more contrast with Breeze Dark". REPOSITORY R266 Breeze Icons BRANCH arcpatch-D17469 (branched from master) REVISION DETAIL

D17469: Give applications-games more contrast with Breeze Dark

2019-03-14 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons BRANCH arcpatch-D17469 (branched from master) REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe, ngraham Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh,

D17469: Give applications-games more contrast with Breeze Dark

2019-03-14 Thread Noah Davis
ndavis updated this revision to Diff 53925. ndavis added a comment. Update input-gaming and make circle around dpad and colored buttons darker REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17469?vs=53839=53925 BRANCH arcpatch-D17469 (branched from

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowdecorations icons

2019-03-14 Thread Noah Davis
ndavis added a comment. In D19733#431237 , @GB_2 wrote: > In D19733#431139 , @ndavis wrote: > > > I think `preferences-desktop-theme-applications` is fine, but

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowdecorations icons

2019-03-14 Thread Noah Davis
ndavis added a comment. I think `preferences-desktop-theme-applications` is fine, but `preferences-desktop-theme-windowdecorations` looks odd with that border. F6691545: Screenshot_20190314_135511.png REVISION DETAIL

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis added a comment. In D17469#430647 , @ngraham wrote: > Hmm, I see what you mean. Okay, let's go with this one! > > BTW, the duplicate test fails as of f2509047d4372f1a59d8f8b5ff009146f1a6adcb

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis added a comment. In D17469#430616 , @ngraham wrote: > No room? Instead of widening the grips by one pixel to make room for the contrast border, how about building it in, effectively narrowing the grips by one pixel? That's what I

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh, bruns

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh, bruns

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis updated this revision to Diff 53839. ndavis added a comment. Darken the handle grips REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17469?vs=53836=53839 BRANCH arcpatch-D17469 (branched from master) REVISION DETAIL

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis added a comment. In D17469#430573 , @ngraham wrote: > Better, but the dark parts of the dark version still disappear against a dark background. Most of the time we solve this with a transparent outline (we did this for the clock icon

D19736: Add screen-rotate-auto-on/off

2019-03-13 Thread Noah Davis
ndavis added a comment. In D19736#430545 , @ngraham wrote: > Hmm, TBH it feels too busy for me, especially at the smaller sizes. I wonder if it might benefit from using a single circular line with an arrowhead instead, like this comically crude

D17469: Give applications-games more contrast with Breeze Dark

2019-03-13 Thread Noah Davis
ndavis retitled this revision from "applications-games icon for dark theme" to "Give applications-games more contrast with Breeze Dark". REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe Cc: abetts, ngraham, pstefan, ndavis,

D17469: applications-games icon for dark theme

2019-03-13 Thread Noah Davis
ndavis edited the summary of this revision. ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh, bruns

D17469: applications-games icon for dark theme

2019-03-13 Thread Noah Davis
ndavis updated this revision to Diff 53836. ndavis added a comment. Change style for Breeze and Breeze Dark REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17469?vs=47254=53836 BRANCH arcpatch-D17469 (branched from master) REVISION DETAIL

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowborders icons

2019-03-13 Thread Noah Davis
ndavis added a comment. We did agree to have Qt and GTK settings separate in T8871 REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19733 To: GB_2, #vdg Cc: ngraham, ndavis, kde-frameworks-devel, #vdg, michaelh, bruns

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowborders icons

2019-03-13 Thread Noah Davis
ndavis added a comment. BTW, when you change an icon into a symlink or vice versa, can you delete the icon you're changing in a separate commit and then add the new version of the icon? `arc patch` doesn't work when a file has been changed from or to a symlink. REPOSITORY R266 Breeze

D19733: Add preferences-desktop-theme-applications and preferences-desktop-theme-windowborders icons

2019-03-13 Thread Noah Davis
ndavis added a comment. I don't think the light/dark window background looks good. It also gives the wrong message because the Colors KCM is what controls the Window background. The Window Decoration KCM icon has the same issue, but that already exists. Perhaps you should put buttons or

D19736: Add screen-rotate-auto-on/off

2019-03-13 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19736 To: ndavis, #vdg, apol Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D19736: Add screen-rotate-auto-on/off

2019-03-13 Thread Noah Davis
ndavis updated this revision to Diff 53826. ndavis edited the test plan for this revision. ndavis added a comment. - Increase spacing between outside rotation marks and screen REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19736?vs=53825=53826 BRANCH

D19736: Add screen-rotate-auto-on/off

2019-03-13 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19736 To: ndavis, #vdg, apol Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D19736: Add screen-rotate-auto-on/off

2019-03-13 Thread Noah Davis
ndavis created this revision. ndavis added reviewers: VDG, apol. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY Screen auto rotate icons for D19690 REPOSITORY

D17469: applications-games icon for dark theme

2019-03-13 Thread Noah Davis
ndavis commandeered this revision. ndavis edited reviewers, added: guoyunhe; removed: ndavis. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: ndavis, #breeze, #vdg, guoyunhe Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh, bruns

D17469: applications-games icon for dark theme

2019-03-12 Thread Noah Davis
ndavis added a comment. @guoyunhe Do you mind if I commandeer this? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17469 To: guoyunhe, #breeze, #vdg, ndavis Cc: abetts, ngraham, pstefan, ndavis, kde-frameworks-devel, michaelh, bruns

D19575: Add collapse/expand-all, window-shade/unshade

2019-03-12 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:b128afbbfe62: Add collapse/expand-all, window-shade/unshade (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19575?vs=53746=53751

D19575: Add collapse/expand-all, window-shade/unshade

2019-03-12 Thread Noah Davis
ndavis retitled this revision from "Add collapse-all and expand-all action icons" to "Add collapse/expand-all, window-shade/unshade". ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19575 To: ndavis, #vdg, ngraham,

D19575: Add collapse-all and expand-all action icons

2019-03-12 Thread Noah Davis
ndavis updated this revision to Diff 53746. ndavis added a comment. - Add window-shade/unshade REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19575?vs=53744=53746 BRANCH arcpatch-D19575 REVISION DETAIL https://phabricator.kde.org/D19575

D19575: Add collapse-all and expand-all action icons

2019-03-12 Thread Noah Davis
ndavis updated this revision to Diff 53744. ndavis added a comment. Lower down arrow on expand-all REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19575?vs=53522=53744 BRANCH arcpatch-D19575 REVISION DETAIL https://phabricator.kde.org/D19575

D19575: Add collapse-all and expand-all action icons

2019-03-11 Thread Noah Davis
ndavis commandeered this revision. ndavis edited reviewers, added: trickyricky26; removed: ndavis. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19575 To: ndavis, #vdg, ngraham, trickyricky26 Cc: ndavis, ngraham, kde-frameworks-devel, michaelh, bruns

D19575: Add collapse-all and expand-all action icons

2019-03-11 Thread Noah Davis
ndavis added a comment. @trickyricky26 do you mind if I commandeer this so I can get it finished today? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19575 To: trickyricky26, #vdg, ngraham, ndavis Cc: ndavis, ngraham, kde-frameworks-devel, michaelh, bruns

D19633: Improve consistency of window-* and add more

2019-03-10 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:3bc4079c220e: Improve consistency of window-* and add more (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19633?vs=53571=53607 REVISION

D19633: Improve consistency of window-* and add more

2019-03-10 Thread Noah Davis
ndavis added a comment. window-shade can be added in a later patch so that this one isn't unnecessarily held back REPOSITORY R266 Breeze Icons BRANCH window-states (branched from master) REVISION DETAIL https://phabricator.kde.org/D19633 To: ndavis, #vdg, ngraham Cc: ngraham,

D19633: Improve consistency of window-* and add more

2019-03-09 Thread Noah Davis
ndavis edited the summary of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19633 To: ndavis, #vdg Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D19633: Improve consistency of window-* and add more

2019-03-09 Thread Noah Davis
ndavis updated this revision to Diff 53571. ndavis added a comment. - Add window-unpin-symbolic REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19633?vs=53569=53571 BRANCH window-states (branched from master) REVISION DETAIL

D19633: Improve consistency of window-* and add more

2019-03-09 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19633 To: ndavis, #vdg Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D19633: Improve consistency of window-* and add more

2019-03-09 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19633 To: ndavis, #vdg Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D19633: Improve consistency of window-* and add more

2019-03-09 Thread Noah Davis
ndavis updated this revision to Diff 53569. ndavis added a comment. - Add window-unpin REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D19633?vs=53516=53569 BRANCH window-states (branched from master) REVISION DETAIL

D19638: Make go-bottom/first/last/top look more like media-skip*

2019-03-09 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:24067c74c2d6: Make go-bottom/first/last/top look more like media-skip* (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE

<    4   5   6   7   8   9   10   11   12   13   >