Replace default raster icons with SVG analogs #21

Closed
vorot93 opened this Issue Jan 25, 2013 · 26 comments

Projects

None yet

4 participants

@vorot93
Contributor
vorot93 commented Jan 25, 2013

Vector graphics has lesser size and higher quality. Replacing PNG with SVG would be a progress. Also it is possible to remove icons from q4wine package and add dependency on Nuvola / Oxygen icon themes.

@brezerk brezerk was assigned Jan 25, 2013
@LazyKent

Also it is possible to remove icons from q4wine package and add dependency on Nuvola / Oxygen icon themes.

I think, that is a bad idea.
As Xfce user I am not interested in additional useless files in my system.

@brezerk
Owner
brezerk commented Jan 25, 2013

I think the best solution for this will be:

  1. to add support for handling icon themes from /usr/share/icons folder.
  2. leave "embedded" icons, b\c on some systems (OS X and maybe some other distros) there are no separate package for Nuvola / Oxygen icon themes :)
@brezerk brezerk modified the milestone: 2.0, 1.1, Future Dec 26, 2015
@brezerk brezerk modified the milestone: 1.3, Future Jan 15, 2016
@brezerk brezerk changed the title from Use SVG icons to Replace default raster icons with SVG analogs Jan 15, 2016
@brezerk brezerk closed this Jan 19, 2016
@tehnick
Collaborator
tehnick commented Jan 25, 2016

New program icon looks very ugly in dark Qt themes.
Please add white border or something similar.
Ideally new svg icon should look exactly as old png icon.

@tehnick tehnick reopened this Jan 25, 2016
@brezerk
Owner
brezerk commented Jan 25, 2016

Hi @tehnick ,

it does have an light-grey border:
icon

I can make it white if you wish so.
Anyway, can you attach an screenshot, please?
Just wonder how it looks on your system :)

@tehnick
Collaborator
tehnick commented Jan 25, 2016

desktop_529
desktop_531
desktop_532
desktop_533

@tehnick
Collaborator
tehnick commented Jan 25, 2016

Debian unstable, Qt 5.5.1, Plasma desktop 5.4.3, KF5 5.16.0
q4wine from master branch vs q4wine-1.1-r2
Just find 10 differences... =)

@brezerk
Owner
brezerk commented Jan 26, 2016

Can you try it now, please?

@tehnick
Collaborator
tehnick commented Jan 26, 2016

desktop_534
desktop_535

@tehnick
Collaborator
tehnick commented Jan 26, 2016

As you can see the difference in quite small. I propose to use strictly white and opaque border. Inkscape shows me that now it is transparent for 80%.

@tehnick
Collaborator
tehnick commented Jan 26, 2016

Also I have to note that almost always tiny and small raster icons looks much better than scaled vector icons. For example, previous picture of heart in png variant is much nicer than in scaled vector one.

@tehnick
Collaborator
tehnick commented Jan 26, 2016

I propose to use strictly white and opaque border.

If you agree I can fix it myself.

@brezerk
Owner
brezerk commented Jan 26, 2016

Hi @tehnick ic. try this one: f674374

If you agree I can fix it myself.

Feel free to fix or play around whatever you wish to.

@tehnick
Collaborator
tehnick commented Jan 26, 2016

desktop_536
desktop_537

@tehnick
Collaborator
tehnick commented Jan 26, 2016

Теперь гораздо лучше и баг можно считать исправленным.

По моему мнению, все градиенты из иконки лучше убрать, сделав ее плоской и контрастной. Толщину ножки бокала в самом узком месте желательно увеличить, а верхние углы скруглить. Ну и толщину белой границы можно увеличить. =) Но это уже дело вкуса и к данной задаче не относится.

@tehnick
Collaborator
tehnick commented Jan 26, 2016

Для иллюстрации своей идеи добавил пример плоской иконки: 7363ba6

Я не дизайнер и с векторными изображениями имею дело редко, поэтому просьба сильно не придираться... =)

@tehnick
Collaborator
tehnick commented Jan 26, 2016

desktop_539

@brezerk
Owner
brezerk commented Jan 27, 2016

Та яж тоже не дизайнер, лол :)
да, без градиента лучше смотрится на мелких размерах, q4wine-flat очень ок.

@LazyKent

А не забыли ли вы, джентльмены, про монохромные иконки?
Как бы не хотелось об этом говорить (и видеть), но возникают такие требования: https://developer.gnome.org/hig/stable/icons-and-artwork.html.ru
Да и KDE, похоже, к этому склоняется.
Конкретно, в системном лотке стало модно показывать монохромный значок.

@tehnick
Collaborator
tehnick commented Jan 27, 2016

Есть такая мода, да. Но лично я с ней не согласен.

А что касается иконок на "кнопках", то в сборках программы из мастер-ветки уже используются системные иконки. Как минимум в KDE это работает. Что видно по скриншотам выше.

@brezerk
Owner
brezerk commented Jan 27, 2016

@LazyKent в комплекте идет и устанавливается q4wine-ambiance.svg он монохромный.
наверно стоит добавить опцию в настройках, которая позволит пользователю переключать значок программы на желаемый.

ps: и да, интересно, как подхват системных иконок работает в других ДЕ (:

pps: @tehnick я поправил то, что ты просил в жаббире.

@LazyKent

@brezerk
Использую Xfce. Протестирую, если скажешь, что нужно попробовать и как.

@tehnick
Collaborator
tehnick commented Jan 27, 2016

@brezerk
Если возражений нет, предлагаю иконку с градиентами вообще убрать и поставить вместо нее плоскую.

Что касается монохромной иконки в системном трее, то опция была бы хорошим вариантом.

И еще стало интересно, есть ли какой-нибудь более-менее универсальный (не зависящий от DE) метод автоматического определения того, что пользователь выбрал использование монохромных иконок в настройках своего DE.

@brezerk
Owner
brezerk commented Jan 28, 2016

@tehnick да, убирай. опцию запилю на недельке.

@LazyKent просто запустить q4wine и посмотреть отличается ли иконки в программе от системных. скорее всего, потребуется какой-то пакет совместимости gtk<->qt4 или gtk<->qt5 на подобии kde-plasma/kde-gtk-config (см: https://wiki.archlinux.org/index.php/Uniform_look_for_Qt_and_GTK_applications)

@tehnick увы, стандарт (http://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html) не описывает, как и где должны хранится сами настройки текущей темы, каждая ДЕ пилит свой ни с чем не совместимый велосипед. В итоге народ пилит костыли для каждого способа хранения, что-то типа этого: https://sourceforge.net/p/doublecmd/code/HEAD/tree/trunk/src/platform/unix/uunixicontheme.pas

В qt есть стандартный метод: http://doc.qt.io/qt-5/qicon.html#fromTheme но оно берет настройки из самого qt, т.е. без прослойки совместимости, сиравно никак :)

@brezerk
Owner
brezerk commented Jan 30, 2016

добaвил опцию. вроде даже работает.

@brezerk brezerk closed this Feb 14, 2016
@tehnick
Collaborator
tehnick commented Feb 17, 2016

1.2-r2 vs 1.2-r2-54-g5e03363
desktop_552
desktop_540

@tehnick tehnick reopened this Feb 17, 2016
@brezerk
Owner
brezerk commented Feb 18, 2016

Thanks, should be fixed with: d8fba61

Reopen if needed.

@brezerk brezerk closed this Feb 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment