Idea: use a typedef to change the many #if/#else blocks to a general form.

From

#ifdef ENABLE_WEBENGINE
QWebEngineView            *m_tableView;
#else
KWebView                  *m_tableView;
#endif

in many places to

#ifdef ENABLE_WEBENGINE
typedef QWebEngineView KMMPrintView;
#else
typedef KWebView KMMPrintView;
#endif

and

KMMPrintView                  *m_tableView;

... hit the submit button too early ...

to reduce the number of #if/#else blocks throughout the patch. In some it's still needed though due to different methods of the two objects.

I am not sure if placing the typedefs in config-kmymoney.h.cmake is the right solution and does not confuse cmake but that would make a nice place.

In D6975#129929, @tbaumgart wrote:

... hit the submit button too early ...

to reduce the number of #if/#else blocks throughout the patch. In some it's still needed though due to different methods of the two objects.

I am not sure if placing the typedefs in config-kmymoney.h.cmake is the right solution and does not confuse cmake but that would make a nice place.

I wouldn't elaborate on these things too much because it doesn't bring us nothing practical. This patch means to provide stable transition to new QWebEngine and routines here aren't meant to last forever.