-
Notifications
You must be signed in to change notification settings - Fork 434
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
Resize the debug watch window whenever something gets added / removed #3251
Conversation
I was ready to merge this but, ngl, these seg faults on unit tests are worrying me, dev branch doesn't have this issue, so it's possibly a related to the change Edit: confirmed, happens on hl tests locally, too:
|
gonna merge later today, lmk if you have issues which my changes, otherwise we can fix in post |
here was the demo WITHOUT noreposition.mp4this one is WITH withreposition.mp4 |
maybe it's becuase i tested on html5, but i saw no difference. will try again, what target is that? |
Yup, my mistake, added back |
Thanks! |
woohoo :)
…On Tue, Sep 24, 2024 at 2:20 PM George Kurelic ***@***.***> wrote:
Thanks!
—
Reply to this email directly, view it on GitHub
<#3251 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFJTCU3HGJCB6J6Z5FWAMULZYGUPJAVCNFSM6AAAAABOKHKTWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZRHE4TIOJUGU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
The debugger Watch window would overflow the contents and show offscreen until the user clicks+drags the window around (which then automatically resizes it properly)
This PR makes it so that whenever something is added to the watch window, it will resize itself.
Using FNF (apologies for the .mov files!)
Before
Screen.Recording.2024-09-16.at.6.29.53.PM.mov
After
Screen.Recording.2024-09-16.at.6.32.52.PM.mov