Bug 26826 - [UMC] CSS und images aufräumen
[UMC] CSS und images aufräumen
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P2 normal (vote)
: UCS 3.1
Assigned To: Jascha Geerds
Dirk Wiesenthal
: interim-1
: 27837 (view as bug list)
Depends on: 27837
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-18 13:09 CEST by Florian Best
Modified: 2012-12-12 21:08 CET (History)
2 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
cssdiff.py (3.14 KB, text/x-python)
2012-09-03 11:31 CEST, Florian Best
Details
Screenshot UDM (17.92 KB, image/png)
2012-09-13 15:47 CEST, Alexander Kläser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2012-04-18 13:09:07 CEST
Die CSS Dateien des UMC-frontends sollten überarbeitet werden:
Dabei ist besonders aufgefallen css/dojox/grid/enhanced/resources/umc/EnhancedGrid.css:
sehr viele Duplikationen, unnütze Blöcke
dinge aus umc.css kopiert
(24 Anpassungen für Internet Explorer 6)
Ich würde ebenfalls den Pfad der Datei ändern.

Im Moment wird die Abtrennung zwischen Benutzername und Hostname mit einem Bild gemacht, anstatt einem css-border (der header bereich ist eh komisch, siehe auch Bug #26264).

Außerdem sollten die images Ordner aufgeräumt werden, da befinden sich teilweise Sachen aus 2.4 drin, zudem ist die Ordnung nicht so toll strukturiert (4 Ordner für images) und mehrere leere Ordner.

Dazu kommen Duplikate: z.B. images/top-link.png, images/bg/top-link.png sowie images/button.png und css/dijit/themes/umc/form/images/button.png
Comment 1 Alexander Kläser univentionstaff 2012-04-18 15:10:34 CEST
Folgende Dinge könnten dabei auch berücksichtigt werden:

* Die umc.css-Datei ist basiert derzeit auf der "gebauten" (d.h. die Kommentare wurden herausgefiltert) claro.css-Datei. Ideal wäre es, wenn die CSS-Dateien des UMC-Themes auf die ursprünglichen Dojo-Dateien verweisen (via @import) und nur unsere spezifischen Regeländerungen mitbringen. Wichtig dafür wäre es den Bauvorgang in Bezug auf die CSS-Dateien zu berücksichtigen, da die CSS-Dateien auch komprimiert werden.

* Ggf. kann eine eigene color.css-Datei erstellt werden, die es für dritte einfach macht, das Branding der UMC anzupassen
Comment 2 Alexander Kläser univentionstaff 2012-08-01 11:39:51 CEST
Dies würde uns für zukünftige Dojo-Updates viel Zeit ersparen, da wir momentan nur schwer Änderung an den CSS-Daten mitbekommen.
Comment 3 Dirk Wiesenthal univentionstaff 2012-08-01 15:06:55 CEST
Wenn wir schon mal dabei sind könnte man mal nachforschen, was aus so ein paar Dateien geworden ist, die jetzt ständig 404 werfen und entweder mit hinein nehmen, oder die css-klasse anpassen.

Bug 27837
Comment 4 Florian Best univentionstaff 2012-08-07 11:43:31 CEST
Bug #25234 könnte duplikat sein bzw. hier gemacht werden…
Comment 5 Alexander Kläser univentionstaff 2012-08-07 13:12:16 CEST
*** Bug 27837 has been marked as a duplicate of this bug. ***
Comment 6 Florian Best univentionstaff 2012-09-03 11:31:40 CEST
Created attachment 4633 [details]
cssdiff.py

Im Anhang ein Script, dass 2 CSS Dateien durch parsen aller nodes vergleicht und eine neue CSS Datei mit den Änderungen erstellt.
Comment 7 Jascha Geerds univentionstaff 2012-09-13 11:41:10 CEST
Weitere Aufräumarbeiten werden in Bug #28489 ausgelagert. In diesem Bug geht es ab jetzt nur noch darum, dass das Frontend mit allen Browsern genau wie zuvor aussieht.
Comment 8 Alexander Kläser univentionstaff 2012-09-13 15:42:10 CEST
Folgende Sachen sind mir (im FF) noch aufgefallen:
* UDM-Subtabs (und auch die gesamte Page) haben einen Rand
* Subtabs onClick müssten einen Gradientenverlauf haben
* Subtabs onHover sollte Dunkelgrau sein
* In den Gridzeilen ist bei onMouseDown noch ein Gradientenverlauf drin, der vorher nicht da war
* In UDM sehen die Checkboxen irgendwie lustig aus :)
* Die ComboBox scheint noch etwas zu hoch zu sein, der Pfeil verschiebt sich auch
* Die Menüs (Univention und Benutzer) müssten noch ein paar Pixel nach unten versetzt werden
Comment 9 Alexander Kläser univentionstaff 2012-09-13 15:47:05 CEST
Created attachment 4656 [details]
Screenshot UDM

Noch ein Anzeigefehler (FF), scheinbar wenn sich eine deaktivierte ComboBox neben einer aktivierten befindet. (Hier sind auch die Checkboxen sichtbar.)
Comment 10 Jascha Geerds univentionstaff 2012-09-14 12:59:18 CEST
(In reply to comment #8)
> Folgende Sachen sind mir (im FF) noch aufgefallen:
> * UDM-Subtabs (und auch die gesamte Page) haben einen Rand
> * Subtabs onClick müssten einen Gradientenverlauf haben
> * Subtabs onHover sollte Dunkelgrau sein
> * In den Gridzeilen ist bei onMouseDown noch ein Gradientenverlauf drin, der
> vorher nicht da war
> * In UDM sehen die Checkboxen irgendwie lustig aus :)
> * Die ComboBox scheint noch etwas zu hoch zu sein, der Pfeil verschiebt sich
> auch
> * Die Menüs (Univention und Benutzer) müssten noch ein paar Pixel nach unten
> versetzt werden

Bis auf die ComboBox wurden alle Probleme behoben. Für letzteres (und alle Fehler die zukünftig auftauchen) wurde ein extra Bug angelegt: Bug #28516

univention-management-console-frontend (2.0.54-1)

Nach Absprache wurde hierfür kein eigener Changelog geschrieben. Es wurde der Eintrag für das Dojo Update (Bug #26857) um die Bugnummer dieses Bugs erweitert, da hier vorrangig nur Probleme gefixed wurden, welche durch das Dojo Update entstanden sind.
Comment 11 Dirk Wiesenthal univentionstaff 2012-09-14 14:28:52 CEST
2-zeilige Tooltips haben keinen weichen Verlauf, sondern einen harten Schnitt (z.B. UCR "Schlüsselwort"). Formulare zeigen manchmal kleine, blaue Tooltips an (z.B. UCR Hover über das Suchformular). In System-Setup stimmt etwas auf der Netzwerk-Seite nicht. Die DHCP-Query wird merkwürdig angezeigt.

Außerdem sind überall häufig invalide "da"-Einträge im CSS selbst.

Ich öffne mal wieder und suche weiter.
Comment 12 Dirk Wiesenthal univentionstaff 2012-09-14 15:18:55 CEST
Die Menüs (Klick auf mehr und Rechtsklick) sind nach oben verschoben.
Comment 13 Jascha Geerds univentionstaff 2012-09-17 12:07:09 CEST
(In reply to comment #11)
> 2-zeilige Tooltips haben keinen weichen Verlauf, sondern einen harten Schnitt
> (z.B. UCR "Schlüsselwort"). Formulare zeigen manchmal kleine, blaue Tooltips an
> (z.B. UCR Hover über das Suchformular). In System-Setup stimmt etwas auf der
> Netzwerk-Seite nicht. Die DHCP-Query wird merkwürdig angezeigt.
> 
> Außerdem sind überall häufig invalide "da"-Einträge im CSS selbst.
> 
> Ich öffne mal wieder und suche weiter.

Ich habe hier noch die fehlerhaften "da"-Einträge korrigiert. Die restlichen Fehler wurden in Bug #28516 ausgelagert.

(In reply to comment #12)
> Die Menüs (Klick auf mehr und Rechtsklick) sind nach oben verschoben.

Das wurde in der neuen Version angepasst.

Paket baut gerade (univention-management-console-frontend (2.0.59-1))
Comment 14 Dirk Wiesenthal univentionstaff 2012-09-17 15:04:09 CEST
Zur Abwechlung mal weniger CSS und mehr Images (auch wenn es wohl am Ende doch auf CSS hinausläuft): Die UVMM Status-Icons fehlen, dadurch ist das Grid von UVMM eigentlich kaum benutzbar.
Comment 15 Jascha Geerds univentionstaff 2012-09-17 15:42:10 CEST
(In reply to comment #14)
> Zur Abwechlung mal weniger CSS und mehr Images (auch wenn es wohl am Ende doch
> auf CSS hinausläuft): Die UVMM Status-Icons fehlen, dadurch ist das Grid von
> UVMM eigentlich kaum benutzbar.

Das ist vermutlich ein JavaScript Problem. Habs mal an Bug #28551 geschrieben.
Comment 16 Dirk Wiesenthal univentionstaff 2012-09-17 18:35:26 CEST
Keine kritischen Fehldarstellungen gefunden. Alles andere im Folgebug
Comment 17 Stefan Gohmann univentionstaff 2012-12-12 21:08:43 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

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