Papirus-icon-theme: wrong panel CSS Class

[varlesh]

@SmartFinn у Dark и Light неправильные классы, в work кстати тоже.... Как я это прошляпил - хз!

neon@neon:~/papirus-icon-theme/Papirus-Dark/22x22/panel$ grep -rl 'class="ColorScheme-ButtonBackground"'
whatsapp-tray.svg
wechat-tray.svg
wechat-tray-unread.svg
weather-storm.svg
weather-snow.svg
weather-snow-scattered.svg
weather-snow-rain.svg
weather-showers.svg
weather-showers-scattered.svg
weather-severe-alert.svg
weather-overcast.svg
weather-none-available.svg
weather-hail.svg
weather-freezing-rain.svg
weather-fog.svg
weather-few-clouds.svg
weather-few-clouds-night.svg
weather-clouds.svg
weather-clouds-night.svg
weather-clear.svg
weather-clear-night.svg
vivaldi-tray.svg
systray_reconnecting.svg
systray_quarantined.svg
systray_notconnected.svg
systray_disconnecting.svg
systray-connected.svg
steam_tray_mono.svg
simplescreenrecorder-recording.svg
simplescreenrecorder-paused.svg
simplescreenrecorder-panel.svg
pushbullet-indicator-paused.svg
pushbullet-indicator-disabled.svg
network-wireless-signal-ok.svg
network-wireless-signal-none.svg
network-wireless-signal-low.svg
network-wireless-signal-good.svg
network-wireless-signal-excellent.svg
network-wireless-secure-signal-ok.svg
network-wireless-secure-signal-none.svg
network-wireless-secure-signal-low.svg
network-wireless-secure-signal-good.svg
network-wireless-secure-signal-excellent.svg
network-wireless-offline.svg
network-wireless-no-route.svg
network-wireless-hotspot.svg
network-wireless-connected.svg
network-wireless-acquiring.svg
lyricfier-tray.svg
livewallpaper-indicator.svg
kazam-stopped.svg
kazam-countdown.svg
indicator-keyboard-Zh.svg
indicator-keyboard-Yo.svg
indicator-keyboard-Xs.svg
indicator-keyboard-Wo.svg
indicator-keyboard-Vi.svg
indicator-keyboard-Uz.svg
indicator-keyboard-Ur.svg
indicator-keyboard-Uk.svg
indicator-keyboard-Tr.svg
indicator-keyboard-Tn.svg
indicator-keyboard-Tk.svg
indicator-keyboard-Th.svg
indicator-keyboard-Tg.svg
indicator-keyboard-Te.svg
indicator-keyboard-Ta.svg
indicator-keyboard-Sy.svg
indicator-keyboard-Sw.svg
indicator-keyboard-Sv.svg
indicator-keyboard-Sr.svg
indicator-keyboard-Sq.svg
indicator-keyboard-Sl.svg
indicator-keyboard-Sk.svg
indicator-keyboard-Si.svg
indicator-keyboard-Sd.svg
indicator-keyboard-Sa.svg
indicator-keyboard-Ru.svg
indicator-keyboard-Ro.svg
indicator-keyboard-Pt.svg
indicator-keyboard-Ps.svg
indicator-keyboard-Pl.svg
indicator-keyboard-Ph.svg
indicator-keyboard-Pa.svg
indicator-keyboard-Or.svg
indicator-keyboard-No.svg
indicator-keyboard-Nl.svg
indicator-keyboard-Ne.svg
indicator-keyboard-My.svg
indicator-keyboard-Mt.svg
indicator-keyboard-Mr.svg
indicator-keyboard-Mn.svg
indicator-keyboard-Ml.svg
indicator-keyboard-Mk.svg
indicator-keyboard-Mi.svg
indicator-keyboard-Md.svg
indicator-keyboard-Lv.svg
indicator-keyboard-Lt.svg
indicator-keyboard-Lo.svg
indicator-keyboard-Ku.svg
indicator-keyboard-Ko.svg
indicator-keyboard-Kn.svg
indicator-keyboard-Km.svg
indicator-keyboard-Kk.svg
indicator-keyboard-Ki.svg
indicator-keyboard-Ka.svg
indicator-keyboard-Ja.svg
indicator-keyboard-It.svg
indicator-keyboard-Is.svg
indicator-keyboard-In.svg
indicator-keyboard-Ik.svg
indicator-keyboard-Ig.svg
indicator-keyboard-Ie.svg
indicator-keyboard-Hy.svg
indicator-keyboard-Hu.svg
indicator-keyboard-Hr.svg
indicator-keyboard-Hi.svg
indicator-keyboard-He.svg
indicator-keyboard-Ha.svg
indicator-keyboard-Gu.svg
indicator-keyboard-Gr.svg
indicator-keyboard-Ga.svg
indicator-keyboard-Fr.svg
indicator-keyboard-Fo.svg
indicator-keyboard-Fi.svg
indicator-keyboard-Ff.svg
indicator-keyboard-Fa.svg
indicator-keyboard-Et.svg
indicator-keyboard-Es.svg
indicator-keyboard-Eo.svg
indicator-keyboard-En.svg
indicator-keyboard-Ee.svg
indicator-keyboard-Dz.svg
indicator-keyboard-Dv.svg
indicator-keyboard-De.svg
indicator-keyboard-Da.svg
indicator-keyboard-Cs.svg
indicator-keyboard-Cr.svg
indicator-keyboard-Cm.svg
indicator-keyboard-Ch.svg
indicator-keyboard-By.svg
indicator-keyboard-Bs.svg
indicator-keyboard-Br.svg
indicator-keyboard-Bn.svg
indicator-keyboard-Bm.svg
indicator-keyboard-Bg.svg
indicator-keyboard-Be.svg
indicator-keyboard-Az.svg
indicator-keyboard-Av.svg
indicator-keyboard-Ar.svg
indicator-keyboard-Am.svg
indicator-keyboard-Ak.svg
harmony-tray.svg
hangouts-working.svg
hangouts-offline.svg
hangouts-notification.svg
hangouts-available.svg
gpmdp-tray.svg
gpmdp-tray-play.svg
gpmdp-tray-pause.svg
caffeine-cup-full.svg
caffeine-cup-empty.svg
btsync-gui-paused.svg
bitcoin-indicator.svg
neon@neon:~/papirus-icon-theme/Papirus-Dark/22x22/panel$ 

[varlesh]

В панели для полностью Light и Dark не должно быть класса class="ColorScheme-ButtonBackground. Должен юзаться class="ColorScheme-Text"

[SmartFinn]

Ага, вижу, могу поправить.

[varlesh]

Как мы это прошляпили?

[varlesh]

Там по хорошему надо еще и все остальные проверить

[varlesh]

@SmartFinn Там надо заменить тоько class="ColorScheme-ButtonBackground" на class="ColorScheme-Text"

[SmartFinn]

@varlesh да уж, немало

$ > rg -l 'class="ColorScheme-ButtonBackground"' | wc -l 
1327
$ > 

[varlesh]

там по типу этого поменять класс только для Dark в панели 22 и 24, потом тупо скопировать в Light с заменой и также поменять цвет:

grep -rl class="ColorScheme-ButtonBackground" | xargs sed -i -e 's/class="ColorScheme-ButtonBackground"/class="ColorScheme-Text"/g'

но ты же знаешь, я ща с этими ковычками застряну надолго ))

[varlesh]

@SmartFinn вот так вроде сработало:

grep -rl ButtonBackground | xargs sed -i -e 's/class="ColorScheme-ButtonBackground"/class="ColorScheme-Text"/g'

[SmartFinn]

@varlesh да все проще

sed -i '/class="ColorScheme-ButtonBackground"/ s/ColorScheme-ButtonBackground/ColorScheme-Text/g' **/*.svg(.)

чуть-чуть магии zsh.

[varlesh]

Отлично, теперь Dark

[varlesh]

@SmartFinn Молодец, быстро ты :+1: Там скрипты наверн надо еще проверить

[SmartFinn]

@varlesh а с ePapirus что делать? Там по сути на этом все завязано.

[varlesh]

ePapirus - это же Papirus, ток с другими цветами

[varlesh]

Papirus/ePapirus - темная панель, светлые окна. В остальных везде монохром с class="ColorScheme-Text"

[varlesh]

Проверил grep -rl 'class="ColorScheme-Text"' в Papirus и ePapirus - всё чисто

[varlesh]

баг чисто случайно нашел, сижу в неоне смотрю значок обновлений светлый... не фига не пойму. Потом открыл исходник, смотрю а там класс кнопки фона стоит:

[SmartFinn]

@varlesh cкрипты посмотрю, может в _fix_color_scheme.sh отдельные правила для Papirus-Light и ePapirus добавлю если придется.

[varlesh]

я пойду заменю шаблоны пока

[varlesh]

фпофиксил 79c3255fbf7abab90cfbcb16293bd95bbe4afdf7

[varlesh]

@varlesh cкрипты посмотрю, может в _fix_color_scheme.sh отдельные правила для Papirus-Light и ePapirus добавлю если придется.

Почему ты про ePapirus говоришь, я не пойму? ePapirus - это тот же Papirus, только с другими оттенками цветов, классы идентичны. В Papirus/ePapirus юзается только для панели class="ColorScheme-ButtonBackground", в остальном всё одинаково

[SmartFinn]

@varlesh отлично, проверь еще в шаблонах чтобы CSS не отличались в рамках одной темы. В ePapirus, например, actions и devices - разные.

В Papirus/ePapirus юзается только для панели class="ColorScheme-ButtonBackground", в остальном всё одинаково

Я это забыл просто.

[SmartFinn]

@varlesh panel и actions/devices прощу прощения.

[varlesh]

ок, ща проверю. Со скриптами вроде всё норм, я так бегло глянул... у тебя же по цвету все сделано

[SmartFinn]

В Papirus, Papirus-Dark и Papirus-Light с CSS все в порядке. Нужно только для ePapirus объединить и прописать.

[varlesh]

вроде сделал ef8e1b4451d02bf31c45ae22f860a88948d3ccbb

[SmartFinn]

@varlesh ок, проверю еще CSS в основной теме и на единую заменю если надо.

[varlesh]

там в actions были разные css емнип

[varlesh]

@SmartFinn Серёж я думаю надо стандартизировать ColorScheme-ButtonBackground для Dark и Light в panel. Они хоть и не юзаются, но цвет там не правильный

[varlesh]

я в шаблоне поменяю кароче на родной цвет из actions

[SmartFinn]

Стоит наверное унифицировать набор CSS свойств, в ePapirus/actions, например, 2613 иконок только с .ColorScheme-Text { color:#6e6e6e; }.

[varlesh]

@SmartFinn Сделал всё в едином стиле. Для Light и Dark в panel меняется только класс...
По хорошему это надо бы в главной теме все пофиксить.
Можно ли как-то безболезненно всё это дело провернуть?

[varlesh]

Стоит наверное унифицировать набор CSS свойств, в ePapirus/actions, например, 2613 иконок только с .ColorScheme-Text { color:#6e6e6e; }.

Там не только в ePapirus, надо все монохромные CSS унифицировать

[varlesh]

@SmartFinn Скажи реально ли это сделать, долго и стоит ли?

[SmartFinn]

@varlesh конечно, для ePapirus уже заканчиваю.

CSS в ddc7de6e5c9f05926fe3e7e5e21788a7421a0205 ломает скрипт _fix_color_scheme.sh, предполагалось что цвет #5c616c будет только в Papirus.

[varlesh]

я хотел же унифицировать, что б не путались.... надо подумать как проще сделать

[SmartFinn]

@varlesh можно оставить если придумаешь как отличить иконку Papirus от Dark, Dark от Light и т.д.

[varlesh]

@SmartFinn Слушай а зачем мы прописываем 3 переменных? По сути у нас юзается только 2 - это основной цвет и хайлайт. Может тупо делетнем 3-ю переменную?

[SmartFinn]

@varlesh в Papirus предлагаешь ColorScheme-Text удалить?

[varlesh]

не только в папирус а везде где монохром - оставить только 2 переменных

[varlesh]

Вот к примеру из Papirus actions:

<style type="text/css" id="current-color-scheme">
   .ColorScheme-Text { color:#5c616c; } .ColorScheme-Highlight { color:#5294e2; } .ColorScheme-ButtonBackground { color:#d3dae3; }
  </style>

Здесь ColorScheme-ButtonBackground вообще не юзается

[varlesh]

а в панели наоборот - удалить ColorScheme-Text (если речь про Papirus)

[SmartFinn]

Не получится унификации, ColorScheme-Text используется в actions но неиспользуется в panel. Это все усложняет.

[varlesh]

Наоборот - это всё упрощает, потому что мы выкинем ненужные цвета.

[varlesh]

Другой вопрос - как это всё поменять в темах, может ипользовать шаблон для замены... я хз

[SmartFinn]

Заменить CSS во всей теме будет проблематично, и вместо 4 CSS таблиц получим минимум 8.

[varlesh]

да ты прав, возни больше.

[varlesh]

@SmartFinn В текущем состоянии ломается Papirus-Dark?

[varlesh]

из-за .ColorScheme-ButtonBackground { color:#5c616c; } в panel?

[varlesh]

Papirus Light тоже наверн будет ломаться, так как идентичен он

[varlesh]

блин как же унифицировать-то...

[SmartFinn]

@varlesh если заменю CSS на тот который в шаблоне — да. Если для Dark найти уникальный паттерн то можно просто скрипт поменять.

[SmartFinn]

Можно правда матчить прям по всей CSS таблице, но выглядеть будет уродливо.

[varlesh]

тут походу только 2 варианта - это либо удалить вообще переменную, либо прописать како-нить рандомный цвет

[SmartFinn]

Оба варианта неплохие :)

[varlesh]

Так что решим? Тут походу для дарк и лайт придется отдельные таблицы делать... я хз как еще. Надо подумать.
Вообще по хорошему как бы либо один CSS на тему, либо удалять лишние значения цветов которые не юзаются

[varlesh]

Смотри,
Papirus и ePapirus - 3 значения цветов
Papirus Light и Papirus Dark - 2 значения цветов
По сути ePapirus - это Papirus, а потому его можно вообще отбросить (тупо седом цвет поменяем и всё) Тем самым у нас 3 разных CSS. Будет ли конфликт в этом случае?

[varlesh]

т.е если вот такие CSS
Papirus:

<style type="text/css" id="current-color-scheme">
   .ColorScheme-Text { color:#5c616c; } .ColorScheme-Highlight { color:#5294e2; } .ColorScheme-ButtonBackground { color:#d3dae3; }
  </style>

Papirus Dark:

<style type="text/css" id="current-color-scheme">
   .ColorScheme-Text { color:#d3dae3; } .ColorScheme-Highlight { color:#5294e2; }
  </style>

Papirus Light - стой, это тот же Dark, только с реверсом цветов ))

[varlesh]

И по сути у нас только 2 CSS, в остальном у нас только вариация отттенка или реверс цветов

[SmartFinn]

@varlesh пойдет, придумаю как в скрипте выкрутится. Поправь в шаблонах, а я по ним в основных темах поправлю.

[varlesh]

@SmartFinn смотри нужно пофиксить только Papirus и Papirus Dark, остальное всё тупо седом меняем и всё. ща шаблоны поправлю

[varlesh]

всё, удалил .ColorScheme-ButtonBackground { color:#5c616c; } из Dark и Light. Тепрь надо основные темы унифицировать

[varlesh]

@SmartFinn Стой, там косяки есть, ща поправлю

[SmartFinn]

@varlesh в Papirus/actions/[email protected] неполная таблица, ты про этот?

[varlesh]

да, пофиксил

[varlesh]

@SmartFinn ты всё? я могу портировать Light и ePapirus

[varlesh]

или только по actions прошелся?

[SmartFinn]

@varlesh постой, ePapirus я уже посикфил, в Dark один паттерн забыл обработать, а Light сам сделаю.

[varlesh]

в Dark надо удалить из панели .ColorScheme-ButtonBackground { color:#d3dae3; }

[varlesh]

@SmartFinn и еще пропустил Papirus-Dark/16x16/extra/clementine

[varlesh]

@SmartFinn Как закончишь скажи, там еще надо пофиксить
16x16/extra/clementine/as_disabled.svg
16x16/extra/clementine/as_light.svg
16x16/extra/clementine/as.svg

[SmartFinn]

@varlesh вроде закончил

[varlesh]

Красавчик, я ща клементин экстра проверю

[SmartFinn]

@varlesh ах, одного не учли, скрипты convert-to-dark.sh и convert-to-light.sh не будут удалять класс .ColorScheme-ButtonBackground, и не думаю что будет правильно их этому учить.

[varlesh]

а почему не правильно? мы же с тобой опять потеряем унификацию

[varlesh]

оно же всегда у нас портируется в строгом порядке, я не вижу ничего криминального в удалении этой строчки...

[SmartFinn]

В этом случае если прогнать иконку из Papirus/panel через эти скрипты она обязательно сломается.

[varlesh]

ну мы тогда унификацию ломаем. Может для дарк и лайт какие костыли придумать или методы обхода

[SmartFinn]

Правильно наверное какой-нибудь warning выдавать на такие случаи. Автоматизировать врядли получится.

[varlesh]

я все-таки не понимаю почему будут проблемы с panel, ведь по сути просто удаляется паттерн, который не используется. Почему оно ломать ее будет? Там же меняется и цвет и класс

[varlesh]

типа двойной проход - сначала замена класса, потом замена цвета ну третье это удаление ненужного класса

[varlesh]

костыль тольк для Dark к примеру, а Light достаточно инверсии цветов из Dark

[SmartFinn]

Если удалять то получим такой файл:

<?xml version="1.0"?>
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="22" version="1.1" height="22" id="svg2">
 <defs id="defs12">
  <style type="text/css" id="current-color-scheme">
   .ColorScheme-Text { color:#5c616c; } .ColorScheme-Highlight { color:#5294e2; }
  </style>
 </defs>
 <g transform="matrix(1.0909 0 0 1.0909 -1.0907 -1123)" id="g4">
  <path style="fill:currentColor" d="m7 ... z" fill-rule="evenodd" transform="translate(0,1028.4)" id="path6"/>
  <path style="fill:currentColor" d="m4 .. z" transform="translate(0,1028.4)" id="path8"/>
 </g>
</svg>

Как видишь у пафов не будет цвета.

[SmartFinn]

Исходник:

<?xml version="1.0"?>
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="22" version="1.1" height="22" id="svg2">
 <defs id="defs12">
  <style type="text/css" id="current-color-scheme">
   .ColorScheme-Text { color:#5c616c; } .ColorScheme-Highlight { color:#5294e2; } .ColorScheme-ButtonBackground { color:#d3dae3; }
  </style>
 </defs>
 <g transform="matrix(1.0909 0 0 1.0909 -1.0907 -1123)" id="g4">
  <path style="fill:currentColor" d="m7 .. z" fill-rule="evenodd" transform="translate(0,1028.4)" id="path6" class="ColorScheme-ButtonBackground"/>
  <path style="fill:currentColor" d="m4 .. z" transform="translate(0,1028.4)" id="path8" class="ColorScheme-ButtonBackground"/>
 </g>
</svg>

[varlesh]

Ну меняется класс ButtonBackground на Text, потом меняется цвет Text в CSS, после этого делетается класс .ColorScheme-ButtonBackground { color:#d3dae3; }

[SmartFinn]

Может сработать, поэкспериментирую. В convert-to-e.sh только замена цвета, никаких дополнительных движений не надо?

[varlesh]

Да, эт тот же папирус, ток сдругими оттенками

[varlesh]

Papirus to Papirus Dark:

• actions, devices, places - меняется только цвет
  
  
  
  • panel - меняется класс и цвет
  
  
  • удаляется лишний паттерн .ColorScheme-ButtonBackground { color:#d3dae3; }
  
  

[SmartFinn]

@varlesh обновил скрипты, все работает как надо.

[varlesh]

Круто, закрываю тему