qml: Add DropShadow to the OptionSwitch indicatorButton

[shaavan]

• This PR is a follow-up to Add and demo ConnectionSettings and ContinueButton components #101 and addresses the suggestions given there after it was merged.
• This PR also adds the DropShadow for the OptionSwitch's indicatorButton, as per the main design file of the project.

Some points to note:

• Used === instead of == to avoid type coercion.
• Chose sample value based on official recommended guidelines (sample = 2*radius + 1) https://doc.qt.io/qt-5/qml-qtgraphicaleffects-dropshadow.html#samples-prop
• Used rgba to hex convertor for shadow color: https://rgbacolorpicker.com/rgba-to-hex

[shaavan]

Running the current version of the PR will cause the following error.

qrc:/qml/controls/OptionSwitch.qml:7:1: module "QtGraphicalEffects" is not installed

And to create dropShadow, we require this module. I tried to figure out some workaround, but I thought it's better to ask and go with the correct approach on the get-go.

[shaavan]

Updated from e2d1635 to 2e8b096.

Changes:

• Made appropriate styling changes.

Diff:

diff --git a/src/qml/components/ConnectionSettings.qml b/src/qml/components/ConnectionSettings.qml
index 090e8e793..35f8e5d3a 100644
--- a/src/qml/components/ConnectionSettings.qml
+++ b/src/qml/components/ConnectionSettings.qml
@@ -15,21 +15,21 @@ ColumnLayout {
     }
     Setting {
         Layout.fillWidth: true
-        header:  qsTr("Daily upload limit")
+        header: qsTr("Daily upload limit")
     }
     Setting {
         Layout.fillWidth: true
-        header:  qsTr("Connection limit")
+        header: qsTr("Connection limit")
     }
     Setting {
         Layout.fillWidth: true
-        header:  qsTr("Listening enabled")
-        description:  qsTr("Reduces data usage.")
+        header: qsTr("Listening enabled")
+        description: qsTr("Reduces data usage.")
     }
     Setting {
         last: true
         Layout.fillWidth: true
-        header:  qsTr("Blocks Only")
-        description:  qsTr("Do not transfer unconfirmed transactions. Also disabled listening.")
+        header: qsTr("Blocks Only")
+        description: qsTr("Do not transfer unconfirmed transactions. Also disabled listening.")
     }
 }

[promag]

Running the current version of the PR will cause the following error.

qrc:/qml/controls/OptionSwitch.qml:7:1: module "QtGraphicalEffects" is not installed

@shaavan try adding

Q_IMPORT_PLUGIN(QtGraphicalEffectsPlugin)
Q_IMPORT_PLUGIN(QtGraphicalEffectsPrivatePlugin)

gui-qml/src/qml/bitcoin.cpp

Lines 43 to 52 in ef98ffa

	#if defined(QT_STATICPLUGIN)
	#include <QtPlugin>
	Q_IMPORT_PLUGIN(QtQmlPlugin)
	Q_IMPORT_PLUGIN(QtQmlModelsPlugin)
	Q_IMPORT_PLUGIN(QtQuick2Plugin)
	Q_IMPORT_PLUGIN(QtQuick2WindowPlugin)
	Q_IMPORT_PLUGIN(QtQuickLayoutsPlugin);
	Q_IMPORT_PLUGIN(QtQuickControls2Plugin);
	Q_IMPORT_PLUGIN(QtQuickTemplates2Plugin);
	#endif

[shaavan]

Updated from 2e8b096 to 109682b (pr111.01 -> pr112.02)
Addressed @jarolrod, @promag suggestions.

Changes:

• Follow-ups related to Add and demo ConnectionSettings and ContinueButton components #101 have been removed from this PR. The stated follow-ups were merged in qml: Follow-up to PR#101 #112.
• Used Q_IMPORT_PLUGIN to import relevant modules in src/qml/bitcoin.cpp file.

The PR is still in WIP. Currently, using the Q_IMPORT_PLUGIN caused the following error.

/usr/bin/ld: qt/libbitcoinqt.a(qt_libbitcoinqt_a-bitcoin.o): in function `_GLOBAL__sub_I__Z10QmlGuiMainiPPc':
bitcoin.cpp:(.text.startup+0xc2): undefined reference to `qt_static_plugin_QtGraphicalEffectsPlugin()'
/usr/bin/ld: bitcoin.cpp:(.text.startup+0xd2): undefined reference to `qt_static_plugin_QtGraphicalEffectsPrivatePlugin()'
collect2: error: ld returned 1 exit status

[jarolrod]

@shaavan as properly supporting this requires OpenGL support, let's leave the dropshadows for later.

[shaavan]

supporting this requires OpenGL support, let's leave the dropshadows for later.

I understand. I shall close this PR for now. We can restart work on this once OpenGL support is appropriately integrated.

[shaavan]

This PR is moved to #132.

Rationale:

• For some reason. Force pushing the PR branch was not updating the PR. Hence decided to move the PR.