The New Activity Password and Activity Password dialogs currently do not comply with the corresponding HIG (http://community.kde.org/Plasma/Active/Development/ActiveHIG/Dialogs) in several ways and thus should be changed: 1. The buttons (OK/Cancel and Unlock/Dismiss) are still placed at the bottom of the dialog. They have to be moved to the dialog title (as in the Activity Configuration dialog) 2. The button labels do not comply with the HIG and are inconsistent. To make them compliant, a) The "Dismiss" button in the Activity Password dialog has to be renamed to "Cancel" b) The "OK" button in the "New Activity Password" dialog has to be renamed to "Save" 3. The dialog titles do not describe actions. They should be changed to: a) "Unlock Activity" instead of "Activity Password" b) "Set Activity Password" instead of "New Activity Password" With these changes, the dialogs would be HIG compliant, more consistent with other dialogs and with user expectations. Reproducible: Always Steps to Reproduce: 1. Set an Activity to private 2. Tap "Save and Enter Password" or 1. Open a private Activity Actual Results: Dialogs inconsistent with HIG and other dialogs appear Expected Results: Consitent dialogs appear
(In reply to comment #0) > 3. The dialog titles do not describe actions. They should be changed to: > a) "Unlock Activity" instead of "Activity Password" > b) "Set Activity Password" instead of "New Activity Password" Have we agreed in using capital words in title? Currently we use "New activity password", should we change it to "Set activity password" or "Set Activity Password"?
Git commit 93de229b3bebbc03544c8d68961c5352532626f8 by Lamarque V. Souza. Committed on 29/04/2012 at 21:05. Pushed by lvsouza into branch 'master'. Change translatable strings to comply to Plasma Active's HIG. Still needs to move the "Save" and "Cancel" buttons to the dialog's title bar. M +1 -1 service/ActivityManager.cpp M +1 -1 service/ui/plugins/declarativeui/package/contents/ui/NewPasswordDialog.qml M +2 -2 service/ui/plugins/declarativeui/package/contents/ui/uihandler.qml http://commits.kde.org/kactivities/93de229b3bebbc03544c8d68961c5352532626f8
Git commit 171d81c731a3385c11a40cbbd891e5a24780fd22 by Lamarque V. Souza. Committed on 29/04/2012 at 21:13. Pushed by lvsouza into branch 'master'. Use i18n() around translatable strings. M +12 -12 service/ui/plugins/declarativeui/package/contents/ui/uihandler.qml http://commits.kde.org/kactivities/171d81c731a3385c11a40cbbd891e5a24780fd22
> Have we agreed in using capital words in title? Currently we use "New > activity password", should we change it to "Set activity password" or "Set > Activity Password"? There was no official agreement. However if nobody objects, I'd like to set title capitalization as the standard for, well, titles.
I am more concerned about coerence. If we change that it should be done for all strings as quickly as possible.
(In reply to comment #5) > I am more concerned about coerence. If we change that it should be done for > all strings as quickly as possible. Definitely. The Edit Activity dialog is currently being adapted by Marco. For the Add Items dialog, I've opened Bug 299102. These are all dialogs I am aware of. If anyone finds another one, just file a bug.
Ah, talking about Wording I would like to propose to change the current "Add items" title in the add-resource dialog to "Add Content" as it sounds far more familiar and less technical.
If there are no objections about comments #6 and #7 I will do the changes until next week.
Git commit 21bdc611904d421c3e81fba302511f9d463912d2 by Lamarque V. Souza. Committed on 25/05/2012 at 19:15. Pushed by lvsouza into branch 'master'. Make activity password dialogs for private Activities use capital title names to comply with HIG. M +4 -4 service/ActivityManager.cpp M +1 -1 service/ui/plugins/declarativeui/package/contents/ui/NewPasswordDialog.qml http://commits.kde.org/kactivities/21bdc611904d421c3e81fba302511f9d463912d2
This change is postponed until when Private Activities are reintroduced.
Private Activities feature was removed