TagsWidget: Rely on QTableWidget::removeRow()

dg0yt · dg0yt · commit 374561863b28 · 2021-03-10T18:32:41.000+01:00
... when removing the first row. Fixes a quirk with the value of the
new first row being invisible or lost.
diff --git a/src/gui/widgets/tags_widget.cpp b/src/gui/widgets/tags_widget.cpp
@@ -206,13 +206,6 @@ void TagsWidget::cellChange(int row, int column)
 				// Jump to previous row
 				tags_table->setCurrentCell(row - 1, 1);
 			}
-			else
-			{
-				// Reset current row
-				tags_table->item(row, 1)->setText({});
-				auto value_item = tags_table->item(row, 1);
-				value_item->setFlags(value_item->flags() & ~Qt::ItemIsEnabled);
-			}
 		}
 		else if (!key.isEmpty())
 		{

Original file line number	Diff line number	Diff line change
`@@ -206,13 +206,6 @@ void TagsWidget::cellChange(int row, int column)`
`206`	`206`	`// Jump to previous row`
`207`	`207`	`tags_table->setCurrentCell(row - 1, 1);`
`208`	`208`	`}`
`209`		`- else`
`210`		`- {`
`211`		`- // Reset current row`
`212`		`- tags_table->item(row, 1)->setText({});`
`213`		`- auto value_item = tags_table->item(row, 1);`
`214`		`- value_item->setFlags(value_item->flags() & ~Qt::ItemIsEnabled);`
`215`		`- }`
`216`	`209`	`}`
`217`	`210`	`else if (!key.isEmpty())`
`218`	`211`	`{`