Bug 39042 - Improve rendering of button on app detail page
Improve rendering of button on app detail page
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 4.0
Other Linux
: P5 normal (vote)
: UCS 4.0-2-errata
Assigned To: Alexander Kläser
Dirk Wiesenthal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-07-29 20:43 CEST by Alexander Kläser
Modified: 2015-08-06 17:53 CEST (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
Sample screenshots of new button arrangement. (86.48 KB, image/png)
2015-07-29 20:49 CEST, Alexander Kläser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2015-07-29 20:43:12 CEST
In our usability tests, we found that the tester would oversee the "open" button in an app module easily. It would be helpful to rearrange the buttons in order to better recognize their meaning.
Comment 1 Alexander Kläser univentionstaff 2015-07-29 20:49:09 CEST
Created attachment 7070 [details]
Sample screenshots of new button arrangement.

Fixed. See the screenshot of some sample screenshots attached.

2015-05-28-univention-management-console-module-appcenter.yaml:
r62506 | Bug #39042: added YAML entry

univention-management-console-module-appcenter (4.1.20-40):
r62505 | Bug #39042: improve the display of buttons on an app detail page
Comment 2 Alexander Kläser univentionstaff 2015-07-29 20:57:42 CEST
Merge into 4.1-0 branch.

univention-management-console-module-appcenter (5.0.0-1):
r62507 | Bug #39042: improve the display of buttons on an app detail page
Comment 3 Alexander Kläser univentionstaff 2015-07-29 20:59:45 CEST
@Dirk: Can you do the QA for this bug?
Comment 4 Alexander Kläser univentionstaff 2015-07-29 21:01:15 CEST
BTW, I also replace the (x) button with a (←) button if an app is opened.
Comment 5 Dirk Wiesenthal univentionstaff 2015-08-03 00:33:40 CEST
OK, looks fine.

Minor issues:
 * "Open" label is now webinterface ? "Open website" : "Open module"
   but code is: module ? module.open() : website.open()
   see nagios or agorum
 * Could you use this new "Open" label for the "Installations thoughout the domain", too? Maybe put the code into App.js?
 * (super minor) Currently no problems, but I would like to note that umcAppButtonFirstRow does not work as expected if you do not group those buttons:
    buttons: [buttonWithFirstRow1, buttonWithoutFirstRow1, buttonWithFirstRow2] will not render the last button on the first row.
   As this is no problem right now and more buttons are unlikely to be added that would break the layout, this can be ignored.


(In reply to Alexander Kläser from comment #4)
> BTW, I also replace the (x) button with a (←) button if an app is opened.

Nice!
Comment 6 Alexander Kläser univentionstaff 2015-08-05 09:07:59 CEST
(In reply to Dirk Wiesenthal from comment #5)
> OK, looks fine.
> 
> Minor issues:
>  * "Open" label is now webinterface ? "Open website" : "Open module"
>    but code is: module ? module.open() : website.open()
>    see nagios or agorum
>  * Could you use this new "Open" label for the "Installations thoughout the
> domain", too? Maybe put the code into App.js?

Jep, good hint, I adjusted this behaviour. Nagios app opens what is written on the button label now. The code has been put into App.js (getOpenLabel()).

>  * (super minor) Currently no problems, but I would like to note that
> umcAppButtonFirstRow does not work as expected if you do not group those
> buttons:
>     buttons: [buttonWithFirstRow1, buttonWithoutFirstRow1,
> buttonWithFirstRow2] will not render the last button on the first row.
>    As this is no problem right now and more buttons are unlikely to be added
> that would break the layout, this can be ignored.

Ok. I will ignore this ;) .

Packages are building, changes have been merged to 4.1-0. YAML file has not been updated-

univention-management-console-module-appcenter (4.1.20-46):
r62650 | Bug #39042: adjusted button labeling

univention-management-console-module-appcenter (5.0.1-1):
r62651 | Bug #39042: adjusted button labeling
Comment 7 Alexander Kläser univentionstaff 2015-08-05 09:11:43 CEST
(In reply to Alexander Kläser from comment #6)
> Ok. I will ignore this ;) .
> 
> Packages are building, changes have been merged to 4.1-0. YAML file has not
> been updated-

Done.

2015-05-28-univention-management-console-module-appcenter.yaml:
r62652 | Bug #39042: adapted package version
Comment 8 Dirk Wiesenthal univentionstaff 2015-08-05 12:22:02 CEST
Code: OK
YAML: OK
Comment 9 Janek Walkenhorst univentionstaff 2015-08-06 17:53:50 CEST
<http://errata.univention.de/ucs/4.0/267.html>