Add icon Hackgregator/Nostalgia/Notejot/alt Transmission

[igor-dyatlov]

No description provided.

[SmartFinn]

Ох, не нравится мне идея плодить альтернативные иконки, тем более когда нет противоречий среди пользователей. Можем обойтись без них?

[igor-dyatlov]

@SmartFinn я бы оставил свой вариант, ведь иконки не конфликтуют и занимают незначительное количество КБ в репозитории.
Дело не в том, что нет противоречий у пользователей, с этим никто не спорит, просто я предложил вариант иконки такой как она выглядит в оригинале, но в стиле набора.
Большинство пользователей не меняют в ручную иконки. Тем более это изменение можно не вносить в релизнот. Тем самым о ней узнают только кто смотрит коммиты и тщательно настраивает свою систему.
В любом случае решать вам с @varlesh

[varlesh]

@igor-dyatlov Во всех значках залетел лишний код ;stop-color:#000000, удалите его. Такое бывает от грязных исходников, к примеру от гномовцев часто прилетает. Иногда приходится делетнуть кеш и конфиг инкскейпа, что бы оно не лезло опять во все значки.
Значок Notejot 64px, вертикальную линию можно сделать 2px. Понимаю, тяжко работать с 16px, но 64 пикселей полно же, зачем делать 1,5px? Горизонтальные можно оставить так, не бросаются в глаза.
Так же заметил, что вы везде делаете тень в 20%, обратите внимание на значки в репе, если бэкграунд светлый, а логотип/объект темный, то тень лучше либо не делать, либо сделать 10%, т.к. логотип/объект будет сливаться с бэкраундом и значок будет казаться размытым. Так же, если бэкраунд темный, то верхняя белая обводка будет 10% (пример - значок терминала).
Еще по поводу обводок, если бэкграунд светлый, а логотип темный, то мы не делаем обводку логотипа, т.к. она создает эффект размытия.
Теперь по-поводу трансмиссии, ну если уж вам так сильно нужен этот значок, то пусть будет. Но давайте не будем это делать традицией... Большинство альтернативных значков в нашем репе лежат из-за разногласий пользователей. И как я говорил ранее, вы первый, кто пожаловался на значок трансмиссии.
Но, нужно уменьшить размер. 44px - очень большой значок для квадратного стиля. Квадрат всегда кажется больше круга, а тут 44px вместо 40px.

[varlesh]

Еще заметил, Notejot почему-то тень карандаша сверху карандаша. И там она правильная в 10%, т.к. карандаш темней блокнота. А вот верхняя белая обводка шапки должна быть 10%, у вас она 20%.

[varlesh]

На будущее. Если вы модифицируете чужой исходник, особенно от гнома или кед. После prepare.sh, перед put-into-theme.sh откройте на всякий случай значок в текстовом редакторе. Пробежавшись глазами можно сразу увидеть лишний/ненужный код. Там часто бывают лишние фильтры, классы, градиенты, шрифты и прочая ерунда.

[igor-dyatlov]

@varlesh у трансмиссии исправлять только иконку 48 пикселей?

PS так же убрал лишний код из прошлых коммитов

[SmartFinn]

@igor-dyatlov создавайте, пожалуйста, отдельные бранчи для PR. При использовании master очень высока вероятность столкнутся с конфликтами, как сейчас. Этот солью руками когда @varlesh одобрит.

[varlesh]

Я упустил этот момент, что значок трансмиссии был обновлен. Глянул сайт лого, все вроде так же, а оказывается в репе аж 1,5 года назад обновился... Ну придется и нам обновить, но вот ломаю голову с этим рычагом. В новом стиле рычаг ассоциируется со стрелкой загрузки и с использованием градиента она не такая сильно выраженная и кажется ручкой, и стрелкой. Но бляха, как же нам это реализовать?
Если я делаю рычаг светлей треугольника, то теряется смысл стрелки/загрузки, а если делаю в одном цвете, то теряется смысл рычага/перехода. И что самое интересное, какого цвета его делать? Серый с красным вообще не реально сочетать, серый вообще хреновый цвет, я постоянно с ним мучаюсь. Остается только темный, ближе к черному.
Но это еще не все ребята, тут еще трей надо как-то обмозговать.
PR, закрываю. Делайте новый, дэлетайте альт-значки. Будем перерисовывать трансмиссию.

[varlesh]

Вот типа такого что-то будет:

[SmartFinn]

@igor-dyatlov слил, заново отправлять не нужно 😉

[igor-dyatlov]

@SmartFinn про отдельные ветки понял, спасибо

@varlesh иконка вышла – огонь. Только у кого гном transmission-symbolic ещё старую тянет.

[varlesh]

@igor-dyatlov Мы отказались от symbolic значков. Они юзаются в гноме только, нет смысла их рисовать, проще отключить.
Так же для не-гномо приложений оно все равно не работает вроде.
https://extensions.gnome.org/extension/1304/no-symbolic-icons/

[SmartFinn]

Мы отказались от symbolic значков

Нет приоритета на добавление новых в symbolic/apps точнее было б сказать. Добавлять можно, а обновлять тем более приветствуется.