Fix dialogs having focus on the wrong element

pgallo725 · Nov 30, 2019 · bd08d7e · bd08d7e
1 parent 9b5afb1
commit bd08d7e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/textedit/NewDocumentWindow.cpp b/textedit/NewDocumentWindow.cpp
@@ -48,6 +48,9 @@ void NewDocumentWindow::resetFields()
 	//Reset all fields
 	ui->lineEdit_fileName->setText("");
 	ui->label_incorrectFilename->setText("");
+
+	//Reset the focus
+	ui->lineEdit_fileName->setFocus();
 }
 
 void NewDocumentWindow::acceptClicked()
@@ -64,7 +67,6 @@ void NewDocumentWindow::acceptClicked()
 
 void NewDocumentWindow::rejectClicked()
 {
-	//Reset fields and closes
-	resetFields();
+	//Close the window
 	this->close();
 }
diff --git a/textedit/OpenUriWindow.cpp b/textedit/OpenUriWindow.cpp
@@ -41,14 +41,17 @@ void OpenUriWindow::resetFields()
 	//Reset fields
 	ui->label_incorrectUri->setText("");
 	ui->lineEdit_uri->setText("");
+
+	//Reset focus
+	ui->lineEdit_uri->setFocus();
 }
 
 void OpenUriWindow::acceptClicked()
 {
-	//Gets uri
+	//Gets URI inserted by user
 	_uri = ui->lineEdit_uri->text();
 
-	//If filename is valid it closes and confirm operation else print an error
+	//If filename is valid it closes and confirms operation else print an error
 	if (!_uri.isEmpty())
 		this->done(QDialog::Accepted);
 	else
@@ -57,7 +60,6 @@ void OpenUriWindow::acceptClicked()
 
 void OpenUriWindow::rejectClicked()
 {
-	//Reset fields and closes
-	resetFields();
+	//Close the window
 	this->close();
 }