OhmNomNom / thyme

A fork of mintty, for the modern world
GNU General Public License v3.0
0 stars 0 forks source link

Work around bbLean resizing bug #239

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run mitty and try to resize window (probably not in explorer but alternative 
shell, bb4win for ex).
2. Notice tooltip didn't hidden and not changed columns/lines numbers in 
terminal.

What is the expected output? What do you see instead?
Tooltip after window resizing should be hidden and terminal should get proper 
columns/lines numbers.

What versions of mintty, Cygwin/MSYS, and Windows are you using?
mintty svn rev 1097. Last Changed Date: 2010-12-19 18:17:16 +0300 (Sun, 19 Dec 
2010)
Cygwin 1.7.7
Windows XP with bbLean 1.17.1

Please provide any additional information below.
Patch which resolve this issue for me in attachment.

Original issue reported on code.google.com by errr%voh...@gtempaccount.com on 20 Dec 2010 at 2:53

Attachments:

GoogleCodeExporter commented 9 years ago
Thank you very much for providing a solution along with the problem. This is 
clearly a bug in bbLean though: it simply neglects to send a WM_EXITSIZEMOVE 
message when the resizing ends. If this happened with the default window 
manager, I'd have got an awful lot of complaints about it. Have you reported 
the issue to the bbLean developer(s)?

However, since bbLean development seems to have stopped and since the 
workaround is small and hopefully doesn't affect anything else, I might as well 
include it.

Original comment by andy.koppe on 20 Dec 2010 at 6:05

GoogleCodeExporter commented 9 years ago
Workaround implemented in r1101. Can you give that a try?

Original comment by andy.koppe on 20 Dec 2010 at 6:26

GoogleCodeExporter commented 9 years ago
Thanks. r1101 works fine.

I'm not completely sure about various WM's, but if you use plain cmd as WM 
(like in SafeMode) the WM_EXITSIZEMOVE massage also isn't sent.

Original comment by errr%voh...@gtempaccount.com on 21 Dec 2010 at 2:06

GoogleCodeExporter commented 9 years ago

Original comment by andy.koppe on 21 Dec 2010 at 6:47