search

evcc-io / evcc

Sonne tanken ☀️🚘
https://evcc.io
MIT License
3.32k stars 606 forks source link

Fehler bei Vollbild in iOS #12941

Closed princemaxwell closed 6 months ago

princemaxwell commented 6 months ago

Ich habe mir in iOS ein Lesezeichen zu EVCC auf den Homescreen gelegt, damit ich EVCC immer schnell starten kann. Dabei wird der Safari immer im Vollbild geöffnet, ohne Adressleiste, vor- und zurück Button, etc., quasi wie ne "App".

Seit der 0.124.10 ist das Bild nun komplett am oberen Rand und hinter der Notch und Leiste oben. Dadurch lassen sich auch die Einstellungen über das Symbol oben rechts nicht mehr bedienen.

Möglich, dass es etwas mit #12792 zu tun hat?

Bildschirmfoto 2024-03-14 um 17 44 24 2

mucki12 commented 6 months ago

Wäre vielleicht noch interessant zu wissen auf welchem iPhone überhaupt? Auf einem 14 pro und einem 15 pro kann ich das nicht bestätigen.

StevieC121176 commented 6 months ago

Kann ich auch nicht bestätigen. Läuft auf dem 15pro mit 17.4 ohne Darstellungsfehler.

princemaxwell commented 6 months ago

@mucki12 @StevieC121176 iPhone 14 Pro mit 17.4 Neustart und Safari Website-Daten löschen hab ich schon versucht...

https://github.com/evcc-io/evcc/assets/37151645/be0f5a1f-871d-4760-8245-ccc337be317c

mucki12 commented 6 months ago

A7B51993-BAC2-44E1-94A8-303726FF7FE5_1_201_a

princemaxwell commented 6 months ago

@mucki12 Den einzigen Unterschied den ich sehe ist, dass ich die aktuellste nightly habe und du bist auf der stable.

mucki12 commented 6 months ago

@princemaxwell Habe zwei Docker am Laufen (natürlich nicht gleichzeitig). Habe gerade die aktuellste 0.124.10 nightly gezogen und kann das Problem damit ebenfalls nicht nachstellen. Hast du die Max Version vom iPhone?

princemaxwell commented 6 months ago

@mucki12 Nö, kleines iPhone 14 Pro. Wenn ich es normal in Safari aufrufe, ist das Problem nicht da. Betrifft nur den Vollbild Modus.

mucki12 commented 6 months ago

Habe beides getestet und weder direkt in Safari noch im Vollbildmodus das Problem.

schuepc commented 6 months ago

Ich habe den Fehler auch. Das Bild ist komplett am oberen Rand. iOS Version 17.4, iPhone 13 Pro, EVCC 0.124.10

naltatis commented 6 months ago

Ich kann das leider auch nicht reproduzieren. Auf meinen 15 Pro mit 17.4 passt alles. Habs gerade mit dem 14 Pro im Simulator ausprobiert. Da sitzt es auch korrekt.

Bildschirmfoto 2024-03-14 um 22 27 12

Irgend einen Unterschied muss es da noch geben 🤔

@princemaxwell Mir ist auf deinem Screenshot aufgefallen, dass dein Mobilfunk-Icon irgendwie anders aussieht. Die vier Balken haben ne Unterbrechnung unten. Bei mir gehen die durch.

naltatis commented 6 months ago

@princemaxwell @schuepc habt ihr den gleichen Effekt wenn ihr demo.evcc.io als Homescreen App hinzufügt?

mucki12 commented 6 months ago

Die vier Balken haben ne Unterbrechnung unten.

Ist imho die Standardanzeige bei Nutzung von Dual-SIM.

naltatis commented 6 months ago

Ist imho die Standardanzeige bei Nutzung von Dual-SIM.

Ah ok. Daran wirds sicher nicht liegen :D

StefanSchoof commented 6 months ago

Gibt es einen Unterschied, ob es EU iPhone sind? (Ich weiß es ist ein Long Shot, aber alle offensichtlich Dinge, sind ja schon ausgeschlossen)

Edit: Und wie sieht es mit http vs https aus? Das hat Einfluss, ob es eine PWA oder nur ein Link ist.

princemaxwell commented 6 months ago

@naltatis @StefanSchoof Das Offensichtlichste habe ich leider vorher nicht ausprobiert, aber es hat jetzt die Lösung gebracht. Nachdem die DEMO ging habe ich das Lesezeichen auf dem HomeScreen einfach neu angelegt und schon gehts. Spooky.

Hier die 3 Links im Vergleich, ALT > DEMO > NEU

@schuepc Probier das mal, geht es dann bei Dir auch wieder korrekt?

https://github.com/evcc-io/evcc/assets/37151645/825e8e13-9158-44a6-8a7e-b6e578102a10

naltatis commented 6 months ago

Ok, merkwürdig, dass das einen Unterschied macht. Wir haben aber in Vorbereitung für die App ein paar Dinge an den Metadaten Markup geändert. Kann natürlich sein, dass iOS beim Installieren der PWA irgendeine Entscheidung/Unterscheidung abhängig vom Inhalt der Seite trifft und diese dann auch später nicht mehr anpasst. Falls jemand herausfindet, was da genau passiert, wäre ich interessiert.

Aber schön, dass es jetzt erstmal gelöst ist.

An-Wi commented 6 months ago

Auch wenn es kein iOS-, sondern ein Browserproblem ist, packe ich meine Beobachtungen hier mal mit hin, da es vermutlich ein Seiteneffekt der letzten Anpassungen ist: ScreenShot_9394_

Die Darstellung in der Hauptansicht ist verändert zur letzten Version. Die Überschrift ist nun sehr weit nach oben gerutscht und passt in der Höhe nicht mehr zu den Ansichten "Ladevorgänge" und "Configuration". Sieht für mich "unschön" aus, zumal in Browseransichten oftmals mehr Platz in der Höhe zur Verfügung steht - dadurch wirken die Freiräume nun noch größer.

Ist mir erst mit dem letzten Update auf 0.124.10 aufgefallen. Ich denke, in 0.124.9 war es noch "normal".

Auf dem iPhone 12 mit iOS 16 sind die oben von @princemaxwell beschriebenen Probleme mit meinem alten Lesezeichen nicht aufgetreten - das sieht ok aus. Aber auch hier kann man beim Wechseln der Ansichten sehen, dass die Hauptansicht-Caption deutlich weiter oben ist, wirkt aber nicht störend. Allerdings springen mit der veränderten Überschriftenzeile dann beim Ansichtenwechsel auch die oberen Buttons (Menu, Meldungen, Home).

naltatis commented 6 months ago

@An-Wi Ja, guter Punkt. Ich hab daraus mal eigenes Issue erstellt.

schuepc commented 6 months ago

@naltatis @StefanSchoof Das Offensichtlichste habe ich leider vorher nicht ausprobiert, aber es hat jetzt die Lösung gebracht. Nachdem die DEMO ging habe ich das Lesezeichen auf dem HomeScreen einfach neu angelegt und schon gehts. Spooky.

Hier die 3 Links im Vergleich, ALT > DEMO > NEU

@schuepc Probier das mal, geht es dann bei Dir auch wieder korrekt?

RPReplay_Final1710486338.mp4

Bei mir hat es auch funktioniert... Gelöscht und neu angelegt... alles war wieder gut.