From 03c15af20eb0df4e1e575d01cd40a7c127304bec Mon Sep 17 00:00:00 2001 From: "moneromooo.monero" Date: Fri, 16 Dec 2016 00:27:16 +0000 Subject: [PATCH] Don't claim a password is "medium" too easily --- components/PrivacyLevelSmall.qml | 2 +- wizard/WizardPassword.qml | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/components/PrivacyLevelSmall.qml b/components/PrivacyLevelSmall.qml index cb1cd36e..47543f52 100644 --- a/components/PrivacyLevelSmall.qml +++ b/components/PrivacyLevelSmall.qml @@ -72,7 +72,7 @@ Item { width: row.x color: { - if(item.fillLevel < 3) return "#FF6C3C" + if(item.fillLevel < 5) return "#FF6C3C" if(item.fillLevel < 13) return "#FFE00A" return "#36B25C" } diff --git a/wizard/WizardPassword.qml b/wizard/WizardPassword.qml index 1e5013cf..80bd893d 100644 --- a/wizard/WizardPassword.qml +++ b/wizard/WizardPassword.qml @@ -76,8 +76,14 @@ Item { wizard.nextButton.enabled = passwordItem.password === retypePasswordItem.password - // scorePassword returns value from 1..100 + // scorePassword returns value from 0 to... lots var strength = walletManager.getPasswordStrength(passwordItem.password); + // consider anything below 10 bits as dire + strength -= 10 + if (strength < 0) + strength = 0 + // use a slight parabola to discourage short passwords + strength = strength ^ 1.2 / 3 // privacyLevel component uses 1..13 scale privacyLevel.fillLevel = Utils.mapScope(1, 100, 1, 13, strength)