Merge pull request 'Stylesheets: Breeze: misc fixes' (#135) from tobtoht/feather:stylesheet_breeze_fixes into master

Reviewed-on: feather/feather#135

src/appcontext.cpp

@@ -9,12 +9,14 @@
 #include <QDesktopWidget>
 
 #include "appcontext.h"
+#include "globals.h"
 #include "utils/tails.h"
 #include "utils/whonix.h"
 #include "utils/utils.h"
 #include "utils/prices.h"
 #include "utils/networktype.h"
 #include "utils/wsclient.h"
+#include "utils/config.h"
 
 // libwalletqt
 #include "libwalletqt/WalletManager.h"
@@ -28,7 +30,6 @@
 #include "model/SubaddressModel.h"
 #include "utils/keysfiles.h"
 #include "utils/networktype.h"
-#include "utils/config.h"
 
 
 Prices *AppContext::prices = nullptr;
@@ -195,7 +196,7 @@ void AppContext::initWS() {
 
 void AppContext::onCancelTransaction(PendingTransaction *tx, const QString &address) {
     // tx cancelled by user
-    double amount = tx->amount() / AppContext::cdiv;
+    double amount = tx->amount() / globals::cdiv;
     emit createTransactionCancelled(address, amount);
     this->currentWallet->disposeTransaction(tx);
 }
@@ -234,8 +235,8 @@ void AppContext::onCreateTransaction(const QString &address, const double amount
         return;
     }
 
-    auto balance = this->currentWallet->balance() / AppContext::cdiv;
-    auto unlocked_balance = this->currentWallet->unlockedBalance() / AppContext::cdiv;
+    auto balance = this->currentWallet->balance() / globals::cdiv;
+    auto unlocked_balance = this->currentWallet->unlockedBalance() / globals::cdiv;
     if(!all && amount > unlocked_balance) {
         emit createTransactionError("Not enough money to spend");
         return;
@@ -244,7 +245,7 @@ void AppContext::onCreateTransaction(const QString &address, const double amount
         return;
     }
 
-    auto amount_num = static_cast<quint64>(amount * AppContext::cdiv);
+    auto amount_num = static_cast<quint64>(amount * globals::cdiv);
     qDebug() << "creating tx";
     if(all || amount == balance)
         this->currentWallet->createTransactionAllAsync(address, "", this->tx_mixin, this->tx_priority);
@@ -691,19 +692,19 @@ AppContext::~AppContext() {
 // ############################################## LIBWALLET QT #########################################################
 
 void AppContext::onMoneySpent(const QString &txId, quint64 amount) {
-    auto amount_num = amount / AppContext::cdiv;
+    auto amount_num = amount / globals::cdiv;
     qDebug() << Q_FUNC_INFO << txId << " " << QString::number(amount_num);
 }
 
 void AppContext::onMoneyReceived(const QString &txId, quint64 amount) {
     // Incoming tx included in a block.
-    auto amount_num = amount / AppContext::cdiv;
+    auto amount_num = amount / globals::cdiv;
     qDebug() << Q_FUNC_INFO << txId << " " << QString::number(amount_num);
 }
 
 void AppContext::onUnconfirmedMoneyReceived(const QString &txId, quint64 amount) {
     // Incoming transaction in pool
-    auto amount_num = amount / AppContext::cdiv;
+    auto amount_num = amount / globals::cdiv;
     qDebug() << Q_FUNC_INFO << txId << " " << QString::number(amount_num);
 
     if(this->currentWallet->synchronized()) {
@@ -790,12 +791,12 @@ void AppContext::storeWallet() {
 
 void AppContext::updateBalance() {
     if(!this->currentWallet)
-        throw std::runtime_error("this should not happen, ever");
+        return;
 
-    AppContext::balance = this->currentWallet->balance() / AppContext::cdiv;
+    AppContext::balance = this->currentWallet->balance() / globals::cdiv;
     auto balance_str = QString::number(balance, 'f');
 
-    double unlocked = this->currentWallet->unlockedBalance() / AppContext::cdiv;
+    double unlocked = this->currentWallet->unlockedBalance() / globals::cdiv;
     auto unlocked_str = QString::number(unlocked, 'f');
 
     emit balanceUpdated(balance, unlocked, balance_str, unlocked_str);

src/appcontext.h

@@ -69,7 +69,6 @@ public:
     const unsigned int kdfRounds = 1;
     PendingTransaction::Priority tx_priority = PendingTransaction::Priority::Priority_Low;
     quint32 tx_mixin = static_cast<const quint32 &>(10);
-    static constexpr const double cdiv = 1e12;
     QString seedLanguage = "English";  // 14 word `monero-seed` only has English
 
     QNetworkAccessManager *network;

src/coinswidget.cpp

@@ -42,14 +42,14 @@ CoinsWidget::CoinsWidget(QWidget *parent)
     // context menu
     ui->coins->setContextMenuPolicy(Qt::CustomContextMenu);
 
-    m_thawOutputAction = new QAction("Thaw output");
-    m_freezeOutputAction = new QAction("Freeze output");
+    m_thawOutputAction = new QAction("Thaw output", this);
+    m_freezeOutputAction = new QAction("Freeze output", this);
 
-    m_freezeAllSelectedAction = new QAction("Freeze selected");
-    m_thawAllSelectedAction = new QAction("Thaw selected");
+    m_freezeAllSelectedAction = new QAction("Freeze selected", this);
+    m_thawAllSelectedAction = new QAction("Thaw selected", this);
 
-    m_viewOutputAction = new QAction(QIcon(":/assets/images/info.png"), "Details");
-    m_sweepOutputAction = new QAction("Sweep output");
+    m_viewOutputAction = new QAction(QIcon(":/assets/images/info.png"), "Details", this);
+    m_sweepOutputAction = new QAction("Sweep output", this);
     connect(m_freezeOutputAction, &QAction::triggered, this, &CoinsWidget::freezeOutput);
     connect(m_thawOutputAction, &QAction::triggered, this, &CoinsWidget::thawOutput);
     connect(m_viewOutputAction, &QAction::triggered, this, &CoinsWidget::viewOutput);
@@ -65,7 +65,7 @@ CoinsWidget::CoinsWidget(QWidget *parent)
 void CoinsWidget::setModel(CoinsModel * model, Coins * coins) {
     m_coins = coins;
     m_model = model;
-    m_proxyModel = new CoinsProxyModel;
+    m_proxyModel = new CoinsProxyModel(this);
     m_proxyModel->setSourceModel(m_model);
     ui->coins->setModel(m_proxyModel);
     ui->coins->setColumnHidden(CoinsModel::Spent, true);
@@ -207,6 +207,7 @@ void CoinsWidget::onSweepOutput() {
     qCritical() << "key image: " << keyImage;
 
     emit sweepOutput(keyImage, dialog->address(), dialog->churn(), dialog->outputs());
+    dialog->deleteLater();
 }
 
 void CoinsWidget::copy(copyField field) {

src/dialog/txconfdialog.cpp

@@ -8,6 +8,7 @@
 #include "model/ModelUtils.h"
 #include "libwalletqt/WalletManager.h"
 #include "txconfadvdialog.h"
+#include "globals.h"
 
 #include <QMessageBox>
 
@@ -31,15 +32,15 @@ TxConfDialog::TxConfDialog(AppContext *ctx, PendingTransaction *tx, const QStrin
     };
 
     QString amount = WalletManager::displayAmount(tx->amount());
-    QString amount_fiat = convert(tx->amount() / AppContext::cdiv);
+    QString amount_fiat = convert(tx->amount() / globals::cdiv);
     ui->label_amount->setText(QString("%1 (%2 %3)").arg(amount, amount_fiat, preferredCur));
 
     QString fee = WalletManager::displayAmount(tx->fee());
-    QString fee_fiat = convert(tx->fee() / AppContext::cdiv);
+    QString fee_fiat = convert(tx->fee() / globals::cdiv);
     ui->label_fee->setText(QString("%1 (%2 %3)").arg(fee, fee_fiat, preferredCur));
 
     QString total = WalletManager::displayAmount(tx->amount() + tx->fee());
-    QString total_fiat = convert((tx->amount() + tx->fee()) / AppContext::cdiv);
+    QString total_fiat = convert((tx->amount() + tx->fee()) / globals::cdiv);
     ui->label_total->setText(QString("%1 (%2 %3)").arg(total, total_fiat, preferredCur));
 
     ui->label_address->setText(ModelUtils::displayAddress(address, 2));

src/globals.h

@@ -0,0 +1,14 @@
+// SPDX-License-Identifier: BSD-3-Clause
+// Copyright (c) 2020, The Monero Project.
+
+#ifndef FEATHER_GLOBALS_H
+#define FEATHER_GLOBALS_H
+
+#include <QtGlobal>
+
+namespace globals
+{
+    const qreal cdiv = 1e12;
+}
+
+#endif //FEATHER_GLOBALS_H

src/mainwindow.cpp

@@ -660,10 +660,18 @@ void MainWindow::onWalletOpened() {
 
 void MainWindow::onBalanceUpdated(double balance, double unlocked, const QString &balance_str, const QString &unlocked_str) {
     qDebug() << Q_FUNC_INFO;
+    bool hide = config()->get(Config::hideBalance).toBool();
+
     auto label_str = QString("Balance: %1 XMR").arg(unlocked_str);
     if(balance > unlocked)
         label_str += QString(" (+%1 XMR unconfirmed)").arg(QString::number(balance - unlocked, 'f'));
+
+    if (hide) {
+        label_str = "Balance: HIDDEN";
+    }
+
     m_statusLabelBalance->setText(label_str);
+    m_balanceWidget->setHidden(hide);
 }
 
 void MainWindow::onSynchronized() {

src/model/CoinsModel.cpp

@@ -6,6 +6,7 @@
 #include "Coins.h"
 #include <wallet/api/wallet2_api.h>
 #include "ModelUtils.h"
+#include "globals.h"
 
 #include <QDebug>
 #include <QHash>
@@ -190,10 +191,14 @@ QVariant CoinsModel::parseTransactionInfo(const CoinsInfo &cInfo, int column, in
         case SpentHeight:
             return cInfo.spentHeight();
         case Amount:
-            return QString::number(cInfo.amount() / 1e12, 'f', 12);
+        {
+            if (role == Qt::UserRole) {
+                return cInfo.amount() / globals::cdiv;
+            }
+            return QString::number(cInfo.amount() / globals::cdiv, 'f', 12);
+        }
         case Frozen:
             return cInfo.frozen();
-
         default:
         {
             qCritical() << "Unimplemented role";

src/model/CoinsProxyModel.cpp

@@ -7,6 +7,7 @@
 CoinsProxyModel::CoinsProxyModel(QObject *parent)
     : QSortFilterProxyModel(parent)
 {
+    setSortRole(Qt::UserRole);
 }
 
 bool CoinsProxyModel::filterAcceptsRow(int sourceRow, const QModelIndex &sourceParent) const

src/model/TransactionHistoryModel.cpp

@@ -4,6 +4,7 @@
 #include "TransactionHistoryModel.h"
 #include "TransactionHistory.h"
 #include "TransactionInfo.h"
+#include "globals.h"
 
 #include <QDateTime>
 #include <QDebug>
@@ -151,7 +152,7 @@ QVariant TransactionHistoryModel::parseTransactionInfo(const TransactionInfo &tI
         }
         case Column::Amount:
         {
-            QString amount = QString::number(tInfo.atomicAmount() / 1e12, 'f', 4);
+            QString amount = QString::number(tInfo.atomicAmount() / globals::cdiv, 'f', 4);
             amount = (tInfo.direction() == TransactionInfo::Direction_Out && tInfo.amount() > 0) ? "-" + amount : "+" + amount;
             return amount;
         }

src/settings.cpp

@@ -22,6 +22,11 @@ Settings::Settings(QWidget *parent) :
     connect(ui->btnCopyToClipboard, &QPushButton::clicked, this, &Settings::copyToClipboard);
     connect(ui->checkBox_checkForAppUpdates, &QCheckBox::clicked, this, &Settings::checkboxExternalLinkWarn);
     connect(ui->checkBox_externalLink, &QCheckBox::clicked, this, &Settings::checkboxExternalLinkWarn);
+    connect(ui->checkBox_hideBalance, &QCheckBox::toggled, [this](bool toggled){
+        config()->set(Config::hideBalance, toggled);
+        m_ctx->updateBalance();
+    });
+
     connect(ui->closeButton, &QDialogButtonBox::accepted, this, &Settings::close);
 
     // nodes
@@ -32,6 +37,7 @@ Settings::Settings(QWidget *parent) :
     // setup checkboxes
     ui->checkBox_externalLink->setChecked(config()->get(Config::warnOnExternalLink).toBool());
     ui->checkBox_checkForAppUpdates->setChecked(config()->get(Config::checkForAppUpdates).toBool());
+    ui->checkBox_hideBalance->setChecked(config()->get(Config::hideBalance).toBool());
 
     // setup comboboxes
     auto settingsHomeWidget = config()->get(Config::homeWidget).toString();

src/settings.ui

@@ -7,7 +7,7 @@
     <x>0</x>
     <y>0</y>
     <width>1019</width>
-    <height>358</height>
+    <height>396</height>
    </rect>
   </property>
   <property name="windowTitle">
@@ -175,6 +175,11 @@
            <string>moneroblocks.info</string>
           </property>
          </item>
+         <item>
+          <property name="text">
+           <string>blockchair.com</string>
+          </property>
+         </item>
         </widget>
        </item>
        <item row="4" column="0">
@@ -191,6 +196,13 @@
          </property>
         </widget>
        </item>
+       <item row="6" column="0">
+        <widget class="QCheckBox" name="checkBox_hideBalance">
+         <property name="text">
+          <string>Hide balance</string>
+         </property>
+        </widget>
+       </item>
       </layout>
      </widget>
      <widget class="QWidget" name="tab_node">

src/ui/BreezeStyleSheets/dark.qss

@@ -57,13 +57,13 @@ QWidget
     outline: 0;
 }
 
-QWidget:item:hover
+QWidget::item:hover
 {
     background-color: #3daee9;
     color: #eff0f1;
 }
 
-QWidget:item:selected
+QWidget::item:selected
 {
     background-color: #3daee9;
 }
@@ -525,7 +525,7 @@ QTextEdit
 
 QPlainTextEdit
 {
-    background-color: #232629;;
+    background-color: #232629;
     color: #eff0f1;
     border-radius: 0.2ex;
     border: 0.1ex solid #76797c;

src/ui/BreezeStyleSheets/light.qss

@@ -54,13 +54,13 @@ QWidget
     outline: 0;
 }
 
-QWidget:item:hover
+QWidget::item:hover
 {
     background-color: #33A4DF;
     color: #31363B;
 }
 
-QWidget:item:selected
+QWidget::item:selected
 {
     background-color: #33A4DF;
 }
@@ -567,7 +567,7 @@ QMainWindow::separator:hover
     color: white;
     padding-left: 0.4ex;
     border: 0.1ex solid #BAB9B8;
-    spacing: 0.2x;
+    spacing: 0.2ex;
 }
 
 QMenu::separator
@@ -1659,12 +1659,14 @@ QPushButton:flat {
     border: none;
 }
 
-QScrollBar::handle:vertical
+QScrollBar::handle:vertical,
+QScrollBar::handle:horizontal
 {
     background-color: #BAB9B8;
 }
 
-QScrollBar:vertical
+QScrollBar:vertical,
+QScrollBar:horizontal
 {
     background-color: #eff0f1;
     border: 0.1ex transparent #eff0f1;

src/utils/config.cpp

@@ -47,6 +47,7 @@ static const QHash<Config::ConfigKey, ConfigDirective> configStrings = {
         {Config::geometry, {QS("geometry"), {}}},
         {Config::windowState, {QS("windowState"), {}}},
         {Config::firstRun,{QS("firstRun"), false}},
+        {Config::hideBalance, {QS("hideBalance"), false}}
 };
 
 

src/utils/config.h

@@ -44,7 +44,8 @@ public:
         showTabXMRig,
         geometry,
         windowState,
-        firstRun
+        firstRun,
+        hideBalance
     };
 
     ~Config() override;

src/utils/utils.cpp

@@ -382,6 +382,11 @@ QString Utils::blockExplorerLink(const QString &blockExplorer, NetworkType::Type
             return QString("https://moneroblocks.info/tx/%1").arg(txid);
         }
     }
+    else if (blockExplorer == "blockchair.com") {
+        if (nettype == NetworkType::MAINNET) {
+            return QString("https://blockchair.com/monero/transaction/%1").arg(txid);
+        }
+    }
 
     switch (nettype) {
         case NetworkType::MAINNET:

src/utils/xmrtoorder.cpp

@@ -7,6 +7,7 @@
 
 #include "libwalletqt/Wallet.h"
 #include "appcontext.h"
+#include "globals.h"
 #include "utils/xmrto.h"
 
 XmrToOrder::XmrToOrder(AppContext *ctx, UtilsNetworking *network, QString baseUrl, bool clearnet, XmrToRates *rates, QObject *parent) :
@@ -38,7 +39,7 @@ void XmrToOrder::onTransactionCancelled(const QString &address, double amount) {
 void XmrToOrder::onTransactionCommitted(bool status, PendingTransaction *tx, const QStringList& txid) {
     // listener for all outgoing transactions - will try to match the exact amount to this order.
     if(this->state == OrderState::Status_OrderUnpaid){
-        if(tx->amount() / AppContext::cdiv == this->incoming_amount_total) {
+        if(tx->amount() / globals::cdiv == this->incoming_amount_total) {
             if(!status) {
                 this->errorMsg = "TX failed to commit";
                 this->changeState(OrderState::Status_OrderFailed);
@@ -221,7 +222,7 @@ void XmrToOrder::changeState(OrderState _state) {
         case OrderState::Status_OrderUnpaid:
             // need to send Monero
             if(!m_paymentRequested) {
-                auto unlocked_balance = m_ctx->currentWallet->unlockedBalance() / AppContext::cdiv;
+                auto unlocked_balance = m_ctx->currentWallet->unlockedBalance() / globals::cdiv;
                 if (this->incoming_amount_total >= unlocked_balance) {
                     this->state = OrderState::Status_OrderFailed;
                     emit orderFailed(this);

src/widgets/ccsprogressdelegate.cpp

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: BSD-3-Clause
 // Copyright (c) 2020, The Monero Project.
 
-#include "cssprogressdelegate.h"
+#include "ccsprogressdelegate.h"
 
 #include <QApplication>
 

src/widgets/ccswidget.cpp

@@ -8,9 +8,9 @@
 #include <QDesktopServices>
 
 #include "ccswidget.h"
-#include "ui_csswidget.h"
+#include "ui_ccswidget.h"
 #include "utils/utils.h"
-#include "cssprogressdelegate.h"
+#include "ccsprogressdelegate.h"
 
 CCSWidget::CCSWidget(QWidget *parent) :
         QWidget(parent),

src/widgets/ccswidget.ui

@@ -59,7 +59,7 @@
         <bool>false</bool>
        </property>
        <property name="text">
-        <string>Powered by css.getmonero.org</string>
+        <string>Powered by ccs.getmonero.org</string>
        </property>
        <property name="alignment">
         <set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>