Skip to content
Permalink
Browse files

remove QgsScrollAreaWidgetPlugin from custom widgets

it had to be removed form the widget list in uic plugin. It might be better to provide the real list of implemented widgets rather than the full gui list.

fixes #16428
  • Loading branch information
3nids committed Jul 24, 2017
1 parent b842043 commit 500175b8e8aaff73d0ce8caac6006fd79bff9752
@@ -50,6 +50,8 @@
def moduleInformation():
try:
import qgis.gui
return "qgis.gui", dir(qgis.gui)
widget_list=dir(qgis.gui)
widget_list.remove('QgsScrollArea')

This comment has been minimized.

Copy link
@3nids

3nids Jul 24, 2017

Author Member

@nyalldawson there it was....

return "qgis.gui", widget_list
except ImportError:
return "", []
@@ -36,7 +36,7 @@ SET (QGIS_CUSTOMWIDGETS_SRCS
qgsrelationreferencewidgetplugin.cpp
qgsscalerangewidgetplugin.cpp
qgsscalewidgetplugin.cpp
qgsscrollareawidgetplugin.cpp
# qgsscrollareawidgetplugin.cpp
qgsspinboxplugin.cpp
)

@@ -66,7 +66,7 @@ SET (QGIS_CUSTOMWIDGETS_MOC_HDRS
qgsrelationreferencewidgetplugin.h
qgsscalerangewidgetplugin.h
qgsscalewidgetplugin.h
qgsscrollareawidgetplugin.h
# qgsscrollareawidgetplugin.h
qgsspinboxplugin.h
)

@@ -39,7 +39,7 @@
#include "qgsrelationreferencewidgetplugin.h"
#include "qgsscalerangewidgetplugin.h"
#include "qgsscalewidgetplugin.h"
#include "qgsscrollareawidgetplugin.h"
//#include "qgsscrollareawidgetplugin.h"
#include "qgsspinboxplugin.h"


@@ -69,7 +69,7 @@ QgisCustomWidgets::QgisCustomWidgets( QObject *parent )
mWidgets.append( new QgsRelationReferenceWidgetPlugin( this ) );
mWidgets.append( new QgsScaleRangeWidgetPlugin( this ) );
mWidgets.append( new QgsScaleWidgetPlugin( this ) );
mWidgets.append( new QgsScrollAreaWidgetPlugin( this ) );
// mWidgets.append( new QgsScrollAreaWidgetPlugin( this ) ); // this is causing troubles at the moment
mWidgets.append( new QgsSpinBoxPlugin( this ) );
}

4 comments on commit 500175b

@Gustry

This comment has been minimized.

Copy link
Collaborator

@Gustry Gustry replied Jul 24, 2017

Whaou, thanks a lot, I will be able to use Processing again!

@nyalldawson

This comment has been minimized.

Copy link
Contributor

@nyalldawson nyalldawson replied Jul 24, 2017

Nice catch @3nids!

@Gustry

This comment has been minimized.

Copy link
Collaborator

@Gustry Gustry replied Jul 24, 2017

@3nids I shouldn't have this kind of UI anymore?
screen shot 2017-07-24 at 23 42 35

Or did I miss something? Thanks

@3nids

This comment has been minimized.

Copy link
Member Author

@3nids 3nids replied Jul 24, 2017

Please sign in to comment.
You can’t perform that action at this time.