Bug 53366 - Adjust ucs-school-umc-computerroom UMC module for UCS 5 Design
Adjust ucs-school-umc-computerroom UMC module for UCS 5 Design
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: General
UCS@school 5.0
Other Linux
: P5 normal (vote)
: UCS@school 5.0 v1
Assigned To: Johannes Keiser
Florian Best
: interim-4
Depends on: 53779
Blocks:
  Show dependency treegraph
 
Reported: 2021-06-02 10:23 CEST by Florian Best
Modified: 2021-11-29 17:19 CET (History)
3 users (show)

See Also:
What kind of report is it?: Development Internal
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):
Max CVSS v3 score:


Attachments
Screenshot (181.78 KB, image/png)
2021-07-12 18:28 CEST, Florian Best
Details
Screenshot (52.92 KB, image/png)
2021-07-12 18:29 CEST, Florian Best
Details
Screenshot (70.62 KB, image/png)
2021-07-12 18:30 CEST, Florian Best
Details
Screenshot (177.74 KB, image/png)
2021-07-12 18:32 CEST, Florian Best
Details
Screenshot (62.68 KB, image/png)
2021-07-12 18:34 CEST, Florian Best
Details
Info box does not have the same information as before (37.85 KB, image/png)
2021-08-18 10:32 CEST, Christian Castens
Details
old naming convention "slave" (67.90 KB, image/png)
2021-08-26 16:28 CEST, Christian Castens
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2021-06-02 10:23:46 CEST
The package ucs-school-umc-computerroom needs to be adjusted for the new UCS 5.0 design.
Comment 1 Florian Best univentionstaff 2021-06-14 20:32:54 CEST
Additionally an icon for the module has to be added.
Comment 3 Johannes Keiser univentionstaff 2021-07-12 16:35:14 CEST
e91adbcd1 Bug #53366: add icons
36afdc03e Bug #53366: escape message
df2cabb2b Bug #53366: rename css class
2b9978388 Bug #53366: debian changelog
132ac3abc Bug #53366: adjust ucs-school-umc-computerroom for UCS 5 design
Comment 4 Florian Best univentionstaff 2021-07-12 18:28:39 CEST
Created attachment 10776 [details]
Screenshot

REOPEN: There is not border/container around the screenshot view.
Comment 5 Florian Best univentionstaff 2021-07-12 18:29:38 CEST
Created attachment 10777 [details]
Screenshot

REOPEN: The buttons are overlapping in presentation mode.
Comment 6 Florian Best univentionstaff 2021-07-12 18:30:35 CEST
Created attachment 10778 [details]
Screenshot

When clicking on the context menu then, the Beobachten button gets moved.
Comment 7 Florian Best univentionstaff 2021-07-12 18:32:37 CEST
Created attachment 10779 [details]
Screenshot

REOPEN: The screenshots used to be large - now the "large" screenshot is only 1/3 of the width.
Comment 8 Florian Best univentionstaff 2021-07-12 18:34:07 CEST
Created attachment 10780 [details]
Screenshot

The default alternative image looks old school.
Comment 9 Johannes Keiser univentionstaff 2021-07-15 16:53:47 CEST
(In reply to Florian Best from comment #5)
> Created attachment 10777 [details]
> Screenshot
> 
> REOPEN: The buttons are overlapping in presentation mode.
Fixed

(In reply to Florian Best from comment #6)
> Created attachment 10778 [details]
> Screenshot
> 
> When clicking on the context menu then, the Beobachten button gets moved.
Fixed


(In reply to Florian Best from comment #7)
> Created attachment 10779 [details]
> Screenshot
> 
> REOPEN: The screenshots used to be large - now the "large" screenshot is
> only 1/3 of the width.
Adjusted design

(In reply to Florian Best from comment #4)
> Created attachment 10776 [details]
> Screenshot
> 
> REOPEN: There is not border/container around the screenshot view.
Adjusted design

(In reply to Florian Best from comment #8)
> Created attachment 10780 [details]
> Screenshot
> 
> The default alternative image looks old school.
Adjusted design
Comment 11 Christian Castens univentionstaff 2021-08-18 10:32:35 CEST
Created attachment 10802 [details]
Info box does not have the same information as before

When screenshots of a computer cannot be shown due to missing permissions of the current user (e.g. Administrator) there used to be an information text that can be seen on the previous screenshot. That info text is currently missing as seen on the latest screenshot.
Comment 12 Christian Castens univentionstaff 2021-08-26 16:28:23 CEST
Created attachment 10811 [details]
old naming convention "slave"

old naming convention ("slave") can still be found when creating a School
Comment 13 Florian Best univentionstaff 2021-08-26 16:36:11 CEST
(In reply to Christian Castens from comment #12)
> Created attachment 10811 [details]
> old naming convention "slave"
> 
> old naming convention ("slave") can still be found when creating a School

This is not yet adjusted, will be done in Bug #52602.
Comment 14 Dirk Wiesenthal univentionstaff 2021-08-31 09:32:12 CEST
All issues have been resolved, the module looks UCS 5-y again.

As pointed out in Comment 11, the text about a logged in Administrator no longer appears, but instead the default "No screenshot can be shown, try again later" is shown.

I did not find the place where they used the original message in 4.4. So I cannot say why this is not done anymore. But anyway, I think it is good enough as is.
Comment 17 Johannes Keiser univentionstaff 2021-09-09 10:54:19 CEST
fixed with 7349ec044daa87149b37eb400b0f3fa8774032d9
Comment 18 Florian Best univentionstaff 2021-09-09 18:52:06 CEST
(In reply to Christian Castens from comment #11)
> Created attachment 10802 [details]
> Info box does not have the same information as before
> 
> When screenshots of a computer cannot be shown due to missing permissions of
> the current user (e.g. Administrator) there used to be an information text
> that can be seen on the previous screenshot. That info text is currently
> missing as seen on the latest screenshot.

REOPEN: I am still seeing this issue.
Maybe the SVG (or rendering of SVG) is broken?
Comment 19 Johannes Keiser univentionstaff 2021-09-13 14:09:12 CEST
305901593 Bug #53366: make svg cover while width
1afb8933e fixup! Bug #53366: ucs-school-umc-computerroom adjust screenshot view
7e869999d Bug #53665: set correct mimetype for screenshot and set initial screenshot
Comment 20 Florian Best univentionstaff 2021-09-13 14:58:51 CEST
OK: SVG's are correctly shown and scaled.
Comment 21 Jürn Brodersen univentionstaff 2021-11-29 17:19:38 CET
UCS@school 5.0 v1 has been released.

https://docs.software-univention.de/release-notes-ucsschool-5.0v1-de.html

If this error occurs again, please clone this bug.