-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The About page is out of bounds #47
Comments
Perhaps that's the same problem as with #27 |
Ok, it seems that the changes in 856c455 introduced this bug. The text wrapper relies on the visible rect, not the out-of-window content rect. @have-fun-was-taken: |
Yeah, my reading comprehension obviously had better days :-) @veger: Perhaps you can help. |
I wrote my thoughts in #31 and #34, as well as in the commit. Unfortunately my memory on this is quite spotty as I made these changes over 2 years ago: ShadowTheAge#145 While it was definitely required for something, and I do remember the pain of diving into this GUI code (the style of closely combining the rendering GUIs with performing actions, is very alien to me), I cannot remember for what. Besides it being needed for #38. It had something to do with the widths/height of panel/areas not being fully calculated but only the visible part. (Obviously) causing issues with scrollareas as those are typically not fully rendered or there wouldn't a need for scrollbars to begin with. |
The text was updated successfully, but these errors were encountered: