Bug 22899 - UMC2: Software-Management
UMC2: Software-Management
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.0 - RC
Assigned To: Frank Greif
Andreas Büsching
: interim-2
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-06-28 08:42 CEST by Andreas Büsching
Modified: 2011-12-13 15:51 CET (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):
Max CVSS v3 score:


Attachments
Nacharbeiten am Design (5.12 KB, patch)
2011-11-17 15:01 CET, Frank Greif
Details | Diff
Patch nochmal neu gezogen (5.13 KB, patch)
2011-11-18 11:19 CET, Frank Greif
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Büsching univentionstaff 2011-06-28 08:42:16 CEST
Das Software-Management-Modul muss für die neue Generation von UMC angepasst werden
Comment 1 Andreas Büsching univentionstaff 2011-11-02 13:56:38 CET
Das Modul ist integriert und funktioniert grundsätzlich. Evtl. müssen hier noch kleinere Anpassungen gemacht werden:

- Suchmaske vereinheitlichen
- Kleinere Layout-Korrekturen
Comment 2 Alexander Kläser univentionstaff 2011-11-02 15:43:41 CET
Folgende kleinere Anpassungen sollten ebenfalls mit übernommen werden:

* Entfernen des Reset-Buttons (haben wir sonst jetzt auch gemacht)
* Suche-Button neben dem Feld "Suchbegriff" rechts anordnen
* Sektion → in Kategorien umbenennen (damit es einheitlich ist)
* "Package Management" → "Package management"
* "Installation Status" → "Instalation status"
* "Package Description" → "Package description"
Comment 3 Frank Greif 2011-11-03 11:27:29 CET
Mir ist noch aufgefallen, daß im Detailfeld für Komponenten die Felder 'password' und 'username' so gerendert werden, als seien es die betreffenden Felder eines Login-Forms...

Wenn ich bisher auf solche Spezialbehandlung anhand des Feldnamens gestoßen bin, habe ich das betreffende Feld im Dialog einfach anders genannt, aber das ist hier eher umständlich, da diese Feldnamen direkt an der UCR als Variablennamen verwendet werden.

Sollte also mal einer nachdenken, wie man das umgehen kann. Eigentlich sollten CSS-Styles, die nur im Login-Form sinnvoll sind, nirgendwo anders heimlich nachgetragen werden.
Comment 4 Alexander Kläser univentionstaff 2011-11-07 10:44:30 CET
(In reply to comment #3)
> Mir ist noch aufgefallen, daß im Detailfeld für Komponenten die Felder
> 'password' und 'username' so gerendert werden, als seien es die betreffenden
> Felder eines Login-Forms...
> 
> Wenn ich bisher auf solche Spezialbehandlung anhand des Feldnamens gestoßen
> bin, habe ich das betreffende Feld im Dialog einfach anders genannt, aber das
> ist hier eher umständlich, da diese Feldnamen direkt an der UCR als
> Variablennamen verwendet werden.
> 
> Sollte also mal einer nachdenken, wie man das umgehen kann. Eigentlich sollten
> CSS-Styles, die nur im Login-Form sinnvoll sind, nirgendwo anders heimlich
> nachgetragen werden.

Das gehört sicherlich zu Bug 22898. Das ist nicht ganz klar, password und username werden durch das Framework nicht gesondert behandelt.
Comment 5 Stefan Gohmann univentionstaff 2011-11-07 11:07:22 CET
Rückmeldung vom Test:

> - Ähnlich wie in UDM sollte die Schaltfläche zum Suchen nach rechts (dort
> ist noch Platz), die Schaltfläche für "Zurücksetzen" kann ganz weg. Dann
> gewinnen wir Platz für die Ergebnisliste.
Comment 6 Frank Greif 2011-11-07 12:19:58 CET
(In reply to comment #5)
Bitte nicht vergessen, daß sich umc.widgets.SearchForm dann auch irgendwann entsprechend diesen Guidelines verhalten sollte. Momentan wird der automatisch addierte Search Button immer noch auf eine neu angelegte Layout-Zeile plaziert.
Comment 7 Frank Greif 2011-11-10 11:21:27 CET
(In reply to comment #4)
> Mir ist noch aufgefallen, daß im Detailfeld für Komponenten die Felder
> 'password' und 'username' so gerendert werden, als seien es die betreffenden
> Felder eines Login-Forms...

Habe festgestellt, daß es eine Browser-Eigenschaft von Opera ist, daß es für alle Felder eines Formulars, die automatisch ausgefüllt werden können, einen zusätzlichen Style anwendet. Also bitte nicht tragisch nehmen.
Comment 8 Sönke Schwardt-Krummrich univentionstaff 2011-11-10 11:26:04 CET
Auch wenn der Bug noch nicht zu ist: mir ist gerade aufgefallen, dass nur installierte Pakete angezeigt werden (auch wenn die Checkbox deaktiviert ist).
Comment 9 Frank Greif 2011-11-10 12:03:06 CET
(In reply to comment #8)
Möglicherweise ein Problem, welchen Wert eine Checkbox zeigt und welchen Wert sie wirklich hat. Sollte noch einmal geprüft werden, nachdem #24409 behoben ist.
Comment 10 Sönke Schwardt-Krummrich univentionstaff 2011-11-10 12:08:12 CET
(In reply to comment #9)
> (In reply to comment #8)
> Möglicherweise ein Problem, welchen Wert eine Checkbox zeigt und welchen Wert
> sie wirklich hat. Sollte noch einmal geprüft werden, nachdem #24409 behoben
> ist.

Sobald man einmal mit der Maus über die Checkbox führt, zeigt sie den "richtigen" Zustand an, der dann auch per Klick umgeschaltet werden kann. Die Checkbox behält dann auch den ausgewählten Zustand.
Comment 11 Alexander Kläser univentionstaff 2011-11-11 11:28:44 CET
(In reply to comment #10)
> Sobald man einmal mit der Maus über die Checkbox führt, zeigt sie den
> "richtigen" Zustand an, der dann auch per Klick umgeschaltet werden kann. Die
> Checkbox behält dann auch den ausgewählten Zustand.

→ Diese Fehlverhalten wird in Bug 24521 behandelt.
Comment 12 Frank Greif 2011-11-17 15:01:35 CET
Created attachment 3818 [details]
Nacharbeiten am Design

Patch behebt die angesprochenen Mängel in der Gestaltung:
* Beschriftungen bzw. Übersetzungen korrigiert
* Reset-Button entsorgt
* Submit-Button im Suchformular nach rechts verschoben.

Das Problem, daß die Ergebnisliste nicht mit dem Wert der Checkbox "nur installierte Pakete anzeigen" korrespondiert, habe ich nicht zu Gesicht bekommen, weder nach dem Aufklappen des Moduls noch nach der Rückkehr aus einer Installer-Aktion. Bitte noch einmal prüfen.
Comment 13 Andreas Büsching univentionstaff 2011-11-18 10:52:17 CET
(In reply to comment #12)
> Created an attachment (id=3818) [details]
> Nacharbeiten am Design

@Frank: Der Patch passt leider nicht. Kannst du den noch einmal anpassen, dass er auf die aktuelle Version passt?

faktum > patch --dry-run -p0 -i design.patch 
patching file umc/de.po
Hunk #1 FAILED at 15.
1 out of 1 hunk FAILED -- saving rejects to file umc/de.po.rej
patching file umc/js/de.po
Hunk #1 FAILED at 33.
Hunk #2 FAILED at 50.
Hunk #3 FAILED at 75.
Hunk #4 FAILED at 84.
Hunk #5 FAILED at 103.
Hunk #6 FAILED at 142.
Hunk #7 FAILED at 155.
Hunk #8 FAILED at 164.
Hunk #9 FAILED at 192.
9 out of 9 hunks FAILED -- saving rejects to file umc/js/de.po.rej
can't find file to patch at input line 126
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|Index: umc/js/packages.js
|===================================================================
|--- umc/js/packages.js (revision 5030)
|+++ umc/js/packages.js (working copy)
--------------------------
File to patch: ^C
Comment 14 Frank Greif 2011-11-18 11:19:25 CET
Created attachment 3823 [details]
Patch nochmal neu gezogen

Patch setzt jetzt auf rev. 5075 auf.
Comment 15 Andreas Büsching univentionstaff 2011-11-18 13:18:34 CET
(In reply to comment #14)
> Created an attachment (id=3823) [details]
> Patch nochmal neu gezogen
> 
> Patch setzt jetzt auf rev. 5075 auf.

Danke. Der Patch passt.
Comment 16 Andreas Büsching univentionstaff 2011-11-29 16:19:35 CET
ich habe Pakete gesucht:

- aus verschiedenen kategorien
- nur installierte
- unterschiedliche Suchmuster


Pakete installieren, deinstallieren und Details anzeigen funktioniert auch.
Comment 17 Sönke Schwardt-Krummrich univentionstaff 2011-12-13 15:51:03 CET
UCS 3.0-0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer
neueren Version von UCS erneut auftreten, so sollte dieser Bug dupliziert
werden: "Clone This Bug"