D18199: Add Kotlin (.kt) mimetype icons

2019-01-13 Thread Nathaniel Graham 
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:33ca64ca5c0d: Add Kotlin (.kt) mimetype icons (authored 
by trickyricky26, committed by ngraham).

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18199?vs=49379=49396

REVISION DETAIL
  https://phabricator.kde.org/D18199

AFFECTED FILES
  icons-dark/mimetypes/16/text-x-kotlin.svg
  icons-dark/mimetypes/22/text-x-kotlin.svg
  icons-dark/mimetypes/32/text-x-kotlin.svg
  icons-dark/mimetypes/64/text-x-kotlin.svg
  icons/mimetypes/16/text-x-kotlin.svg
  icons/mimetypes/22/text-x-kotlin.svg
  icons/mimetypes/32/text-x-kotlin.svg
  icons/mimetypes/64/text-x-kotlin.svg

To: trickyricky26, #vdg, ngraham, ndavis
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-13 Thread Noah Davis 
ndavis accepted this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg, ngraham, ndavis
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-13 Thread Nathaniel Graham 
ngraham accepted this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg, ngraham
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-13 Thread TrickyRicky 
trickyricky26 edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg, ngraham
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-13 Thread TrickyRicky 
trickyricky26 updated this revision to Diff 49379.
trickyricky26 added a comment.
This revision is now accepted and ready to land.


  - Use the gradient on the 32px version as well

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18199?vs=49353=49379

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

AFFECTED FILES
  icons-dark/mimetypes/16/text-x-kotlin.svg
  icons-dark/mimetypes/22/text-x-kotlin.svg
  icons-dark/mimetypes/32/text-x-kotlin.svg
  icons-dark/mimetypes/64/text-x-kotlin.svg
  icons/mimetypes/16/text-x-kotlin.svg
  icons/mimetypes/22/text-x-kotlin.svg
  icons/mimetypes/32/text-x-kotlin.svg
  icons/mimetypes/64/text-x-kotlin.svg

To: trickyricky26, #vdg, ngraham
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-12 Thread TrickyRicky 
trickyricky26 planned changes to this revision.
trickyricky26 added a comment.


  I'm going to add the gradient to the `32px` as well

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg, ngraham
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-12 Thread Nathaniel Graham 
ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  That's much better! I like the subtle gradient.
  
  #VDG  folks, is everyone else good with 
this?

REPOSITORY
  R266 Breeze Icons

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg, ngraham
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-12 Thread TrickyRicky 
trickyricky26 added a subscriber: alex-l.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg
Cc: alex-l, ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-12 Thread TrickyRicky 
trickyricky26 edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-12 Thread TrickyRicky 
trickyricky26 updated this revision to Diff 49353.
trickyricky26 added a comment.


  - Use a nicer purple color, similar to the official logo

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18199?vs=49301=49353

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

AFFECTED FILES
  icons-dark/mimetypes/16/text-x-kotlin.svg
  icons-dark/mimetypes/22/text-x-kotlin.svg
  icons-dark/mimetypes/32/text-x-kotlin.svg
  icons-dark/mimetypes/64/text-x-kotlin.svg
  icons/mimetypes/16/text-x-kotlin.svg
  icons/mimetypes/22/text-x-kotlin.svg
  icons/mimetypes/32/text-x-kotlin.svg
  icons/mimetypes/64/text-x-kotlin.svg

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-11 Thread TrickyRicky 
trickyricky26 edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-11 Thread TrickyRicky 
trickyricky26 updated this revision to Diff 49301.
trickyricky26 edited the summary of this revision.
trickyricky26 added a comment.


  - Use a lighter color for 16px and 22px

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18199?vs=49293=49301

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

AFFECTED FILES
  icons-dark/mimetypes/16/text-x-kotlin.svg
  icons-dark/mimetypes/22/text-x-kotlin.svg
  icons-dark/mimetypes/32/text-x-kotlin.svg
  icons-dark/mimetypes/64/text-x-kotlin.svg
  icons/mimetypes/16/text-x-kotlin.svg
  icons/mimetypes/22/text-x-kotlin.svg
  icons/mimetypes/32/text-x-kotlin.svg
  icons/mimetypes/64/text-x-kotlin.svg

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-11 Thread TrickyRicky 
trickyricky26 edited the summary of this revision.
trickyricky26 edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-11 Thread Nathaniel Graham 
ngraham added a reviewer: VDG.
ngraham added a comment.


  Pretty pictures? :) As discussed in chat, you'll need to submit a merge 
request to add support in `shared-mime-info` too.
  
  Here is a commit where someone else added a new mimetype: 
https://gitlab.freedesktop.org/xdg/shared-mime-info/commit/7dea7e65c4f6872d70d1b17141caa11196e6405b
  
  Hopefully you can use that as a template/inspiration for yours.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D18199

To: trickyricky26, #vdg
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D18199: Add Kotlin (.kt) mimetype icons

2019-01-11 Thread TrickyRicky 
trickyricky26 created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
trickyricky26 requested review of this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  add-kotlin-icons (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18199

AFFECTED FILES
  icons-dark/mimetypes/16/text-x-kotlin.svg
  icons-dark/mimetypes/22/text-x-kotlin.svg
  icons-dark/mimetypes/32/text-x-kotlin.svg
  icons-dark/mimetypes/64/text-x-kotlin.svg
  icons/mimetypes/16/text-x-kotlin.svg
  icons/mimetypes/22/text-x-kotlin.svg
  icons/mimetypes/32/text-x-kotlin.svg
  icons/mimetypes/64/text-x-kotlin.svg

To: trickyricky26
Cc: kde-frameworks-devel, michaelh, ngraham, bruns