Commit a046f58c authored by Dan Leinir Turthra Jensen's avatar Dan Leinir Turthra Jensen 🌈
Browse files

sketch: don't grab the config widget config for color transfer

This function causes some hefty slowdowns on startup in Windows
if we don't do it like this. The reason is that the url completion
widget from kdelibs is deadly slow to destruct on windows, causing
about eight seconds of slowdown on startup (at least on my machine)
parent 5f66d90a
......@@ -124,7 +124,12 @@ void FiltersModel::addFilter(KisFilterSP filter)
int newRow = d->filters.count();
beginInsertRows(QModelIndex(), newRow, newRow);
d->filters << filter;
if(filter->showConfigurationWidget()) {
// We're not asking for the config widget config for color transfer
// The reason for this is that the completion widget is VERY slow to destruct on
// Windows. This can be removed once that bug has been alleviated at some later
// point in time, but for now it has no side effects, as this filter's default
// config is fine anyway.
if(filter->showConfigurationWidget() && filter->id() != QLatin1String("colortransfer")) {
KisConfigWidget* wdg = filter->createConfigurationWidget(0, d->view->activeNode()->original());
wdg->deleteLater();
d->configurations << KisSafeFilterConfigurationSP(static_cast<KisFilterConfiguration*>(wdg->configuration()));
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment