Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

Commit

Permalink
fix: hide option to Load automatically for encrypted profiles
Browse files Browse the repository at this point in the history
There have been numerous issues about it from confused users,
e.g. #2698, #2870, #3638. Clearly tooltip alone is not enough.

Could have been done long time ago in #2818, if not for naive belief
that tooltip will suffice.
  • Loading branch information
zetok committed Aug 20, 2016
1 parent 54350fd commit b5efc2b
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions src/widget/loginscreen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -191,16 +191,15 @@ void LoginScreen::onLoginUsernameSelected(const QString &name)
ui->loginPassword->show();
// there is no way to do autologin if profile is encrypted, and
// visible option confuses users into thinking that it is possible,
// thus disable it (and hope that users won't think that it's a bug)
ui->autoLoginCB->setEnabled(false);
ui->autoLoginCB->setToolTip(tr("Password protected profiles can't be automatically loaded."));
// thus hide it
ui->autoLoginCB->hide();
}
else
{
ui->loginPasswordLabel->hide();
ui->loginPassword->hide();
ui->autoLoginCB->setEnabled(true);
ui->autoLoginCB->setToolTip("");
ui->autoLoginCB->show();
ui->autoLoginCB->setToolTip(tr("Password protected profiles can't be automatically loaded."));
}
}

Expand Down

0 comments on commit b5efc2b

Please sign in to comment.