Fix transaction history table column size behavior

Author: jamescowens

src/qt/bitcoingui.cpp

@@ -1420,9 +1420,12 @@ void BitcoinGUI::gotoHistoryPage()
     historyAction->setChecked(true);
     centralWidget->setCurrentWidget(transactionView);
 
+    transactionView->resizeTableColumns();
+
     exportAction->setEnabled(true);
     disconnect(exportAction, &QAction::triggered, nullptr, nullptr);
     connect(exportAction, &QAction::triggered, transactionView, &TransactionView::exportClicked);
+
 }
 
 void BitcoinGUI::gotoAddressBookPage()

src/qt/transactiontablemodel.h

@@ -26,6 +26,8 @@ class TransactionTableModel : public QAbstractTableModel
         Amount = 4
     };
 
+    static constexpr std::initializer_list<ColumnIndex> all_ColumnIndex = {Status, Date, Type, ToAddress, Amount};
+
     /** Roles to get specific information from a transaction row.
         These are independent of column.
     */

src/qt/transactionview.cpp

@@ -1,5 +1,6 @@
 #include "transactionview.h"
 
+#include "logging.h"
 #include "transactionfilterproxy.h"
 #include "transactionrecord.h"
 #include "walletmodel.h"
@@ -37,6 +38,8 @@ TransactionView::TransactionView(QWidget *parent)
     , transactionProxyModel(nullptr)
     , transactionView(nullptr)
     , searchWidgetIconAction(new QAction())
+    , m_table_column_sizes({23, 120, 120, 400, 100})
+    , m_init_column_sizes_set(false)
 {
     setContentsMargins(0, 0, 0, 0);
 
@@ -110,7 +113,7 @@ TransactionView::TransactionView(QWidget *parent)
     view->horizontalHeader()->setHighlightSections(false);
     transactionView = view;
 
-    QVBoxLayout *tableViewLayout = new QVBoxLayout();
+    QVBoxLayout* tableViewLayout = new QVBoxLayout();
     tableViewLayout->setContentsMargins(9, 9, 9, 9);
     QFrame *tableViewFrame = new QFrame(this);
     tableViewFrame->setObjectName("historyTableFrame");
@@ -180,17 +183,6 @@ void TransactionView::setModel(WalletModel *model)
         transactionView->setSortingEnabled(true);
         transactionView->sortByColumn(TransactionTableModel::Date, Qt::DescendingOrder);
         transactionView->verticalHeader()->hide();
-
-        transactionView->horizontalHeader()->resizeSection(
-                TransactionTableModel::Status, 23);
-        transactionView->horizontalHeader()->resizeSection(
-                TransactionTableModel::Date, 120);
-        transactionView->horizontalHeader()->resizeSection(
-                TransactionTableModel::Type, 120);
-        transactionView->horizontalHeader()->setSectionResizeMode(
-                TransactionTableModel::ToAddress, QHeaderView::Stretch);
-        transactionView->horizontalHeader()->resizeSection(
-                TransactionTableModel::Amount, 100);
     }
 
     if (model && model->getOptionsModel()) {
@@ -454,3 +446,42 @@ void TransactionView::updateIcons(const QString& theme)
 {
     searchWidgetIconAction->setIcon(QIcon(":/icons/" + theme + "_search"));
 }
+
+void TransactionView::resizeTableColumns()
+{
+    if (!model) return;
+
+    if (!m_init_column_sizes_set) {
+        for (int i = 0; i < (int) m_table_column_sizes.size(); ++i) {
+            transactionView->horizontalHeader()->resizeSection(i, m_table_column_sizes[i]);
+        }
+
+        m_init_column_sizes_set = true;
+
+        return;
+    }
+
+    const int width = transactionView->horizontalHeader()->width() - 5;
+
+    int orig_header_width = 0;
+
+    for (const auto& iter : TransactionTableModel::all_ColumnIndex) {
+        orig_header_width += transactionView->horizontalHeader()->sectionSize(iter);
+    }
+
+    if (!width || !orig_header_width) return;
+
+    for (const auto& iter : TransactionTableModel::all_ColumnIndex) {
+        int section_size = transactionView->horizontalHeader()->sectionSize(iter);
+
+        transactionView->horizontalHeader()->resizeSection(
+                    iter, section_size * width / orig_header_width);
+    }
+}
+
+void TransactionView::resizeEvent(QResizeEvent *event)
+{
+    resizeTableColumns();
+
+    QWidget::resizeEvent(event);
+}

src/qt/transactionview.h

@@ -14,6 +14,7 @@ class QModelIndex;
 class QMenu;
 class QFrame;
 class QDateTimeEdit;
+
 QT_END_NAMESPACE
 
 /** Widget showing the transaction list for a wallet, including a filter row.
@@ -39,6 +40,9 @@ class TransactionView : public QFrame
         Range
     };
 
+protected:
+    void resizeEvent(QResizeEvent *event) override;
+
 private:
     WalletModel *model;
     TransactionFilterProxy *transactionProxyModel;
@@ -58,6 +62,9 @@ class TransactionView : public QFrame
 
     QWidget *createDateRangeWidget();
 
+    std::vector<int> m_table_column_sizes;
+    bool m_init_column_sizes_set;
+
 private slots:
     void contextualMenu(const QPoint &);
     void dateRangeChanged();
@@ -79,7 +86,7 @@ public slots:
     void changedAmount(const QString &amount);
     void exportClicked();
     void focusTransaction(const QModelIndex&);
-
+    void resizeTableColumns();
 };
 
 #endif // BITCOIN_QT_TRANSACTIONVIEW_H