Merge pull request #1756

eb311b9 receive: qrcode fixes (mmbyday)
monero-project · Dec 1, 2018 · 401aafa · 401aafa
2 parents eb7e271 + eb311b9
commit 401aafa
Showing 1 changed file with 17 additions and 12 deletions.
diff --git a/pages/Receive.qml b/pages/Receive.qml
@@ -53,15 +53,15 @@ Rectangle {
     property alias addressText : pageReceive.current_address
 
     function makeQRCodeString() {
-        var s = "monero:"
-        var nfields = 0
-        s += current_address;
-        var amount = amountToReceiveLine.text.trim()
-        if (amount !== "" && amount.slice(-1) !== ".") {
-          s += (nfields++ ? "&" : "?")
-          s += "tx_amount=" + amount
+        var XMR_URI_SCHEME = "monero:"
+        var XMR_AMOUNT = "tx_amount"
+        var qrCodeString =""
+        var amount = amountToReceiveLine.text
+        qrCodeString += (XMR_URI_SCHEME + current_address)
+        if (amount !== ""){
+          qrCodeString += ("?" + XMR_AMOUNT + "=" + amount)
         }
-        return s
+        return qrCodeString
     }
 
     function update() {
@@ -452,8 +452,13 @@ Rectangle {
                         placeholderText: qsTr("Amount to receive") + translationManager.emptyString
                         fontBold: true
                         inlineIcon: true
+                        onTextChanged: {
+                            if (amountToReceiveLine.text.startsWith('.')) {
+                                amountToReceiveLine.text = '0' + amountToReceiveLine.text;
+                            }
+                        }
                         validator: RegExpValidator {
-                            regExp: /(\d{1,8})([.]\d{1,12})?$/
+                            regExp: /^(\d{1,8})?([\.]\d{1,12})?$/
                         }
                     }
 
@@ -496,14 +501,14 @@ Rectangle {
                         }
                     }
 
-                    LineEdit {
+                    LineEditMulti {
                         id: paymentUrl
                         Layout.fillWidth: true
                         labelText: qsTr("Payment URL") + translationManager.emptyString
-                        text: makeQRCodeString()
-                        onTextUpdated: function() { paymentUrl.cursorPosition = 0; }
+                        text: makeQRCodeString()                        
                         readOnly: true
                         copyButton: true
+                        wrapMode: Text.WrapAnywhere
                     }
                 }
             }