From f8e9fdef65618f453c8f6784fa2dbed3500d5e26 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89milie=20Feral?= Date: Fri, 22 Feb 2019 15:38:10 +0100 Subject: [PATCH] [solver] EquationListView: clean setBraceStyle to always re-layout --- apps/solver/equation_list_view.cpp | 8 ++++---- apps/solver/list_controller.cpp | 1 - 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/apps/solver/equation_list_view.cpp b/apps/solver/equation_list_view.cpp index 78d2e9e0a..c666a0bb3 100644 --- a/apps/solver/equation_list_view.cpp +++ b/apps/solver/equation_list_view.cpp @@ -22,10 +22,10 @@ EquationListView::EquationListView(Responder * parentResponder, TableViewDataSou } void EquationListView::setBraceStyle(BraceStyle style) { - if (m_braceStyle != style) { - m_braceStyle = style; - layoutSubviews(); - } + /* Even if the brace style does not change, we want to relayout in case the + * size of the braced object has changed. */ + m_braceStyle = style; + layoutSubviews(); } void EquationListView::scrollViewDidChangeOffset(ScrollViewDataSource * scrollViewDataSource) { diff --git a/apps/solver/list_controller.cpp b/apps/solver/list_controller.cpp index 359dc1bc2..e5b8d21ab 100644 --- a/apps/solver/list_controller.cpp +++ b/apps/solver/list_controller.cpp @@ -224,7 +224,6 @@ bool ListController::removeModelRow(ExpressionModel * model) { void ListController::reloadBrace() { EquationListView::BraceStyle braceStyle = m_equationStore->numberOfModels() <= 1 ? EquationListView::BraceStyle::None : (m_equationStore->numberOfModels() == m_equationStore->maxNumberOfModels() ? EquationListView::BraceStyle::Full : EquationListView::BraceStyle::OneRowShort); m_equationListView.setBraceStyle(braceStyle); - m_equationListView.layoutSubviews(); } SelectableTableView * ListController::selectableTableView() {