Allows synergy with password managers that rely on the window title in order to supply the appropriate password.
Details
Details
- Reviewers
davidedmundson ivan ngraham - Group Reviewers
Plasma VDG - Commits
- R845:6944aab19e86: Include the Plasma Vault name in the mount dialog title
Compiles and runs as expected both on master and Plasma/5.18 branches.
Diff Detail
Diff Detail
- Repository
- R845 Plasma Vault
- Branch
- password-manager (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 25413 Build 25431: arc lint + arc unit
kded/ui/mountdialog.cpp | ||
---|---|---|
34 | Maybe put the vault name in single quotes? |
Comment Actions
Can you provide your email address in a comment here so that we can land the patch with the correct git authorship information?
Comment Actions
Go ahead, although we're migrating patch review to GitLab in the coming weeks, so the benefit may be short-lived. :)
kded/ui/mountdialog.cpp | ||
---|---|---|
34 | Agreed, I had single quotes initially, but wasn't sure if it was proper. |
Comment Actions
Landed on master since adding or changing strings in the stable branch isn't allowed.
Thanks for the contribution!