kswaldemar / rewind-viewer

Fast match viewer with rewinding support for Russian AI Cup championship series
MIT License
75 stars 23 forks source link

Некорректная работа масштабирования #25

Closed deniskolsanov closed 3 years ago

deniskolsanov commented 6 years ago

Если убрать фокус с viewer'а, а затем поскролить не фокусируясь на окне, то координаты уходят в бесконечность http://recordit.co/jA7DT6GIKq

Karloid commented 6 years ago

Да это пока самый раздражающий фактор для меня

kswaldemar commented 6 years ago

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

Karloid commented 6 years ago

Еще не тестил, но главный прикол в том что в мак оси можно посылать эвенты скролла когда окно не в фокусе, надо координаты курсора проверять на адекватность или что в этом духе

kswaldemar commented 6 years ago

Проверьте на mac os новую версию. Сейчас неактивное окно не должно реагировать на скролл и потреблять процессорное время.

Karloid commented 6 years ago

Проверил на macOS вроде все так и есть, отрисовка останавливается когда не в фокусе. побочный эффект небольшой - если скроллить окно когда оно не в фокусе, а затем зафокусировать его произойдет мгновенная смена масштаба. Но это фигня

deniskolsanov commented 6 years ago

Работает, но простой sleep(100) похож на костыль, когда я переключаюсь между окнами, я же не нажимаю на паузу, а она ставится...

И еще: constexpr static size_t FACILITY_LAYER = 0; стоит поставить больше нуля, иначе нельзя нарисовать примитивы не заслонив постройки.

kswaldemar commented 6 years ago

Отключение отрисовки неактивного окна сделано для того, чтобы впустую не нагружать процессор. Подразумевается, что если визуализатором пользуются, то окно активно.

Поменять слой для построек можно, но действительно ли надо что-то рисовать под постройками?

Karloid commented 6 years ago

Мне норм как сейчас работает. И в плане не активного окна. И в плане отрисовки построек

deniskolsanov commented 6 years ago

Я пп рисую. screenshot_114 Сейчас подумал... а ведь действительно над зданиями можно и не рисовать квадрат просто, или альфу в 0.5 установить

Karloid commented 6 years ago

Рекомендую все квадраты с альфой рисовать, что бы видеть погоду и почву

kswaldemar commented 3 years ago

Закрываю, проблема не повторяется. Если будут проблемы, переоткроем