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

Anpassung des Standard-Templates (fhs-simple-2019) an WebKit-Browser #556

Closed
frase-git opened this issue Mar 14, 2022 · 32 comments
Closed
Labels
Milestone

Comments

@frase-git
Copy link
Contributor

Siehe diesen Post und folgende:
https://cmsimpleforum.com/viewtopic.php?f=16&t=18230&start=10#p83055

@frase-git
Copy link
Contributor Author

Das betrifft wahrscheinlich auch die core.css (was noch zu prüfen wäre).

@cmb69
Copy link
Member

cmb69 commented Jan 23, 2023

Hm, da sollten wir wohl noch für 1.7.6 was machen; iOS ist schließlich keine unwesentliche Plattform. Hat jemand Zeit, sich das anzuschauen?

@cmb69 cmb69 added this to the 1.7.6 milestone Jan 23, 2023
@cmb69 cmb69 added the bug label Jan 23, 2023
@frase-git
Copy link
Contributor Author

frase-git commented Jan 23, 2023

Hat jemand Zeit, sich das anzuschauen?

Zeit? Was ist das?
Ich denke, es ging bisher so und kann bis 1.8 so bleiben (wenn auch mit Bauchschmerzen).
Verschieben.
Dann brauchen wir was Neues.

@olape-git
Copy link
Contributor

Ich schau mal "meine" Templates durch.
Da habe ich immer eine Lösung drin gehabt.

@olape-git
Copy link
Contributor

Aber eigentlich gibt es die Lösung doch schon:
https://cmsimpleforum.com/viewtopic.php?f=16&t=18230&start=20#p83083

cmb69 added a commit to cmb69/cmsimple-xh that referenced this issue Jan 23, 2023
@cmb69
Copy link
Member

cmb69 commented Jan 23, 2023

Ich denke, es ging bisher so und kann bis 1.8 so bleiben (wenn auch mit Bauchschmerzen).

Wenn es so einfach zu beheben ist, wie @olape-git sagt, dann können wir das doch gleich machen: #556 #568.

@olape-git
Copy link
Contributor

So, mal zum Vergleich
wiki ohne Patch
screen_wiki_vor
wiki mit Patch
screen_wiki_nach

@olape-git
Copy link
Contributor

Demo ohne Patch
screen_demo_vor
Demo mit Patch
screen_demo_nach

@olape-git
Copy link
Contributor

Demo ohne Patch
screen_demo_vor2
Zum Vergleich ExpandContract mit Patch

screen_expand_nach

@olape-git
Copy link
Contributor

Es gibt aber eine ganze Reihe weiterer Probleme mit WebKit-Browsern.
Alles was irgendwie mit Formularen zu tun hat benötigt hier eine extra Behandlung.
screen_formular
screen_themeswitcher

Leider kann man diese Dinge nicht mit irgendeinem Emulator testen.

@cmb69
Copy link
Member

cmb69 commented Jan 25, 2023

Danke für die Screenshots, @OlaPe @olape-git! #568 würde ja nur das Search-Form fixen; soll ich das trotzdem schon einspielen, oder brauchen wir gleich noch weitere Nachbesserungen?

PS: doofe Autovervollständigung!

@frase-git
Copy link
Contributor Author

Bin einigermaßen betrübt.
Da ich hier nichts testen kann, kann ich auch keine Unterstützung leisten.

@olape-git
Copy link
Contributor

würde ja nur das Search-Form fixen; soll ich das trotzdem schon einspielen, oder brauchen wir gleich noch weitere Nachbesserungen?

Ich denke ja
mit zusätzlich
#searchbox input[name="search"] { -webkit-appearance: none; -webkit-border-radius: 0;
sieht es glaube ich passend aus.
screen_demo_nach2

@cmb69
Copy link
Member

cmb69 commented Jan 25, 2023

mit zusätzlich
#searchbox input[name="search"] { -webkit-appearance: none; -webkit-border-radius: 0;
sieht es glaube ich passend aus.

Also so?

@olape-git
Copy link
Contributor

Also so?

Ja, so habe ich es jetzt in der Demo drin (Screenshot).

@cmb69 cmb69 closed this as completed in 18e51a1 Jan 25, 2023
@cmb69
Copy link
Member

cmb69 commented Jan 25, 2023

Super! Eingespielt.

olape-git pushed a commit to olape-git/cmsimple-xh that referenced this issue Jan 28, 2023
@lck-git
Copy link

lck-git commented Jan 30, 2023

Bin einigermaßen betrübt. Da ich hier nichts testen kann, kann ich auch keine Unterstützung leisten.

mit zusätzlich
#searchbox input[name="search"] { -webkit-appearance: none; -webkit-border-radius: 0;
sieht es glaube ich passend aus.

Ich kann leider auch nichts prüfen/checken. :(

IMHO, grundsätzlich sollten wir nur Änderungen machen, wenn das Problem mindestens von einer weiteren Person bestätigt wird. Vor allen Dingen auch getestet mit aktuellen iOS-Versionen. iOS8 ist ja schon etwas in die Jahre gekommen (von 2014), mittlerweile gibt es schon iOS16. Welche Browserversion von Safari läuft den da drauf?

BTW: Hier mal ein Artikel von vor einem Jahr zum Safari Browser: "Safari als neuer Internet Explorer"

@olape-git
Copy link
Contributor

IMHO, grundsätzlich sollten wir nur Änderungen machen, wenn das Problem mindestens von einer weiteren Person bestätigt wird.

Dann wären aber schon viele Dinge liegengeblieben.
Ganz grundsätzlich weiß ich, das Apple-User an die runden Button gewöhnt sind.
Steht der Button allein, ist das ja auch nicht wirklich das Problem.
Aber so wie es hier aussah, war es echt Sch...

Ich verstehe dein Problem, wenn man es nicht selber Testen kann ist das Mist, aber das kann man ändern. ;-)

Ich kann es nur empfehlen. Mir ist es regelmäßig ein Graus, wenn ich mich mit Android rumärgern muss.
Ich nutze privat noch ein iPad 5. Generation, Einführung März 2017. Es läuft top, der Akku ist noch sehr gut, es wird immer noch mit Updates versorgt. Das sind jetzt also 6 Jahre. In der Zeit hätte man bei Android schon 2x kaufen müssen, um aktuelle zu bleiben.
Aber diesen Streit möchte ich hier nicht beginnen.

Die Dinge lassen sich leider nicht in irgendwelchen Online-Test-Tools prüfen
Das sollte uns aber nicht veranlassen eine ganze Usergruppe zu vernachlässigen.

iOS8 ist ja schon etwas in die Jahre gekommen (von 2014), mittlerweile gibt es schon iOS16. Welche Browserversion von Safari läuft den da drauf?

Keine Angst, es ist iOS 16.1 gewesen.
https://developer.apple.com/documentation/safari-release-notes/safari-16_1-release-notes

Ich habe noch ein iPad mit iOS8, aber ich glaube da geht so gut wie gar nix mehr.
Könnte ich glatt mal testen.

@cmb69
Copy link
Member

cmb69 commented Jan 30, 2023

Das sollte uns aber nicht veranlassen eine ganze Usergruppe zu vernachlässigen.

Denke ich auch. Und in diesem Fall bemerkt es der Webmaster selbst wahrscheinlich gar nicht; und ein vorbeischauender Besucher meldet dem Webmaster das Problem nicht, weil er keine Lust auf eine "kaputte" Site hat. Und insbesondere -webkit-* Regeln stören mich nicht wirklich.

@lck-git
Copy link

lck-git commented Jan 30, 2023

Keine Angst, es ist iOS 16.1 gewesen.

Ah ja, ok. Im Forum hast du iOS 8 erwähnt, deswegen meine Bedenken.
Ich habe MacOS Monterey mal in einer VirtualBox in Windows 11 installiert. Da konnte ich im Safari keine Probleme feststellen.

Wie sieht es dann mit dem Template overlay_03 aus, muss da nicht auch der Fix hinzu?

@olape-git
Copy link
Contributor

Wie sieht es dann mit dem Template overlay_03 aus, muss da nicht auch der Fix hinzu?

Ganz theoretisch ja. Ich hatte es schon geschrieben. Eigentlich müssten alle Eingabefelder und Button damit versehen werden.
Du siehst das auch in den Screenshots beim Themeswitcher. Aber ich hatte auch schon geschrieben, da wo die Felder oder die Buttons allein stehen, finde ich das jetzt nicht ganz so wichtig. Apple-User kennen das Problem, viele bemerken es vielleicht gar nicht, weil es eben fast immer so ist. Aber an einigen Stellen sah es dann wirklich sehr bescheiden aus.
Ich schau heute Abend overlay_03 nochmal durch. Und wenn ich es heute schaffe, teste ich nochmal mit iOS8 ;-)

@olape-git
Copy link
Contributor

Und wenn ich es heute schaffe, teste ich nochmal mit iOS8 ;-)

Schade, das geht wirklich gar nicht mehr. Also alle Seiten unter cmsimple-xh.org können gar nicht geladen werden. Das liegt aber an irgendwelchen Sicherheitsproblemen, aber dafür habe ich jetzt keinen Nerv. Komischerweise, die Seiten unter meiner Domain und unsere Vereinsseite funktionieren, mit einer Warnung, aber dann geht es und sieht auch alles normal aus. Keine Design-Ausreißer.

@olape-git
Copy link
Contributor

Von der Demo, also noch Version 2.
Ich habe heute auf die Schnelle keine Möglichkeit mehr für Version 3.
Aber, auch in 2 ist nichts ungewöhnliches zu sehen.
Alles wie erwartet.
0459A2CE-81E6-42CA-B732-6DF48606843D
36652CF1-5AD7-4206-89AE-C071EAD9EA12
020F0D30-9780-4C62-A45F-27C21A0E4B67

@lck-git
Copy link

lck-git commented Feb 1, 2023

[“@olape-git wrote:]
Aber, auch in 2 ist nichts ungewöhnliches zu sehen.

Wenn du etwas Zeit hast, schau dir mal die Template-Demo overlay_03 mit dem iPad 5 an und hier das Mailformular. Normalerweise sind da keine abgerundeten Ecken in den input-Feldern.
Ich habe hier folgenden Fix eingebaut.

/* Fix for Browser Safari border-radius issue */
input:not([type="radio"]):not([type="checkbox"]) {
    -webkit-appearance: none;
    -webkit-border-radius: 0;
    appearance: none;
    border-radius: 0;
}

@ Alle
Wenn das so passt, sollten wir das dann nicht gleich in die core.css übernehmen?

@frase-git
Copy link
Contributor Author

frase-git commented Feb 2, 2023

@ Alle
Wenn das so passt, sollten wir das dann nicht gleich in die core.css übernehmen?

So wie es jetzt da steht, würde es ja auf alle Inputs wirken. Außer Radio und Check (warum eigentlich?).
Das könnte u.U. die Absichten des Templates stören.
Ich denke, mit solchen Aktionen sollten wir warten, bis wir den abgetrennten Admin-Modus haben und uns das dann nochmal genauer anschauen. Möglicherweise müsste das noch genauer spezifiziert (#xhContainer) und/oder überhaupt nur im Admin-Template deklariert werden. (Vielleicht schenkt mir bis dahin jemand ein ausgedientes iGerät.) ;-)

@frase-git
Copy link
Contributor Author

Übrigens:
Mit appearance: none; setzt man alle Eigenschaften zurück und muss/sollte diese dann anschließend neu (im gewollten Stil) deklarieren – sonst ist das Ergebnis auch unerwartet.

https://www.mediaevent.de/css/appearance.html

@olape-git
Copy link
Contributor

Wenn du etwas Zeit hast, schau dir mal die Template-Demo overlay_03 mit dem iPad 5 an und hier das Mailformular.

Mach ich

Normalerweise sind da keine abgerundeten Ecken in den input-Feldern.

Ich weiß, aber hier ist es nicht so krass, damit kann man leben denke ich.

Ich denke, mit solchen Aktionen sollten wir warten

Ich denke auch, das sollten wir jetzt nicht übers Knie brechen. Wenn wir erst mal die grobe "Fehler" fixen, ist das für die 1.7.6 erst mal ok.

@olape-git
Copy link
Contributor

@lck-git
Copy link

lck-git commented Feb 2, 2023

Danke. Interessant, sogar die textarea wird nicht verschont.

Ich denke auch, das sollten wir jetzt nicht übers Knie brechen. Wenn wir erst mal die grobe "Fehler" fixen, ist das für die 1.7.6 erst mal ok.

Ok.

@frase-git
Copy link
Contributor Author

Kleiner Hinweis:

overlay_03-screenshot

@lck-git
Copy link

lck-git commented Feb 2, 2023

Danke fürs melden!
Ähnliches Problem hast du mir schon mal gemeldet. Das hab ich auch schon gefixt, aber nur lokal noch nicht online und im Download. :(

@lck-git
Copy link

lck-git commented Feb 5, 2023

Das hab ich auch schon gefixt, aber nur lokal noch nicht online und im Download. :(

Erledigt, Pull-Requests erstellt.

@lck-git lck-git pinned this issue Mar 5, 2023
@lck-git lck-git unpinned this issue Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants