Bug 36789 - UCS4 design incomplete in UCS@school modules
UCS4 design incomplete in UCS@school modules
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC
UCS@school 4.0
Other Linux
: P5 normal (vote)
: UCS@school 4.2 v1
Assigned To: Johannes Keiser
Florian Best
:
: 35057 43028 (view as bug list)
Depends on:
Blocks: 44079 44089
  Show dependency treegraph
 
Reported: 2014-11-18 09:40 CET by Sönke Schwardt-Krummrich
Modified: 2017-04-11 17:45 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.057
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Design, Usability
Max CVSS v3 score:


Attachments
Patches for all relevant modules. (2.94 KB, application/gzip)
2016-10-19 16:17 CEST, Richard Ulmer
Details
Patches for all relevant modules. (3.04 KB, application/gzip)
2016-11-07 10:16 CET, Richard Ulmer
Details
Screenshot of "Users (schools)" (36.29 KB, image/png)
2017-03-10 16:58 CET, Sönke Schwardt-Krummrich
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2014-11-18 09:40:52 CET
There are several module where the UCS4 design in incomplete, e.g. the module for assigning teachers to classes has no save button at the top right, but a  "Änderungen speichern" button at the lower right, which is outside of the screen so the user has to scoll down to be able to click it.
Comment 1 Sönke Schwardt-Krummrich univentionstaff 2014-11-18 09:41:17 CET
All other modules have to be checked too.
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2016-10-13 11:01:25 CEST
Please do not make any changes directly in ucs-school-4.1r2 but attach a patch to this bug, that contains all neccessary changes.
Comment 3 Florian Best univentionstaff 2016-10-13 11:32:00 CEST
Hm shouldn't we wait for UCS 4.2 here as there might be several more changes?!
Comment 4 Richard Ulmer univentionstaff 2016-10-19 16:17:59 CEST
Created attachment 8136 [details]
Patches for all relevant modules.
Comment 5 Richard Ulmer univentionstaff 2016-10-19 16:18:32 CEST
I have made the changes after talking to Florian and Alex.
Comment 6 Florian Best univentionstaff 2016-10-20 18:56:42 CEST
(In reply to Richard Ulmer from comment #5)
> I have made the changes after talking to Florian and Alex.
You didn't commit anything, right? Setting the Bug back to Assigned.
Comment 7 Florian Best univentionstaff 2016-11-07 06:06:27 CET
The "save" button for the distribution module is not called "Create project" anymore when adding a project. I am not sure if we should keep the label. The label is only displayed when hovering the save icon.

Why did you remove the height of the helpdesk text area field?
Comment 8 Richard Ulmer univentionstaff 2016-11-07 10:14:56 CET
The "save" button for the distribution module is now called "Create project", when creating a new project.

I have removed the height, because it lead to problems when trying to extend the TextArea by dragging the lower-right-corner downwards. Trying to use the "rows" attribute of dojo did not work - I've added a comment to the code mentioning this. The design of the helpdesk module has been refined further with the help of Florian.
Comment 9 Richard Ulmer univentionstaff 2016-11-07 10:16:08 CET
Created attachment 8193 [details]
Patches for all relevant modules.
Comment 10 Florian Best univentionstaff 2016-11-16 13:26:38 CET
*** Bug 35057 has been marked as a duplicate of this bug. ***
Comment 11 Florian Best univentionstaff 2016-11-21 17:43:49 CET
*** Bug 43028 has been marked as a duplicate of this bug. ***
Comment 12 Sönke Schwardt-Krummrich univentionstaff 2016-11-22 10:32:39 CET
Slightly raised pain value due to some customer feedback.
Comment 13 Florian Best univentionstaff 2017-01-09 15:19:53 CET
I think it is better for UCS@school 4.2 so that teachers don't get confused :P
Comment 14 Florian Best univentionstaff 2017-02-17 17:18:01 CET
@Richard: you can apply the changes now in the UCS@school 4.2 branch!
Comment 15 Florian Best univentionstaff 2017-02-18 05:21:47 CET
Thanks for the patches! I applied them:

ucs-school-umc-installer (5.0.0-2):
r76845 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-wizards (9.0.1-1):
r76846 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-helpdesk (13.0.0-2):
r76841 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-internetrules (13.0.0-2):
r76842 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-exam (7.0.1-1):
r76839 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-computerroom (9.0.5-1):
r76836 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-distribution (15.0.1-1):
r76838 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-rooms (14.0.0-2):
r76844 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-csv-import (4.0.1-1):
r76837 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-groups (7.0.0-2):
r76840 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard

ucs-school-umc-lessontimes (5.0.0-2):
r76843 | Bug #36789: adjust header buttons to new UMC design; applied patch from Richard
Comment 16 Florian Best univentionstaff 2017-02-18 05:46:09 CET
This looks great!

I see only two issues:
The UCS@school installation wizard has 2 close buttons, if the system is already configured.

The save button of the school wizards are on the bottom and not in the top.
Comment 17 Florian Best univentionstaff 2017-03-09 11:12:04 CET
The "Advanced settings" button of the school wizards should be shown as headerButton.
Comment 18 Richard Ulmer univentionstaff 2017-03-09 11:16:28 CET
The UCS@school installation wizard now has one close button, if the system is already configured.

svn commit: 77498
build: 5.0.0-5A~4.2.0.201703091113
Comment 19 Richard Ulmer univentionstaff 2017-03-09 17:39:05 CET
I've put the "Advanced settings"-button into the header.

svn commit: 77538
build: 9.0.2-3A~4.2.0.201703091736
Comment 20 Sönke Schwardt-Krummrich univentionstaff 2017-03-10 16:58:00 CET
Created attachment 8517 [details]
Screenshot of "Users (schools)"

There is an additional column whose name is not readable.
a) what column is it?
b) please fix it
Comment 21 Florian Best univentionstaff 2017-03-10 17:06:53 CET
(In reply to Sönke Schwardt-Krummrich from comment #20)
> Created attachment 8517 [details]
> Screenshot of "Users (schools)"
> 
> There is an additional column whose name is not readable.
> a) what column is it?
> b) please fix it

It was a column used as workaround for the old grid to add some left padding, I guess. I removed this.

ucs-school-umc-wizards (9.0.3-1):
r77602 | Bug #36789: removed leftover from old grid
Comment 22 Alexander Kläser univentionstaff 2017-03-23 07:52:23 CET
FYI, I adjusted the code of umc/widgets/Page such that one can specify fullWidth=true in order to make sure that the content area takes the full width of the page. The navigation area will then be positioned above the content (as for smaller screen sizes). This might be helpful for some UCS@school modules, as well.
Comment 23 Florian Best univentionstaff 2017-03-27 16:12:34 CEST
The rest will be done at Bug #44089.
Comment 24 Florian Best univentionstaff 2017-03-27 16:59:55 CEST
We also need to adapt the search input fields and add fullWidth to the search pages.
Comment 25 Johannes Keiser univentionstaff 2017-03-27 20:24:29 CEST
r 78382
ucs-school-umc-distribution (15.0.1-2) 
* Bug #36789: adapt search form to new UMC design

r 78384
ucs-school-umc-groups (7.0.0-3) 
* Bug #36789: adapt search form to new UMC design

r 78385
ucs-school-umc-printermoderation (14.0.3-2) 
* Bug #36789: adapt search form to new UMC design

r 78386
ucs-school-umc-rooms (14.0.0-3) 
* Bug #36789: adapt search form to new UMC design

r 78387
ucs-school-umc-users (13.0.0-2) 
* Bug #36789: adapt search form to new UMC design

r 78388
ucs-school-umc-wizards (9.0.3-2) 
* Bug #36789: adapt search form to new UMC design
Comment 26 Johannes Keiser univentionstaff 2017-03-28 11:28:39 CEST
r 78392
r 78393
ucs-school-umc-internetrules (13.0.0-3) 
* Bug #36789: adapt search form to new UMC design
Comment 27 Florian Best univentionstaff 2017-03-28 17:24:11 CEST
There is a "Change school" button below the grid of the "Users (schools)" wizard. As the grid has now unlimited scrolling this needs to be moved to the top module buttons.
Comment 28 Johannes Keiser univentionstaff 2017-03-28 17:49:55 CEST
(In reply to Florian Best from comment #27)
> There is a "Change school" button below the grid of the "Users (schools)"
> wizard. As the grid has now unlimited scrolling this needs to be moved to
> the top module buttons.

r 78438
ucs-school-umc-wizards (9.0.3-3) 
* Bug #36789: Move 'Change school' button in Users (schools) module
into module header
Comment 29 Florian Best univentionstaff 2017-03-28 18:00:44 CEST
brilliant!
Comment 30 Sönke Schwardt-Krummrich univentionstaff 2017-04-11 17:45:41 CEST
UCS@school 4.2 v1 has been released.

http://docs.software-univention.de/release-notes-ucsschool-4.2v1-de.html

If this error occurs again, please clone this bug.