Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use icons from the system icon theme where possible #22

Closed
vorot93 opened this issue Jan 25, 2013 · 12 comments
Closed

Use icons from the system icon theme where possible #22

vorot93 opened this issue Jan 25, 2013 · 12 comments
Assignees
Labels
Milestone

Comments

@vorot93
Copy link
Contributor

@vorot93 vorot93 commented Jan 25, 2013

It would be wise not to duplicate Nuvola and Oxygen icons but rather add dependency on these themes. The latter do not depend on GTK+ or kdelibs.

@ghost ghost assigned brezerk Jan 25, 2013
@LazyKent
Copy link

@LazyKent LazyKent commented Jan 25, 2013

Oxygen icons take 9,5 Mb (at least in openSUSE package). And don't know, what is Nuvola.

@brezerk
Copy link
Owner

@brezerk brezerk commented Jan 25, 2013

* x11-themes/nuvola
     Available versions:  1.0-r1^bs
     Homepage:            http://www.kde-look.org/content/show.php?content=5358
     Description:         Nuvola SVG icon theme

But I am agreed. Current icons size is just ~400K.

@LazyKent
Copy link

@LazyKent LazyKent commented Jan 25, 2013

% zypper se nuvola
Репозиторий 'Local' устарел. Можно запустить 'zypper refresh' от имени администратора для его обновления.
Загрузка данных о репозиториях...
Чтение установленных пакетов...
Не найдено пакетов.
[1]    9495 exit 104   zypper se nuvola

openSUSE doesn't ships nuvola icons. Don't ask why. :)
http://pkgs.org/search/?keyword=nuvola&search_on=smart&distro=0&arch=32-bit&exact=0

@LazyKent
Copy link

@LazyKent LazyKent commented Jan 25, 2013

Thanks, no way.
https://build.opensuse.org/package/binary?arch=x86_64&filename=kdeartwork4-icons-4.9.98-1.1.noarch.rpm&package=kdeartwork4&project=openSUSE%3AFactory&repository=standard

Just in case: Size: 11.4 MB. Are you sure I need all that crap?

Перефразируя кого-то: "когда я слышу KDE4, мне хочется схватиться за пистолет".

@vorot93
Copy link
Contributor Author

@vorot93 vorot93 commented Jan 25, 2013

А теперь посмотри на список зависимостей:
No requires

HDD отвалится установить 11,4 МБ значков? :D

@LazyKent
Copy link

@LazyKent LazyKent commented Jan 25, 2013

Ну да. А нафига мне в кучвайне кдешные иконки, если у меня вся тема с гномовскими?
Такой диссонанс плохо влияет на мою психику.

Всё вышеизложенное было моим личным мнением.

А теперь официальная позиция openSUSE:
такие зависимости неприемлемы.

@vorot93
Copy link
Contributor Author

@vorot93 vorot93 commented Jan 25, 2013

В принципе openSUSE могут для себя отключить возможность выбора тем. Да и лучше опираться на существующие пакеты чем использовать китайский код.

@LazyKent
Copy link

@LazyKent LazyKent commented Jan 25, 2013

Вообще, мне кажется, существует возможность, чтобы приложение использовало значки текущей темы.
Надо freedesktop.org покопать.

@brezerk
Copy link
Owner

@brezerk brezerk commented Jan 26, 2013

да чо там копать, все есть на поверхности: http://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html#icon_lookup

@brezerk
Copy link
Owner

@brezerk brezerk commented Jan 26, 2013

забавно что стандарт не говорит о едином файле конфигурации в котором бы указывалась текущая тема пиктограмм.

В результате народ городит няшные костыли:
http://doublecmd.svn.sourceforge.net/viewvc/doublecmd/trunk/src/platform/unix/uunixicontheme.pas

ох уж этот ваш опенсорц... :)

@brezerk brezerk modified the milestones: 1.2, 1.1, 2.0, Future Dec 26, 2015
@brezerk brezerk changed the title Outsource SVG icons, add dependency on Nuvola and Oxygen icon themes Use icons from the system icon theme where possible Jan 15, 2016
@brezerk brezerk modified the milestones: 1.3, Future Jan 15, 2016
@brezerk
Copy link
Owner

@brezerk brezerk commented Jan 17, 2016

Basic implementation: c71564a
Works for KDE4/5. Any issues with compatibility with other DE should be tracked in separate ticket.

@brezerk brezerk closed this Jan 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.