From 4d3ca2fdc5b74cb94f521a8ca410204fad1d0acd Mon Sep 17 00:00:00 2001
From: kasenvr <52365539+kasenvr@users.noreply.github.com>
Date: Wed, 5 Aug 2020 02:31:10 -0400
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: David Rowe <david@ctrlaltstudio.com>
---
 domain-server/resources/describe-settings.json          | 4 ++--
 interface/resources/qml/LoginDialog/LinkAccountBody.qml | 4 ++--
 interface/src/Menu.cpp                                  | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/domain-server/resources/describe-settings.json b/domain-server/resources/describe-settings.json
index d658e5d8ce..9c220b740e 100644
--- a/domain-server/resources/describe-settings.json
+++ b/domain-server/resources/describe-settings.json
@@ -414,7 +414,7 @@
           "name": "group_permissions",
           "type": "table",
           "caption": "Permissions for Users in Groups",
-          "help": "For groups that are supplied through OAuth, you will need to denote them by putting an \"@\" symbol in front of each item. e.g. \"@silver\"",
+          "help": "For groups that are provided from WordPress you need to denote them by putting an \"@\" symbol in front of each item, e.g., \"@silver\"".
           "categorize_by_key": "permissions_id",
           "can_add_new_categories": true,
           "can_add_new_rows": false,
@@ -543,7 +543,7 @@
           "name": "group_forbiddens",
           "type": "table",
           "caption": "Permissions Denied to Users in Groups",
-          "help": "For groups that are supplied through OAuth, you will need to denote them by putting an \"@\" symbol in front of each item. e.g. \"@silver\"",
+          "help": "For groups that are provided from WordPress you need to denote them by putting an \"@\" symbol in front of each item, e.g., \"@silver\"".
           "categorize_by_key": "permissions_id",
           "can_add_new_categories": true,
           "can_add_new_rows": false,
diff --git a/interface/resources/qml/LoginDialog/LinkAccountBody.qml b/interface/resources/qml/LoginDialog/LinkAccountBody.qml
index bba46e78c5..04e980f0bb 100644
--- a/interface/resources/qml/LoginDialog/LinkAccountBody.qml
+++ b/interface/resources/qml/LoginDialog/LinkAccountBody.qml
@@ -189,7 +189,7 @@ Item {
                 anchors {
                     top: parent.top
                     left: parent.left
-                    right: parent.right;
+                    right: parent.right
                     topMargin: 1.5 * hifi.dimensions.contentSpacing.y
                     // horizontalCenter: mainContainer.horizontalCenter
                 }
@@ -595,7 +595,7 @@ Item {
                      leftMargin: hifi.dimensions.contentSpacing.x
                 }
 
-                text: "<a href='http://metaverse.vircadia.com/users/register'>Sign Up</a>"
+                text: "<a href='https://metaverse.vircadia.com/users/register'>Sign Up</a>"
 
                 linkColor: hifi.colors.blueAccent
                 onLinkActivated: {
diff --git a/interface/src/Menu.cpp b/interface/src/Menu.cpp
index f40082c1e7..64cdf98239 100644
--- a/interface/src/Menu.cpp
+++ b/interface/src/Menu.cpp
@@ -10,7 +10,7 @@
 //
 
 // For happ(ier) development of QML, use these two things:
-// This forces QML files to be pulled from the source as you edit it: HIFI_USE_SOURCE_TREE_RESOURCES=1
+// This forces QML files to be pulled from the source as you edit it: set environment variable HIFI_USE_SOURCE_TREE_RESOURCES=1
 // Use this to live reload: DependencyManager::get<OffscreenUi>()->clearCache();
 
 #include "Menu.h"