![]() |
13.3.1 CXTBrowseDialog modality bug |
Post Reply ![]() |
Author | |
znakeeye ![]() Senior Member ![]() ![]() Joined: 26 July 2006 Status: Offline Points: 1672 |
![]() ![]() ![]() ![]() ![]() Posted: 07 March 2010 at 10:59am |
XTP 13.3.1, Windows 7 64-bit, VS 2008 SP1 with Windows 7 SDK.
CXTBrowseDialog browseDlg(this);
browseDlg.DoModal();
After showing dialog the focus is not restored to this. I tried the same code with any other dialog, and the focus is indeed set to the parent window after displaying the dialog! Also, it doesn't matter if you provide a parent or not - the focus is restored correctly no matter what.
|
|
PokerMemento - http://www.pokermemento.com/
|
|
![]() |
|
kelvincannon ![]() Groupie ![]() ![]() Joined: 12 November 2003 Location: United Kingdom Status: Offline Points: 16 |
![]() ![]() ![]() ![]() ![]() |
Is there any chance of a quick fix for this bug? I have an application going out for beta shortly and this bug locks up my application.
|
|
![]() |
|
cpede ![]() Senior Member ![]() Joined: 13 August 2004 Location: Denmark Status: Offline Points: 680 |
![]() ![]() ![]() ![]() ![]() |
In Vista and 7 you should use CLSID_FileOpenDialog with the FOS_PICKFOLDERS flag instead of the
"old" browse dialog.
-cpede
|
|
Product: Xtreme ToolkitPro (24.0.0)
Platform: Windows 10 (x64) Language: Visual Studio 2017 (C++) |
|
![]() |
|
Algae ![]() Senior Member ![]() ![]() Joined: 08 January 2007 Location: United States Status: Offline Points: 217 |
![]() ![]() ![]() ![]() ![]() |
Yes, it's a bad bug. Problem is in the cut and paste material from MFC "DoModal" code. The framework should be handling all that. I'm not sure why it was added to otherwise functional code.
I reverted the XTBrowseDialog cpp/h to 13.2.1 with no ill-effects. ![]() |
|
![]() |
|
znakeeye ![]() Senior Member ![]() ![]() Joined: 26 July 2006 Status: Offline Points: 1672 |
![]() ![]() ![]() ![]() ![]() |
http://www.codejock.com/products/releasenotes/release_notes.asp
13.3.0:
Fixing source: XTBrowseDialog was modeless
I believe that's where the new code was added. I wonder what they wanted to fix...
|
|
PokerMemento - http://www.pokermemento.com/
|
|
![]() |
|
SuperMario ![]() Admin Group ![]() ![]() Joined: 14 February 2004 Status: Offline Points: 18057 |
![]() ![]() ![]() ![]() ![]() |
I believe this will be fixed in the next release. You can write to support and I'll attache the updated file.
|
|
![]() |
|
znakeeye ![]() Senior Member ![]() ![]() Joined: 26 July 2006 Status: Offline Points: 1672 |
![]() ![]() ![]() ![]() ![]() |
Could this have something to do with this?
|
|
PokerMemento - http://www.pokermemento.com/
|
|
![]() |
|
Oleg ![]() Admin Group ![]() Joined: 21 May 2003 Location: United States Status: Offline Points: 11234 |
![]() ![]() ![]() ![]() ![]() |
Hi,
Yes, it was reverted already. |
|
Oleg, Support Team
CODEJOCK SOFTWARE SOLUTIONS |
|
![]() |
Post Reply ![]() |
|
Tweet
|
Forum Jump | Forum Permissions ![]() You cannot post new topics in this forum You cannot reply to topics in this forum You cannot delete your posts in this forum You cannot edit your posts in this forum You cannot create polls in this forum You cannot vote in polls in this forum |