search

cody0-0 / jahresprojekt

1 stars 0 forks source link

Some changes #128

Closed duevil closed 2 months ago

duevil commented 2 months ago
duevil commented 2 months ago

Ist das Verhalten gewollt?

  • Das Bild (mit Caption) nutzt auf Horizontalen Screens die komplette mittlere Column (daran ist erstmal nichts auszusetzen, hatte mich nur kurz gewundert)

  • Die Previous und Next Buttons sind Mittig in ihrer Row, aber da nichts anderes sich an der Row orientiert, sind sie eigentlich nicht mittig (weder zum Bild noch zur Kombi aus Bild und Caption noch zum kompletten Screen)

    • vielleicht translateY um ((button-size + gap) / 2) nach oben?
image

Position der Buttons wird geändert :) Das mit der Column ist ja genau das, was mit #118 umgesetzt werden soll.

cody0-0 commented 2 months ago

Das Zoomen hier ist auf Touch Geräten weiterhin weird. Das war zwar kein Issue, ist jetzt mit dem Swipen aber noch einmal etwas weirder geworden ^^'

cody0-0 commented 2 months ago

Das mit der Column ist ja genau das, was mit #118 umgesetzt werden soll.

Man könnte es auch auf die 2. Row begrenzen und dann ist das Bild dennoch nicht von dessen Auflösung begrenzt. Aber letztendlich ist mir das relativ egal, ob das Bild nun immer unter dem Close Button bleibt, oder eben (wie aktuell) die komplette Höhe (abgesehen vom Padding) verwendet.

duevil commented 2 months ago

Das mit der Column ist ja genau das, was mit #118 umgesetzt werden soll.

Man könnte es auch auf die 2. Row begrenzen und dann ist das Bild dennoch nicht von dessen Auflösung begrenzt. Aber letztendlich ist mir das relativ egal, ob das Bild nun immer unter dem Close Button bleibt, oder eben (wie aktuell) die komplette Höhe (abgesehen vom Padding) verwendet.

Es sollte schon die gesamte Höhe einnehmen, sonst wird's komisch mit Hochformat-Bildern, denke ich

duevil commented 2 months ago

Das Zoomen hier ist auf Touch Geräten weiterhin weird. Das war zwar kein Issue, ist jetzt mit dem Swipen aber noch einmal etwas weirder geworden ^^'

  • Der Zoom beim Tap springt zu random Positionen (nicht unbedingt zu der auf die getippt wurde)
  • Das Navigieren im Zoom ist natürlich immer noch nicht intuitiv (wurde ja auch nicht geändert), aber jetzt wird zusätzlich durch den drag zum vorherigen oder nächsten Bild navigiert xD
  • Zoom reparieren klingt aber nach vielleicht zu viel Aufwand? -> Sollte aber definitiv "notiert" werden.

Nee, das fixe ich nicht; da müsste man einen recht aufwändigen Algorithmus schreiben und das würde zu viel werden.

duevil commented 2 months ago

Kleine Sache noch:

  • Wenn das Image-Overlay aktiv ist, muss man dennoch erst durch die komplette Seite durchtabben bis man endlich an den Overlay Buttons angekommen ist.
  • Außerdem: esc zum schließen? xD

Ansonsten ist nur aus irgendeinem Grund halt der Zeitfilter jetzt buggy (das war er vorher scheinbar nicht) -> da würde ich also nochmal drüberschauen.

Hab die beiden Punkte noch fix umgesetzt. Das mit dem Zeitfilter ist doof, keine Ahnung was da los ist :/