enhancement
critical
major
nice-to-have
major
enhancement
Features des Theme-Editors
- Theme anlegen/löschen
- CSS-Datei einfügen/erstellen/bearbeiten/löschen
- Theme-Variablen in theme-settings.xml bearbeiten (Icons, CSS-Variablen), überdefinieren (geerbte), anlegen, löschen.
- Theme-Icons hochladen/löschen
Offene Punkte
MergeThemeConfigs liegt im Theme-Editor Paket wird aber aus dem allgemeinen base-build.xml angesprochen.Wenn im Theme X` eine Variable überdefiniert wird, die bisher aus dem Theme `Y geerbt wird, wird diese Variable fälschlicherweise in das Theme Y` zurückgespeichert (statt im Theme `X angelegt).Variable anlegen- ~~Theme schützen - Wenn in der theme.xml`ein Flag `protected gesetzt ist (das an der GUI bei einem neuen Theme nicht gesetzt oder bei einem bestehenden editiert werden kann), dann können bei diesem Theme keine Styles angelegt/editiert/gelöscht werden, keine Variablen angelegt/editiert/gelöscht werden und keine Icons hochgeladen oder gelöscht werden. ~~
Basis-Theme ist bei Neuanlage eine Themes in der GUI eine verpflichtende Eingabe. Ansonsten führt das zwangsläufig dazu, dass das neue Theme nicht funktioniert, weil "mandatory" Variablen wie z.B. MAIN_LAYOUT_CLASS nicht belegt sind.Variable löschen / bzw. Überschreibung löschen - auf Standardwert zurücksetzen, falls es sich um eine geerbte Variable handelt.Style-File löschenIcon löschen / bzw. Überschreibung löschen - auf Standardwert zurücksetzen, falls es sich um ein geerbtes Icon handelt.Theme Eigenschaften bearbeitbar machen.Doppelter Edit-Button bei den Theme-Eigenschaften
Knöpfe "Theme exportieren" und importieren besser ins Menü.Nach dem Speichern von Theme-Eigenschaften verliert die Theme-Tabelle die Selektion und selektiert wieder das erste Element. Dadurch entsteht der Eindruck, dass das Speichern einen komischen Effekt hatte.Die Option "Erweitert" bei der Bearbeitung von Theme-Eigenschaften ist keine Auswahlliste wie bei der Neu-Anlage eines Themes.Es fehlt ein Zyklus-Test. Eine falsche Auswahl/Eingabe crasht die Anwendung komplett.Variablen-Tabelle sollte besser eine Viewport-Tabelle sein. Man muss die Zeilenhöhe an die maximale Höhe der dargestellten Bilder anpassen.Fehlende I18N bei Typ-Auswahl neuer Theme-Variablen
Dialoghöhe für neue Theme-Variable viel zu niedrig.Fehlende Sicherheitsabfrage "Sind sie sicher" beim Löschen von Theme-Variablen, Löschen von Themes, Löschen von Style-Sheets, Löschen von Icons.Beim Speichern von theme.xml wird ge-pretty-printet aber mit Space- statt Tab-Indentation.Gleiches gilt für die theme-settings.xml (falsches Indentation).- Der Maximieren-Knopf des Theme-Editors sollte den kompletten Editor maximieren und nicht nur die Detail-Sicht
Test
- ThemeEditor_Ticket-23854.script.xml
- ThemeEditor_Ticket-23854_SelectAbstractThemes.script.xml