XH Core-Styles - Systemschriftart

[frase]

... dann wäre vielleicht ein „Ersetze PNG-Icons durch FontAwesome-Icons“-Patch ein erster guter Schritt.

Okay. Ich denke, dass ich verstehe, warum du das Step by Step machen willst.
Im Github soll es nachvollziehbar und rückgängig zu machen sein. Und die Diffs sollen übersichtlich bleiben.

Also:
Hier eine Filebrowser-Version, bei der ausschließlich die Images mit FA-Icons ersetzt sind.
Getestet mit einer jungfräulichen XH 1.7.2 Version und den 3 Standard-Templates.

filebrowser_img---fa.zip

geänderte Dateien:
- \plugins\filebrowser\admin.php
- \plugins\filebrowser\tpl\editorbrowser.html
- \plugins\filebrowser\classes\View.php
- \plugins\filebrowser\css\stylesheet.css

Vielleicht könnten wir als nächsten Schritt die Bildgrößenfestlegung für die Bilderanzeige beim Überfahren der Bilder-Links mit der Mouse entfernen. Mir schwebt stattdessen so etwas vor wie max-width und max-height per CSS.
In einigen Templates greift das nämlich nicht.
Beispiel: richu172 in Verbindung mit großen Bildern im Images-Ordner.
Zum Testen:

userfiles-example-images.zip

[frase]

Hier eine Filebrowser-Version, bei der ausschließlich die Images mit FA-Icons ersetzt sind.

Hier noch ein Vorschlag zum obigen Download.
Man könnte die geöffneten Ordner noch mit "fa-folder-open" darstellen.
Dazu in View.php die Zeilen 178 bis 187 ändern:

Code: Select all

    $class = 'folder';
    $fa_class = 'folder';
    if (substr($this->currentDirectory, 0, strlen($folder)) == $folder) {
        $class = 'openFolder';
        $fa_class = 'folder-open';
        }
        $temp = explode('/', $folder);
        $html = '<li class="' . $class . '"><a href="' . $sn . '?'
            . XH_hsc($this->linkParams) . '&amp;subdir=' . $folder . '"><span class="fa fa-' . $fa_class . ' fa-fw fa-lg"></span> '

Und natürlich im Stylesheet ergänzen, Zeile 180

Code: Select all

div[id*='xhFilebrowser'] span.fa-folder,
div[id*='xhFilebrowser'] span.fa-folder-open {
    color: rgb(255,153,0);
}

[cmb]

Okay. Ich denke, dass ich verstehe, warum du das Step by Step machen willst.
Im Github soll es nachvollziehbar und rückgängig zu machen sein. Und die Diffs sollen übersichtlich bleiben.

Ja, genau darum geht es. Wenn man bei Versionsverwaltungen nach den ungeschriebenen Regeln spielt, macht das vieles leichter. So kann man z.B. auf diverse Weisen später nachschauen, was wann vom wem warum geändert wurde. Unübersichtliche Commits, die gleich diverse Dinge auf einmal tun, sind dabei allerdings wenig hilfreich (manchmal stoße ich auf alte Commits, bei denen einfach eine neuere Version die alte ersetzt – dann bin ich so klug als wie zuvor).

Hier eine Filebrowser-Version, bei der ausschließlich die Images mit FA-Icons ersetzt sind.

Super – sehr gefällig! Habe ich gleich mal in einem Branch auf meinem XH-Fork eingestellt (minimale Änderungen: alte PNG-Icons und zwei auskommentierte CSS-Regeln gelöscht).

Und gleich noch ein Vorteil des übersichtlicheren Commits: ich denke, diese Verbesserung sollte erst mit XH 1.8 ausgeliefert werden (für 1.7 sollte es nur noch kleinere Bugfixes geben). Da aber die Neuerungen unter 1.7.2 gemacht wurden, hätten sie ein paar andere Änderungen in master überschrieben; da alles aber sehr überschaubar war, konnte ich das flink beheben.

Man könnte die geöffneten Ordner noch mit "fa-folder-open" darstellen.

Sehr schön! Habe ich auch gleich gepusht.

Vielleicht könnten wir als nächsten Schritt die Bildgrößenfestlegung für die Bilderanzeige beim Überfahren der Bilder-Links mit der Mouse entfernen. Mir schwebt stattdessen so etwas vor wie max-width und max-height per CSS.
In einigen Templates greift das nämlich nicht.
Beispiel: richu172 in Verbindung mit großen Bildern im Images-Ordner.
Zum Testen:
userfiles-example-images.zip

Oh, da muss tatsächlich nachgebessert werden. Das könnte man vielleicht sogar als Bug einstufen, und den Fix für XH 1.7.3 vorbereiten. (Längerfristig sollte der Filebrowser wohl Thumbnails für diese Vorschau ausliefern – spart Bandbreite.)

[frase]

ich denke, diese Verbesserung sollte erst mit XH 1.8 ausgeliefert werden (für 1.7 sollte es nur noch kleinere Bugfixes geben)

Gut. Da haben wir ja wirklich genug Zeit, um es gründlich zu machen.

Du warst ein wenig zu schnell. Ich dachte, du schaust es dir erst an und findest Fehler.
Es gibt nämlich mindestens einen.
In editorbrowser.html endet der FA-Aufruf falsch.

Für mich steht jetzt die Frage: Womit geht es weiter? Was wäre dir recht? Was soll ich zuerst tun?
Es gibt da nämlich noch 'ne ganze Menge.
- Bildgrößen (siehe oben) - könnte ich vorläufig komplett mit CSS abfangen, ohne Code-Änderung
- Ich hätte gerne eine h1-Überschrift "Filebrowser" im CMS-Browser
- Das Stylesheet könnte mächtig eingedampft - und dann auch gleich wieder aufgeblasen werden
Schriftfestlegung, Li-Styles, und und und

Soll ich einfach loslegen und dir hier überschaubare Häppchen liefern, oder hast du Wünsche, wie die Schritte sein sollen?

[cmb]

ich denke, diese Verbesserung sollte erst mit XH 1.8 ausgeliefert werden (für 1.7 sollte es nur noch kleinere Bugfixes geben)

Gut. Da haben wir ja wirklich genug Zeit, um es gründlich zu machen.

Allzu lange sollte 1.8 nicht mehr auf sich warten lassen. Zumindest wirklich mal damit anfangen sollten wir bald.

Du warst ein wenig zu schnell. Ich dachte, du schaust es dir erst an und findest Fehler.
Es gibt nämlich mindestens einen.
In editorbrowser.html endet der FA-Aufruf falsch.

Macht doch nichts – ist ja noch nicht in cmsimple-xh/cmsimple-xh eingepflegt, sondern lediglich in cmb69/cmsimple-xh. Da kann also noch beliebig geändert werden. Jedenfalls nun im Repo gefixt.

Und ja, ich hätte gründlicher schauen können; mein Editor hat das sogar rot hervorgehoben, aber das tut er bei vielen Dingen in editorbrowser.html, eben weil das keine HTML- sondern eine PHP-Datei ist. Sollten wir auch mal ändern (müssen dann aber sicherstellen, dass kein Sicherheitsproblem vorliegt, wenn die Datei direkt abgerufen wird).

- Bildgrößen (siehe oben) - könnte ich vorläufig komplett mit CSS abfangen, ohne Code-Änderung

Das wäre doch ein guter Anfang; am besten mit einem sauberen XH 1.7.2 machen, damit es leichter für XH 1.7.3 berücksichtigt werden kann.

- Ich hätte gerne eine h1-Überschrift "Filebrowser" im CMS-Browser

Oh, ja, die fehlt!

- Das Stylesheet könnte mächtig eingedampft - und dann auch gleich wieder aufgeblasen werden
Schriftfestlegung, Li-Styles, und und und

Ist das relevant, wenn wir die .xhContainer-Sache umsetzen? Falls nicht, dann erst mal so lassen.

Soll ich einfach loslegen und dir hier überschaubare Häppchen liefern, oder hast du Wünsche, wie die Schritte sein sollen?

Letztlich solltest du das entscheiden. Bin schließlich nicht der Chef.

[frase]

- Bildgrößen (siehe oben) - könnte ich vorläufig komplett mit CSS abfangen, ohne Code-Änderung

Das wäre doch ein guter Anfang; am besten mit einem sauberen XH 1.7.2 machen, damit es leichter für XH 1.7.3 berücksichtigt werden kann.

Only for XH 1.7.3.
Insert the following Lines at the end oft the plugin stylesheet.css:

Code: Select all

a.xhfbfile img,
span.xhfbfile img {
    border: none !important;
    margin: .25em 0 0 0 !important;
    max-height: 10em !important;
    max-width: 10em !important;
    box-shadow: none !important;
}

Ich denke, dass 10em genügen. Es gibt ja auch "sehr schmale" Templates.
Box-Shadow wird damit auch gleich entfernt.

[frase]

Ist das relevant, wenn wir die .xhContainer-Sache umsetzen? Falls nicht, dann erst mal so lassen.

Am Filebrowser schraube ich nur herum, weil ich den auch im xhContainer haben will.
Erst der, dann das Andere.

[cmb]

Allzu lange sollte 1.8 nicht mehr auf sich warten lassen. Zumindest wirklich mal damit anfangen sollten wir bald.

Da fällt mir wieder ein, dass unsere Testsuite wegen dem noch nicht PHP 7.3 kompatiblen uopz nicht unter PHP 7.3 läuft. Da sollte dringend nachgebessert werden. Grrr!

Insert the following Lines at the end oft the plugin stylesheet.css:
[…]

Danke! Habe ich gleich eingecheckt.

[Korwin]

I understand less than half of what you are discussing, but if you only knew how happy the simple user is to know that the system is alive and evolving, while carefully preserving its best principles.

Sorry for offtopic.

[frase]

Sorry for offtopic.

Нет проблем. Похвала иногда бывает хорошей.