wrong panel CSS Class #343
Comments
|
В панели для полностью Light и Dark не должно быть класса class="ColorScheme-ButtonBackground. Должен юзаться class="ColorScheme-Text" |
|
Ага, вижу, могу поправить. |
|
Как мы это прошляпили? |
|
Там по хорошему надо еще и все остальные проверить |
|
@SmartFinn Там надо заменить тоько |
|
@varlesh да уж, немало |
|
там по типу этого поменять класс только для Dark в панели 22 и 24, потом тупо скопировать в Light с заменой и также поменять цвет: но ты же знаешь, я ща с этими ковычками застряну надолго )) |
|
@SmartFinn вот так вроде сработало: |
|
@varlesh да все проще чуть-чуть магии zsh. |
|
Отлично, теперь Dark |
|
@SmartFinn Молодец, быстро ты 👍 Там скрипты наверн надо еще проверить |
|
@varlesh а с ePapirus что делать? Там по сути на этом все завязано. |
|
ePapirus - это же Papirus, ток с другими цветами |
|
Papirus/ePapirus - темная панель, светлые окна. В остальных везде монохром с class="ColorScheme-Text" |
|
Проверил |
|
баг чисто случайно нашел, сижу в неоне смотрю значок обновлений светлый... не фига не пойму. Потом открыл исходник, смотрю а там класс кнопки фона стоит: |
|
@varlesh cкрипты посмотрю, может в |
|
я пойду заменю шаблоны пока |
|
фпофиксил 79c3255 |
Почему ты про ePapirus говоришь, я не пойму? ePapirus - это тот же Papirus, только с другими оттенками цветов, классы идентичны. В Papirus/ePapirus юзается только для панели class="ColorScheme-ButtonBackground", в остальном всё одинаково |
|
@varlesh отлично, проверь еще в шаблонах чтобы CSS не отличались в рамках одной темы. В ePapirus, например, actions и devices - разные.
Я это забыл просто. |
|
@varlesh panel и actions/devices прощу прощения. |
|
ок, ща проверю. Со скриптами вроде всё норм, я так бегло глянул... у тебя же по цвету все сделано |
|
В Papirus, Papirus-Dark и Papirus-Light с CSS все в порядке. Нужно только для ePapirus объединить и прописать. |
|
вроде сделал ef8e1b4 |
|
@varlesh ок, проверю еще CSS в основной теме и на единую заменю если надо. |
|
там в actions были разные css емнип |
|
или только по actions прошелся? |
|
@varlesh постой, ePapirus я уже посикфил, в Dark один паттерн забыл обработать, а Light сам сделаю. |
|
в Dark надо удалить из панели .ColorScheme-ButtonBackground { color:#d3dae3; } |
|
@SmartFinn и еще пропустил Papirus-Dark/16x16/extra/clementine |
|
@SmartFinn Как закончишь скажи, там еще надо пофиксить |
|
@varlesh вроде закончил |
|
Красавчик, я ща клементин экстра проверю |
|
@varlesh ах, одного не учли, скрипты |
|
а почему не правильно? мы же с тобой опять потеряем унификацию |
|
оно же всегда у нас портируется в строгом порядке, я не вижу ничего криминального в удалении этой строчки... |
|
В этом случае если прогнать иконку из Papirus/panel через эти скрипты она обязательно сломается. |
|
ну мы тогда унификацию ломаем. Может для дарк и лайт какие костыли придумать или методы обхода |
|
Правильно наверное какой-нибудь warning выдавать на такие случаи. Автоматизировать врядли получится. |
|
я все-таки не понимаю почему будут проблемы с panel, ведь по сути просто удаляется паттерн, который не используется. Почему оно ломать ее будет? Там же меняется и цвет и класс |
|
типа двойной проход - сначала замена класса, потом замена цвета ну третье это удаление ненужного класса |
|
костыль тольк для Dark к примеру, а Light достаточно инверсии цветов из Dark |
|
Если удалять то получим такой файл: <?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>Как видишь у пафов не будет цвета. |
|
Исходник: <?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> |
|
Ну меняется класс ButtonBackground на Text, потом меняется цвет Text в CSS, после этого делетается класс .ColorScheme-ButtonBackground { color:#d3dae3; } |
|
Может сработать, поэкспериментирую. В convert-to-e.sh только замена цвета, никаких дополнительных движений не надо? |
|
Да, эт тот же папирус, ток сдругими оттенками |
|
Papirus to Papirus Dark:
|
|
@varlesh обновил скрипты, все работает как надо. |
|
Круто, закрываю тему |
@SmartFinn у Dark и Light неправильные классы, в work кстати тоже.... Как я это прошляпил - хз!
The text was updated successfully, but these errors were encountered: