wxUiEditor icon indicating copy to clipboard operation
wxUiEditor copied to clipboard

Bug: Custom includes files are not allowed using the common syntax #include <myinclude/myfile.h>

Open rossanoparis opened this issue 1 year ago • 4 comments

Description:

Perhaps I'm missing something, but I noticed that wxUiEditor doesn't allow to use angle brackets to include customized files. As I see it, it seams wrong, because all user customized class must be inserted in a relative manner using the syntax #include "../../../myinclude/myfile.h" instead of #include <myinclude/myfile.h>

Normally a programmer has got its own folders where common files (especially, for a RAD tool, customized controls) are stored and shared among different application. The common way to include such files is to use the syntax #include <myinclude/myfile.h> where the base folders are defined using compiler/IDE include directories. Otherwise every time, for any reason, an application folder or a common library folder is moved, potentially many applications must be modified.

That's why in my opinion the following property, must be editable by user if she/he needs so. In this example I should be able to insert <wxhControls/wxhButton.h> instead of wxhControls/wxhButton.h image

Actually the generated code by wxUiEditor is #include "wxhControls/wxhButton.h" which leads in many compilation errors as the compiler is not able to find the file "wxhControls/wxhButton.h" Such an errors can't be solved, as the user is not able to insert <wxhControls/wxhButton.h>

This wxUiEditor behaviour is weird and it should be avoided in general, and it is blocking for planning to port projects from other RAD tools.

Please take in consideration this last suggestion or both Everything can work as it is, simply moving the "src_preamble" at the beginning of implementation file, which is, in my opinion, the right place for it, not after local includes. image

Or, at least, it is necessary an option for those who uses "precompiled headers"

rossanoparis avatar Nov 02 '23 12:11 rossanoparis