Skip to content
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

Dragging Dockable Panels (when floating) back and forth causes obscene panel growth #61

Open
AlbertWiersch opened this issue Dec 8, 2020 · 6 comments

Comments

@AlbertWiersch
Copy link

AlbertWiersch commented Dec 8, 2020

I undocked a TSpTBXDockablePanel and dragged it around across monitors with different DPIs and the panel size kept growing out of control. It looks like the biggest problem occurred when it grew when dragging it from a higher DPI monitor to a lower DPI monitor. Strangely, the first time it moved to a lower DPI monitor it didn't grow but them moving it back to the higher DPI monitor and then back to the lower DPI monitor caused extreme panel size growth. Just try dragging it back and forth from different DPI monitors to see the bizarre behavior.

@SilverpointDev
Copy link
Owner

Try the latest, I think it's fixed now.

@AlbertWiersch
Copy link
Author

Thanks but I'm putting my per-monitor DPI support on hold for now due to this issue:
https://quality.embarcadero.com/browse/RSP-31931

I hope it's fixed and will test again when the above issue is resolved. I really wanted to support per-monitor DPI and was almost there until I ran into the font size bug.

@pyscripter
Copy link

Thanks but I'm putting my per-monitor DPI support on hold for now due to this issue:
https://quality.embarcadero.com/browse/RSP-31931

This is not a bug. Please see my comment,

@AlbertWiersch
Copy link
Author

AlbertWiersch commented Dec 25, 2020

This is not a bug. Please see my comment,

Very interesting. I'll have to investigate this after the holidays... but what a seemingly bizarre, inconsistent, and problematic way to handle font sizing... at least to my eye.

@SilverpointDev
Copy link
Owner

@AlbertWiersch Can you try the latest commit?

@AlbertWiersch
Copy link
Author

AlbertWiersch commented Apr 10, 2021

@AlbertWiersch Can you try the latest commit?

Sorry, I just now saw this. I've updated to 2.5.8 but have been busy with other things and have not re-investigated per-monitor DPI. I'll put it on my to-do list to get to eventually. :-)

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

No branches or pull requests

3 participants