Anonymous | Login | 2024-11-21 12:08 UTC |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
0002580 | VCMI | GUI - Other | public | 2016-10-23 13:49 | 2023-05-17 18:23 | ||||||||
Reporter | Dydzio | ||||||||||||
Assigned To | Ivan | ||||||||||||
Priority | normal | Severity | major | Reproducibility | always | ||||||||
Status | resolved | Resolution | fixed | ||||||||||
Platform | Microsoft Windows | OS | Windows 7 | OS Version | Home Premium x64 | ||||||||
Product Version | 0.99 | ||||||||||||
Target Version | Fixed in Version | ||||||||||||
Summary | 0002580: Fullscreen game window width is bigger than it should be | ||||||||||||
Description | When running game in fullscreen mode, mouse cursor can go far beyond right edge of the screen. When playing on 800x600 resolution, the effective resolution is 1067x600. When playing on 1024x768, resolution is around 1300x768. | ||||||||||||
Steps To Reproduce | Run game in fullscreen move and move mouse as far as possible to the right side. When moving mouse back to the left side, cursor will go back after moving mouse far left, instead of immediately. | ||||||||||||
Additional Information | I added "logGlobal->warn(std::to_string(x) + " " + std::to_string(y));" at the end of CCursorHandler::cursorMove and confirmed that resolution is bigger than desired. | ||||||||||||
Tags | No tags attached. | ||||||||||||
Attached Files | |||||||||||||
Relationships | |||||||||||
|
Notes | |
(0006859) vmarkovtsev (developer) 2016-10-23 16:22 |
This is Windows specific, no such problem on Linux and macOS. |
(0006864) SXX (administrator) 2016-10-23 21:37 |
I think problem is really related to fact that we don't have actual modal fullscreen mode. All we have is windowed fullscreen and this means that cursor might go outside of window borders into "invisible area". At least for me on Kubuntu 16.04 there actually extra free space in right and bottom part of screen so cursor can always be outside of visible area a bit. There absolutely no reason to have modal fullscreen really, but then we need mouse lock so you wouldn't be able to get cursor outside. This should also fix game behaviour on Windows. |
(0006977) Dydzio (developer) 2017-02-10 15:32 |
I think we should allow modal fullscreen as option to choose for player. It fixes the problem completely. Opposite to SXX, I think there is no reason NOT to have modal fullscreen, while many modern games offer it. Tested implementation: https://github.com/dydzio0614/vcmi/commit/f7dbd207ebe82d36c04f29188399c06c92daf891 [^] |
(0007164) SXX (administrator) 2017-07-22 14:20 |
Please let us know if this fixed yet with your recent PRs. |
(0007165) Dydzio (developer) 2017-07-22 14:40 |
Modal fullscreen workarounds the issue and makes game behave as intended. Without it the issue persists. |
(0007166) SXX (administrator) 2017-07-22 15:04 |
Thanks. |
(0007800) Dydzio (developer) 2019-05-21 15:45 |
As far as I managed to learn, SDL_WINDOW_FULLSCREEN_DESKTOP is not meant to be used together with resizing (SDL_RenderSetLogicalSize etc.) The only workaround I found for this would be changing how mouse events are handled in resized "windowed fullscreen" by calculating mouse position - I find this unacceptable. To keep windowed fullscreen working properly we should start by trying other way around - render fullscreen resolution and use graphics upscaling for that or something... |
(0008628) Ivan (developer) 2023-05-17 18:23 |
Fixed in 1.3 |
Issue History | |||
Date Modified | Username | Field | Change |
2016-10-23 13:49 | Dydzio | New Issue | |
2016-10-23 13:52 | Dydzio | Steps to Reproduce Updated | View Revisions |
2016-10-23 16:22 | vmarkovtsev | Note Added: 0006859 | |
2016-10-23 21:34 | SXX | Relationship added | related to 0002476 |
2016-10-23 21:37 | SXX | Note Added: 0006864 | |
2017-02-10 15:32 | Dydzio | Note Added: 0006977 | |
2017-07-22 14:20 | SXX | Note Added: 0007164 | |
2017-07-22 14:20 | SXX | Assigned To | => SXX |
2017-07-22 14:20 | SXX | Status | new => feedback |
2017-07-22 14:40 | Dydzio | Note Added: 0007165 | |
2017-07-22 14:40 | Dydzio | Status | feedback => assigned |
2017-07-22 15:04 | SXX | Note Added: 0007166 | |
2017-07-22 15:04 | SXX | Assigned To | SXX => |
2017-07-22 15:04 | SXX | Status | assigned => acknowledged |
2019-05-19 18:20 | Dydzio | Relationship added | related to 0003056 |
2019-05-21 15:45 | Dydzio | Note Added: 0007800 | |
2023-05-17 18:23 | Ivan | Note Added: 0008628 | |
2023-05-17 18:23 | Ivan | Status | acknowledged => resolved |
2023-05-17 18:23 | Ivan | Resolution | open => fixed |
2023-05-17 18:23 | Ivan | Assigned To | => Ivan |
Copyright © 2000 - 2024 MantisBT Team |