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

2016-10-07 Thread Aleix Pol Gonzalez

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129106/
---

(Updated Oct. 7, 2016, 5:04 p.m.)


Status
--

This change has been discarded.


Review request for KDE Frameworks, Alessandro Longo, Andreas Kainz, and Andreas 
Kainz.


Repository: breeze-icons


Description
---

Otherwise we get the item listed twice with a different icon.


Diffs
-

  icons-dark/index.theme c319289 

Diff: https://git.reviewboard.kde.org/r/129106/diff/


Testing
---

It's now shown properly both on the icons kcm and kde-gtk-config


Thanks,

Aleix Pol Gonzalez



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

2016-10-07 Thread Andreas Kainz

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129106/#review99837
---



hi

thanks for the fix but I already fixed it yesterday, 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 reply, visit:
> https://git.reviewboard.kde.org/r/129106/
> ---
> 
> (Updated Okt. 7, 2016, 12:16 vorm.)
> 
> 
> Review request for KDE Frameworks, Alessandro Longo, Andreas Kainz, and 
> Andreas Kainz.
> 
> 
> Repository: breeze-icons
> 
> 
> Description
> ---
> 
> Otherwise we get the item listed twice with a different icon.
> 
> 
> Diffs
> -
> 
>   icons-dark/index.theme c319289 
> 
> Diff: https://git.reviewboard.kde.org/r/129106/diff/
> 
> 
> Testing
> ---
> 
> It's now shown properly both on the icons kcm and kde-gtk-config
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>



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

2016-10-05 Thread Michael Stemle

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129106/#review99803
---


Ship it!




Ship It!

- Michael Stemle


On Oct. 5, 2016, 1:21 p.m., Aleix Pol Gonzalez wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129106/
> ---
> 
> (Updated Oct. 5, 2016, 1:21 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: breeze-icons
> 
> 
> Description
> ---
> 
> Otherwise we get the item listed twice with a different icon.
> 
> 
> Diffs
> -
> 
>   icons-dark/index.theme c319289 
> 
> Diff: https://git.reviewboard.kde.org/r/129106/diff/
> 
> 
> Testing
> ---
> 
> It's now shown properly both on the icons kcm and kde-gtk-config
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>



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

2016-10-05 Thread Michael Stemle

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129106/#review99801
---



+1

- Michael Stemle


On Oct. 5, 2016, 1:21 p.m., Aleix Pol Gonzalez wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129106/
> ---
> 
> (Updated Oct. 5, 2016, 1:21 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: breeze-icons
> 
> 
> Description
> ---
> 
> Otherwise we get the item listed twice with a different icon.
> 
> 
> Diffs
> -
> 
>   icons-dark/index.theme c319289 
> 
> Diff: https://git.reviewboard.kde.org/r/129106/diff/
> 
> 
> Testing
> ---
> 
> It's now shown properly both on the icons kcm and kde-gtk-config
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>



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

2016-10-05 Thread Aleix Pol Gonzalez

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129106/
---

Review request for KDE Frameworks.


Repository: breeze-icons


Description
---

Otherwise we get the item listed twice with a different icon.


Diffs
-

  icons-dark/index.theme c319289 

Diff: https://git.reviewboard.kde.org/r/129106/diff/


Testing
---

It's now shown properly both on the icons kcm and kde-gtk-config


Thanks,

Aleix Pol Gonzalez