Skip to content

Conversation

Ygrik2003
Copy link
Contributor

No description provided.

@Ygrik2003
Copy link
Contributor Author

image @MihailRis подскажи плиз как/где реализованы менюшки?

@MihailRis
Copy link
Owner

MihailRis commented Oct 7, 2025

@MihailRis подскажи плиз как/где реализованы менюшки?

Конкретно этот момент сделан через использование glScissors, которое, по идее, можно без проблем перенести.

А в целом - graphics/ui

@Ygrik2003
Copy link
Contributor Author

Точно, я ж там заглушки оставил

@Ygrik2003 Ygrik2003 requested a review from MihailRis October 11, 2025 07:09
@MihailRis
Copy link
Owner

Повышение минимальной версии cmake уже ранее рассматривалось, так что перед продолжением перепроверю некоторые моменты

@MihailRis
Copy link
Owner

MihailRis commented Oct 11, 2025

Из элементов, требующих ввод текста с клавиатуры только TextBox?

Зависит от событий используемых в ядре и доступных в скриптинге. Если уже есть поддержка этого события в загрузчике разметки ui для всех элементов, то должна сохраняться совместимость. Если нет, то пока не требуется

@Ygrik2003
Copy link
Contributor Author

Из элементов, требующих ввод текста с клавиатуры только TextBox?

Зависит от событий используемых в ядре и доступных в скриптинге. Если уже есть поддержка этого события в загрузчике разметки ui для всех элементов, то должна сохраняться совместимость. Если нет, то пока не требуется

я переопределил методы onFocus и defocus для TextBox, для вызова методов включения/выключения TextInput'а
как я понял, в старой реализации он всегда был включен, что в теории могло влиять на перфу (и скорее всего не слабо)

Если будут добавляться в дальнейшем еще компоненты, требующие ввод текста, нужно будет вручную включать/выключать для них эту возможность

@Ygrik2003
Copy link
Contributor Author

Повышение минимальной версии cmake уже ранее рассматривалось, так что перед продолжением перепроверю некоторые моменты

я юзаю cmake 4.1.2, поэтому пришлось чутка поднять минимальную, иначе ругается
у убунты 22 симейк 3.25 поддерживается, на сколько помню, думаю, это не проблема

@MihailRis
Copy link
Owner

Повышение минимальной версии cmake уже ранее рассматривалось, так что перед продолжением перепроверю некоторые моменты

я юзаю cmake 4.1.2, поэтому пришлось чутка поднять минимальную, иначе ругается у убунты 22 симейк 3.25 поддерживается, на сколько помню, думаю, это не проблема

у меня на ноуте вроде ubuntu 21 и там в репозиториях cmake более старый

@Ygrik2003
Copy link
Contributor Author

я ни к чему не призываю, но...
https://pypi.org/project/cmake/

@MihailRis
Copy link
Owner

я ни к чему не призываю, но... https://pypi.org/project/cmake/

обновить cmake не сложно, я про то, что та версия ещё достаточно распространена

@MihailRis
Copy link
Owner

будет проще, если будет названа причина повышения версии

@Ygrik2003
Copy link
Contributor Author

На cmake 4.1.2 такой варнинг
ну справедливости ради можно было указать 3.10

[cmake] CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
[cmake]   Compatibility with CMake < 3.10 will be removed from a future version of
[cmake]   CMake.
[cmake] 
[cmake]   Update the VERSION argument <min> value.  Or, use the <min>...<max> syntax
[cmake]   to tell CMake that the project requires at least <min> but has been updated
[cmake]   to work with policies introduced by <max> or earlier.
[cmake] 
[cmake] 
[cmake] cmake version 4.1.2
[cmake] 
[cmake] CMake suite maintained and supported by Kitware (kitware.com/cmake).
[cmake] CMake Warning (dev) at /usr/share/cmake/Modules/FindOpenGL.cmake:415 (message):
[cmake]   Policy CMP0072 is not set: FindOpenGL prefers GLVND by default when
[cmake]   available.  Run "cmake --help-policy CMP0072" for policy details.  Use the
[cmake]   cmake_policy command to set the policy and suppress this warning.
[cmake] 
[cmake]   FindOpenGL found both a legacy GL library:
[cmake] 
[cmake]     OPENGL_gl_LIBRARY: /usr/lib/libGL.so
[cmake] 
[cmake]   and GLVND libraries for OpenGL and GLX:
[cmake] 
[cmake]     OPENGL_opengl_LIBRARY: /usr/lib/libOpenGL.so
[cmake]     OPENGL_glx_LIBRARY: /usr/lib/libGLX.so
[cmake] 
[cmake]   OpenGL_GL_PREFERENCE has not been set to "GLVND" or "LEGACY", so for
[cmake]   compatibility with CMake 3.10 and below the legacy GL library will be used.
[cmake] Call Stack (most recent call first):
[cmake]   src/CMakeLists.txt:10 (find_package)
[cmake] This warning is for project developers.  Use -Wno-dev to suppress it.

@Ygrik2003
Copy link
Contributor Author

В целом, если вопрос в версии симейка, ее можно откатить, это не проблема
Тут лишь вопрос удобства разработки на новых версиях софта

@MihailRis
Copy link
Owner

Если к теме этого pr оно не относится, то это изменение в нём лишнее и должно рассматриваться отдельно

@MihailRis MihailRis changed the base branch from main to dev October 14, 2025 20:19
@MihailRis
Copy link
Owner

MihailRis commented Oct 14, 2025

Перевёл на dev ветку

udp. у меня на Ubuntu 22,04 не срабатывает захват курсора

Обновил flake.lock поскольку в прежней версии были каналы в которых еще небыло sdl3, и собрать проект было не возможно.
@Ygrik2003 Ygrik2003 marked this pull request as draft October 15, 2025 10:42
@Ygrik2003
Copy link
Contributor Author

@MihailRis проверь пожалуйста, захватывает ли теперь курсор
и за одно, подскажи какая версия sdl3 у тебя, мб еще в этом проблема будет

@MihailRis
Copy link
Owner

@MihailRis проверь пожалуйста, захватывает ли теперь курсор и за одно, подскажи какая версия sdl3 у тебя, мб еще в этом проблема будет

  1. пока без изменений
  2. последнюю версию sdl3 из исходников собирал на прошлой неделе

@Ygrik2003
Copy link
Contributor Author

Похоже проблема в сборке SDL
Скорее всего мне тоже нужно поправить CI/CD
libsdl-org/SDL#13925
При сборке SDL должен быть включен параметр SDL_X11_XINPUT, а для этого нужны зависимости
sudo apt-get install libxi-dev

@Ygrik2003 Ygrik2003 marked this pull request as ready for review October 17, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants