User forums > Help
Build target window not resizing correctly.
oBFusCATed:
Hm, you're using smaller font and all targets are visible. This is not the case for me.
Can someone share a test project?
BTW: Your fix is inappropriate.
Jenna:
--- Quote from: oBFusCATed on July 01, 2014, 01:40:34 am ---Hm, you're using smaller font and all targets are visible. This is not the case for me.
Can someone share a test project?
BTW: Your fix is inappropriate.
--- End quote ---
I attach a sample project-file with 52 targets.
It has a scrollbar on linux, but it shows a similar issue:the list-control is much too large.
I did not try to do any further tests.
I guess the problem is, that we do a dynamic layout after the list-control is populated.
If there are very long target names, it's the same, but in horizontal direction.
oBFusCATed:
Reproduced. I've even modified it to include a long target name.
By dynamic layout you're probably talking about this snippet:
--- Code: --- // make sure everything is laid out properly
// before adding panels from plugins
// we don't want the dialog to become huge ;)
// note that a similar situation in editor settings had been solved by commenting
// the following line out...
GetSizer()->SetSizeHints(this);
--- End code ---
I've commented it and it fixed only the first problem. It didn't fix the very-long-named-target problem.
I've taken a look here http://docs.wxwidgets.org/trunk/classwx_list_box.html and it seems there are problems with the ListBox control.
Have we tried to use a wxListCtrl instead?
oBFusCATed:
I've just tried the wxListCtrl and it behaves pretty good.
Please test if the layout is correct with this test patch on windows: http://cmpt.benbmp.org/codeblocks/patches/project_options_layout_many_target.patch
NOTE: This patch is not meant for production use, it is meant to test the layout only. The dialog is totally broken, because I've not fixed all usages of the wxListBox!
stahta01:
--- Quote from: oBFusCATed on July 02, 2014, 01:18:46 am ---I've just tried the wxListCtrl and it behaves pretty good.
Please test if the layout is correct with this test patch on windows: http://cmpt.benbmp.org/codeblocks/patches/project_options_layout_many_target.patch
NOTE: This patch is not meant for production use, it is meant to test the layout only. The dialog is totally broken, because I've not fixed all usages of the wxListBox!
--- End quote ---
Windows 7 32 bit build with your patch.
https://github.com/stahta01/cb_misc/blob/master/Notes/project_options_layout_many_target.png
Tim S.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version