diff --git a/DblSidedTool.py b/DblSidedTool.py index c9cdbd9c..7446dd8b 100644 --- a/DblSidedTool.py +++ b/DblSidedTool.py @@ -24,7 +24,7 @@ class DblSidedTool(FlatCAMTool): ## Layer to mirror self.object_combo = QtGui.QComboBox() - self.object_combo.setModel(self.app.collection) + self.object_combo.setModel(self.app.collection.model) self.botlay_label = QtGui.QLabel("Bottom Layer:") self.botlay_label.setToolTip( "Layer to be mirrorer." @@ -68,7 +68,7 @@ class DblSidedTool(FlatCAMTool): self.point = EvalEntry() self.point_box_container.addWidget(self.point) self.box_combo = QtGui.QComboBox() - self.box_combo.setModel(self.app.collection) + self.box_combo.setModel(self.app.collection.model) self.point_box_container.addWidget(self.box_combo) self.box_combo.hide() @@ -180,6 +180,12 @@ class DblSidedTool(FlatCAMTool): px = 0.5 * (xmin + xmax) py = 0.5 * (ymin + ymax) + # Ensure that the selected object will display when it is mirrored. + # If an object's plot setting is False it will still be available in + # the combo box. If the plot is not enforced to True then the user + # gets no feedback of the operation. + fcobj.options["plot"] = True; + fcobj.mirror(axis, [px, py]) fcobj.plot() diff --git a/FlatCAMApp.py b/FlatCAMApp.py index 75d8e916..9c341998 100644 --- a/FlatCAMApp.py +++ b/FlatCAMApp.py @@ -555,7 +555,7 @@ class App(QtCore.QObject): self.ui.options_combo.activated.connect(self.on_options_combo_change) self.options_form.units_radio.group_toggle_fn = self.on_toggle_units #Notebook tabs - self.ui.notebook.currentChanged.connect(self.on_tab_change) + #self.ui.notebook.currentChanged.connect(self.on_tab_change) #################### ### Other setups ### diff --git a/FlatCAMObj.py b/FlatCAMObj.py index 317c1d94..f37ff471 100644 --- a/FlatCAMObj.py +++ b/FlatCAMObj.py @@ -30,6 +30,8 @@ class FlatCAMObj(QtCore.QObject): # Instance of the application to which these are related. # The app should set this value. app = None + + option_changed = QtCore.pyqtSignal(QtCore.QObject, str) def __init__(self, name): """ @@ -79,7 +81,8 @@ class FlatCAMObj(QtCore.QObject): setattr(self, attr, d[attr]) def on_options_change(self, key): - self.emit(QtCore.SIGNAL("optionChanged"), key) + #self.emit(QtCore.SIGNAL("optionChanged()"), key) + self.option_changed.emit(self, key) def set_ui(self, ui): self.ui = ui diff --git a/ObjectCollection.py b/ObjectCollection.py index 8d1b44e4..e58442fe 100644 --- a/ObjectCollection.py +++ b/ObjectCollection.py @@ -25,7 +25,8 @@ class KeySensitiveListView(QtGui.QListView): self.keyPressed.emit(event.key()) -class ObjectCollection(QtCore.QAbstractListModel): +#class ObjectCollection(QtCore.QAbstractListModel): +class ObjectCollection(): """ Object storage and management. """ @@ -45,7 +46,7 @@ class ObjectCollection(QtCore.QAbstractListModel): } def __init__(self, parent=None): - QtCore.QAbstractListModel.__init__(self, parent=parent) + #QtCore.QAbstractListModel.__init__(self, parent=parent) ### Icons for the list view self.icons = {} for kind in ObjectCollection.icon_files: @@ -171,9 +172,26 @@ class ObjectCollection(QtCore.QAbstractListModel): self.model.appendRow(item) + obj.option_changed.connect(self.on_object_option_changed) + # Required after appending (Qt MVC) #self.endInsertRows() + def on_object_option_changed(self, obj, key): + self.model.blockSignals(True) + name = obj.options["name"] + state = 0 #Qt.Unchecked + for index in range(self.model.rowCount()): + item = self.model.item(index) + if self.object_list[item.row()].options["name"] == name: + if obj.options["plot"] == True: + state = 2 #Qt.Checked + + item.setCheckState(state) + obj.ui.plot_cb.set_value(state) + break + self.model.blockSignals(False) + def get_names(self): """ Gets a list of the names of all objects in the collection. @@ -327,13 +345,13 @@ class ObjectCollection(QtCore.QAbstractListModel): def delete_all(self): FlatCAMApp.App.log.debug(str(inspect.stack()[1][3]) + "--> OC.delete_all()") - self.beginResetModel() +# self.beginResetModel() self.model.removeRows(0, self.model.rowCount()) self.object_list = [] self.checked_indexes = [] - self.endResetModel() +# self.endResetModel() def get_list(self): return self.object_list