Bug 26274 - Progressbar für Standby-Mixin
Progressbar für Standby-Mixin
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P5 enhancement (vote)
: UCS 3.0-2
Assigned To: Andreas Büsching
Florian Best
: interim-1
Depends on: 26273
Blocks: 26374
  Show dependency treegraph
 
Reported: 2012-02-28 11:54 CET by Andreas Büsching
Modified: 2012-07-20 15:24 CEST (History)
3 users (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:
buesching: Patch_Available+


Attachments
ProgressInfo Widget (3.48 KB, patch)
2012-02-28 14:46 CET, Andreas Büsching
Details | Diff
Patch (975 bytes, patch)
2012-03-01 17:36 CET, Alexander Kläser
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Büsching univentionstaff 2012-02-28 11:54:35 CET
Diese Erweiterung muss für UCS übernommen werden.

+++ This bug was initially created as a clone of Bug #26273 +++

In mehreren Modulen wird ein Widget benötigt, dass einen Fortschrittsbalken mit
Informationen anzeigt, wenn Operationen durchgeführt werden.

Es gibt bereits ein Standby-Mixin dem ein Widget übergeben werden kann. Das
neue Widget sollte so gebaut werden, dass es in dem Standby-Mixin genutzt
werden kann.

<Titel>
|#####                 | 27%
<weitere Informationen>
Comment 1 Andreas Büsching univentionstaff 2012-02-28 14:46:31 CET
Created attachment 4234 [details]
ProgressInfo Widget
Comment 2 Alexander Kläser univentionstaff 2012-03-01 17:36:22 CET
Created attachment 4245 [details]
Patch

Einen weiteren Fix gab es noch für dieses Feature (siehe angehängten Patch).
Comment 3 Andreas Büsching univentionstaff 2012-03-14 16:42:00 CET
Das ist im Patch noch nicht enthalten:

Eine Sache noch: Es wäre sinnvoll, wenn "width: 400px" mit in die CSS-Datei
wandern würde, damit diese Größe einheitlich über die Module hinweg ist. Mit
"width" anstelle von "min-width" wird zu langer Text in die nächste Zeile
umgebrochen, das ist vielleicht besser, anstelle die Breite des Widgets zu
verändern.




Index: css/dijit/themes/umc/umc.css
===================================================================
--- css/dijit/themes/umc/umc.css        (Revision 31381)
+++ css/dijit/themes/umc/umc.css        (Arbeitskopie)
@@ -5212,6 +5212,7 @@
 
 .umcProgressInfo {
        background-color: #fff;
+       min-width: 400px;
 }
 
 .umc .umcMultiObjectSelectDetailDialog form {
Comment 4 Alexander Kläser univentionstaff 2012-04-11 12:35:40 CEST
Im Zuge von Bug viel auf, dass der Aufruf von destroyRecursive() in uninitialize() zu einigen Problemen führte. Bitte die folgende Anpassung mit übernehmen:

====================
--- umc/widgets/ProgressInfo.js (Revision 31967)
+++ umc/widgets/ProgressInfo.js (Arbeitskopie)
@@ -47,11 +47,6 @@
 
        current: 0,
 
-       uninitialize: function() {
-               this.inherited(arguments);
-               this.destroyRecursive();
-       },
-
        buildRendering: function() {
                this.inherited(arguments);
====================
Comment 5 Alexander Kläser univentionstaff 2012-04-11 12:36:37 CEST
(In reply to comment #4)
> Im Zuge von Bug viel auf, ...Bug 26373
Comment 6 Andreas Büsching univentionstaff 2012-04-13 10:02:03 CEST
Patch angewendet und ChangeLog-Eintrag hinzugefügt
Comment 7 Florian Best univentionstaff 2012-06-18 15:00:56 CEST
Das Progressinfo-widget ist vorhanden und funktionsfähig.

Die min-width Änderung ist enthalten.
Die uninitialize-Methode ist rausgenommen.

Changelog ist auch vorhanden.

univention-management-console-frontend 1.0.372-2
Comment 8 Stefan Gohmann univentionstaff 2012-07-20 15:24:26 CEST
UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".