Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Änderung von Icons #346

Open
daencore opened this issue Sep 6, 2018 · 7 comments · May be fixed by #552
Open

Änderung von Icons #346

daencore opened this issue Sep 6, 2018 · 7 comments · May be fixed by #552

Comments

@daencore
Copy link

daencore commented Sep 6, 2018

Hallo, ich würde gerne die Icons in Lansuite ändern.

Ich möchte ich gerne in Modernes Iconset wie FontAwsome oder Bootstrap verwenden. Dies scheint aber leider aufgrund des Aktuellen Aufbaus von Lansuite nicht so einfach möglich zu sein da die Icons als Bild in den Code Eingebettet sind.

Mein erster Ansatz um dies einzubinden war der Folgende.
Ich habe in meinem Template in den <head> der main.htm die FontAwsome Icons Eingebunden mit: <link rel="stylesheet" href="https://use.fontawesome.com/releases/v5.3.1/css/all.css" integrity="sha384-mzrmE5qonljUremFsqc01SB46JvROS7bZs3IO2EmfFsd15uHvIt+Y8vEf7N7fWAU" crossorigin="anonymous">

Danach habe ich in der CSS nach den Icons gesucht und habe auch die der Online/Offline anzeige gefunden. Die habe ich durch folgenden Code Ersetzt.
.user_online:after {
font-family: "Font Awesome 5 Free";
font-weight: 900;
text-shadow: 1px 1px 1px #000;
font-size: 1.3em;
content: "\f007";
color: #008000;
}
.user_offline:after {
font-family: "Font Awesome 5 Free";
font-weight: 900;
text-shadow: 1px 1px 1px #000;
font-size: 1.3em;
content: "\f007";
color: #535353;
}
Mit einem doch recht schönen Ergebnis:
bild 2ftjf

Aber weitere Icons waren nicht zu finden. Nach weiterer suche habe ich dann raus gefunden wie diese Icons definiert werden. Und zwar unter anderem in den Templates in der datei ls_fetch_icon.htm mit folgenden code: <img src="design/images/icon_{$name}.png" alt="{$name}" title="{$hint}" width="20" height="20" border="0" {$additionalhtml} />

Hier dürfte der Tag {$name} dann das Icon definieren.

Jetzt war meine Überlegung das ich einfach eine CSS neue classe erstelle mit den Icon infos und den Code ändere.

Ein Beispiel anhand des Unlock Icons im Benutzer Menü:
CSS Classe:
.fa_icon_unlocked {
font-family: "Font Awesome 5 Free";
font-weight: 900;
text-shadow: 1px 1px 1px #000;
font-size: 1.3em;
content: "\f3c1";
color: #535353;
}
In der datei ls_fetch_icon.htm habe ich dann den Html Code vom Icon Bild ersetzt durch:
<a class="fa_icon_{$name}"></a>

Theoretisch müsste das ja funktionieren, tut es nur nicht. Er ließt mir die CSS Klasse korrekt und sollte das FontAwsome Icon anzeigen. Nur zeigt er es leider nicht an. Folgendes bekomme ich zu sehen auf der Seite im Quellcode was meine Theorie ja bestätigen würde.
bild 3k

So hat jetzt einer ne Idee warum das nicht klappt? Ich habe das Gefühl das sich hier das CSS von .ui-widget-content a darüber schiebt und dadurch das Icon nicht ausgibt.

Oder habt ihr vlt einen anderen Ansatz? Bitte um Hilfe. Danke

@M4LuZ
Copy link
Collaborator

M4LuZ commented Sep 6, 2018

Ich würde jetzt gerne in der Doku nachgucken, aber die ist seit #239 nicht mehr verfügbar.
Ich vermute ganz stark, dass sich das über ein eigenes Design regeln lässt, was generell auch bei kleineren Anpassungen die beste Variante ist.
Muss mal gucken, ob und wie wir das angepackt haben.

@daencore
Copy link
Author

daencore commented Sep 6, 2018

@M4LuZ Ok Danke. Als Basis haben wir das OsX Template an unsere Bedürfnisse Angepasst und sind auch sehr Zufrieden. Einzig die Icons und Buttons machen noch Probleme. Ein Eigenes Design ist Ohne Doku vmtl. eine schwierige Aufgabe.

Aber so ein Bootstrap Responsive Design wäre schon ne geile sache weils auch auf Mobilgeräten Perfekt funktioniert und sich andere Sachen leichter einbauen lassen.

@byte666
Copy link
Collaborator

byte666 commented Sep 6, 2018

Viele Sachen werden in einem Mastertemplate geregelt und dann von dem ausgewählten Design überschrieben. Machts etwas tricky bei der Fehlersuche. Generell kopiert man ein Design und passt es an. An Bootstrap hat sich Knox schon mal versucht, sah ganz gut aus. Siehe altes Forum... wenn dann mal wieder online is. Wollten das schon mal umstellen... aber dann kommst gleich in einen Bereich wo man das ganze Ausgabeframework überarbeiten müsste. Dazu müsstes wegen der vielen Plugins erst mal das Kernsystem strippen und neu aufbauen... etc. ;) Dazu fehlte aber die ernsthafte Manpower... bisher.

@daencore
Copy link
Author

daencore commented Sep 6, 2018

Ich habe es geschafft! Es Funktioniert. Meine Theorie war richtig es fehlte nur ein Kleines Detail in der CSS. Noch für jedes Icon eine eigne CSS klasse angelegt und die Icons werden ausgegeben.

Der Richtige CSS Code lautet
.fa_icon_unlocked:after {
font-family: "Font Awesome 5 Free";
font-weight: 900;
text-shadow: 1px 1px 1px #000;
font-size: 1.3em;
content: "\f3c1";
color: #535353;
}
Und in der In der datei ls_fetch_icon.htm musste ich den Html Code vom Icon Bild ersetzt, jedoch mit nem
<i class="fa_icon_{$name}"></i>

Keine ahnung für was das :after in der CSS steht aber so Funktionierts.
Ergebnis:
bild 4gs

@andygrunwald
Copy link
Collaborator

andygrunwald commented Sep 6, 2018 via email

@daencore
Copy link
Author

daencore commented Sep 6, 2018

@andygrunwald Wenns in deutsch sein darf dann gerne. Wo muss ich das genau machen? Bin jetzt nicht der git Pro. ;-)

@andygrunwald
Copy link
Collaborator

@daencore Sprache ist kein Problem.
Ich habe hier (https://lansuite.github.io/lansuite/docs/documentation.html) versucht zu dokumentieren, wie man einen neuen Dokumenations-Artikel schreibt.

Zur Zeit ist unsere Dokumentation noch nicht Multi-Language-Fähig. Aber das können wir ändern.
Versuch es einfach auf Deutsch zu schreiben und via Pull Request einzureichen.

@maxmarkus maxmarkus linked a pull request Mar 13, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants