AZEN-SGG/amnezia-client
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #460 from amnezia-vpn/bugfix/minor-ui-fixes

Browse source 
added an empty string check for the server name and user name change fields
This commit is contained in:
pokamest 2023-12-21 15:55:33 +00:00 • committed by GitHub
commit d14e8cdee4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
client/ui/qml/Pages2/PageSettingsServerInfo.qml
View file

@ -102,6 +102,7 @@ PageType {
headerText: qsTr("Server name")
textFieldText: name
textField.maximumLength: 30
checkEmptyText: true
}
BasicButtonType {
@ -110,10 +111,14 @@ PageType {
text: qsTr("Save")
onClicked: {
if (serverName.textFieldText === "") {
return
}
if (serverName.textFieldText !== name) {
name = serverName.textFieldText
serverNameEditDrawer.visible = false
}
serverNameEditDrawer.visible = false
}
}
}

14
client/ui/qml/Pages2/PageShare.qml
View file

@ -261,6 +261,7 @@ PageType {
headerText: qsTr("User name")
textFieldText: "New client"
textField.maximumLength: 20
checkEmptyText: true
}
@ -469,7 +470,9 @@ PageType {
imageSource: "qrc:/images/controls/share-2.svg"
onClicked: {
ExportController.generateConfig(root.connectionTypesModel[exportTypeSelector.currentIndex].type)
if (clientNameTextField.textFieldText !== "") {
ExportController.generateConfig(root.connectionTypesModel[exportTypeSelector.currentIndex].type)
}
}
}
@ -623,7 +626,8 @@ PageType {
Layout.fillWidth: true
headerText: qsTr("Client name")
textFieldText: clientName
textField.maximumLength: 30
textField.maximumLength: 20
checkEmptyText: true
}
BasicButtonType {
@ -632,6 +636,10 @@ PageType {
text: qsTr("Save")
onClicked: {
if (clientNameEditor.textFieldText === "") {
return
}
if (clientNameEditor.textFieldText !== clientName) {
PageController.showBusyIndicator(true)
ExportController.renameClient(index,
@ -660,7 +668,7 @@ PageType {
text: qsTr("Revoke")
onClicked: function() {
questionDrawer.headerText = qsTr("Revoke the config for a user - ") + clientName + "?"
questionDrawer.headerText = qsTr("Revoke the config for a user - %1?").arg(clientName)
questionDrawer.descriptionText = qsTr("The user will no longer be able to connect to your server.")
questionDrawer.yesButtonText = qsTr("Continue")
questionDrawer.noButtonText = qsTr("Cancel")