From 5de60a94eddc93182e4e332016a085646aa1f877 Mon Sep 17 00:00:00 2001 From: Sambhav Kothari Date: Wed, 18 Jan 2017 04:32:19 +0530 Subject: [PATCH] Refactor code --- picard/ui/options/interface.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/picard/ui/options/interface.py b/picard/ui/options/interface.py index 821b866d9..9d56bad38 100644 --- a/picard/ui/options/interface.py +++ b/picard/ui/options/interface.py @@ -150,6 +150,8 @@ class InterfaceOptionsPage(OptionsPage): self.ui.up_button.clicked.connect(partial(self.move_item, 1)) self.ui.down_button.clicked.connect(partial(self.move_item, -1)) self.ui.toolbar_layout_list.currentRowChanged.connect(self.update_buttons) + self.ui.toolbar_layout_list.setDragDropMode(QtGui.QAbstractItemView.DragDrop) + self.ui.toolbar_layout_list.setDefaultDropAction(QtCore.Qt.MoveAction) def load(self): self.ui.toolbar_show_labels.setChecked(config.setting["toolbar_show_labels"]) @@ -233,8 +235,6 @@ class InterfaceOptionsPage(OptionsPage): return self.ui.toolbar_layout_list.currentRow() def populate_action_list(self): - self.ui.toolbar_layout_list.setDragDropMode(QtGui.QAbstractItemView.DragDrop) - self.ui.toolbar_layout_list.setDefaultDropAction(QtCore.Qt.MoveAction) self.ui.toolbar_layout_list.clear() for name in config.setting['toolbar_layout']: if self._valid_action(name):