search

ImageProcessing-ElectronicPublications / scantailor-experimental

Scan Tailor Experimental is an interactive post-processing tool for scanned pages.
https://github.com/Tulon/scantailor/tree/experimental
GNU General Public License v3.0
30 stars 0 forks source link

Margins, center_line + dark_theme? #12

Closed zvezdochiot closed 8 months ago

zvezdochiot commented 8 months ago

Hi @noobie-iv .

При использовании dark_theme в "Полях" ("Margins") начисто исчезают центральные метки: stex-margins-center_line Думаю, это связано с изменением размерных единиц. Сам код центральных меток: https://github.com/ImageProcessing-ElectronicPublications/scantailor-experimental/blob/cfcd73196e4776a6d0ef22567e1151c479fa7ca4/src/filters/page_layout/ImageView.cpp#L315-L316 https://github.com/ImageProcessing-ElectronicPublications/scantailor-experimental/blob/cfcd73196e4776a6d0ef22567e1151c479fa7ca4/src/filters/page_layout/ImageView.cpp#L320-L321 Не хотелось бы их "потерять". Очень они даже полезные.

PS: Метки исчезают не везде, а только там, где нет подложки (как в скрине). Ежели поля накладываются на "бумагу", то центральные метки как и положено "светятся" крестом.

noobie-iv commented 8 months ago

А я их вижу на обоих образцах. Просто на темном фоне они практически не заметны. Это та же проблема, из-за которой я заменил надписи со ссылками на кнопки перед добавлением темной темы. В STA вообще темы прошиты в том числе в код, и при переключении еще и дополнительные цвета меняют, кроме тех, что читаются из qss. В STU та же фигня с цветами выделения в окне миниатюр. По-хорошему, под такие цвета надо в язык QSS добавить свои типы объектов, чтобы и их можно было менять из тем, и при загрузке стиля дополнительно анализировать содержимое файла. А по-простому, наверное, можно просто отрисовать такой маркер двумя контрастными линиями 3px+1px, вместо одной. Или заморочиться, и в backgroundColorForMatchSizeMode() сделать выбор цвета под фон, а не возврат одного фисированного как сейчас.

zvezdochiot commented 8 months ago

@noobie-iv say:

А я их вижу на обоих образцах. Просто на темном фоне они практически не заметны.

Понятно. Ладно, в большинстве случаев видно. Только на некоторых не видно. Не стоит "суетиться".

noobie-iv commented 8 months ago

Я вообще только сейчас узнал, что они есть. Они же и на левом образце практически не видны. Я думал, что направляйки надо тырить из STA (а потом еще и править, потому что они глючат, если у картинок DPI разные). Собственно, не сложно поиграть в этой функции перьями и цветами, да подобрать более контрастный вариант. Но это опять день на эксперименты. Починить перевод стандартных кнопок - день гуглить и тестить. Починить размеры окна с темной темой - тоже. А моя главная идея будет простаивать.

На сегодня я в STD полностью вычистил деварп, и теперь можно пробовать пришить версию STEX. Все, что я делал до сегодняшнего дня, мозгов-то не требовало - темы, кнопки, копипаста из STU, удаления. А вот теперь, собственно, и начинается главная проверка: хватит мне соображалки, или надо уже забить на все эти игры с ST. И так несколько месяцев сюда добирался. Пришьется - будет первый релиз STD и планы на дальнейшее, нет - снесу профиль с гитхаба, и пойду сопли утирать.

zvezdochiot commented 8 months ago

@noobie-iv .

Так держать. :+1: