layout rework for landscape orientation

Practice-Looper / practice-looper

The Practice Looper app puts an end to the constant rewinding to loop only parts of your favourite songs from different audio sources - including songs from your Spotify Premium account.

https://www.practice-looper.com

0 stars 0 forks source link

layout rework for landscape orientation #161

Closed simonsymhoven closed 3 years ago

simonsymhoven commented 3 years ago

@emka3 Habs gefixt. Im Landscape hing der in einer Dauerschleife: OnAppearing() wird ausgeführt, dann die ContextAction auf der Default Loop gelöscht und dadurch vermutlich die Zelle neu geladen und OnAppearing() wieder aufgerufen. Habe jetzt die Bedingungen ContextActions?.Count != 0 noch hinzugefügt, damit wird der nur einmal gecleart und die Zelle anschließend vernünftig dargestellt. Wieso das nur im LS passiert kapier ich nicht aber nun gut..

simonsymhoven commented 3 years ago

@emka3 Bin durch, rebase erledigt. Von meiner Seite aus passts.

emka3 commented 3 years ago

Habe noch einige Kleinigkeiten gefunden und gefixt. Landscape sieht soweit gut aus, bis auf den Picker. Wäre cool, wenn die Icons untereinander wären - können auch gerne zusammen drüber schauen :)

IMG_463A6F97970A-1

simonsymhoven commented 3 years ago

Was meinst du mit untereinander? Hatte das extra so umgebaut das der Inhalt als solches zentriert ist und nicht in dem Grid die icons untereinander :D

emka3 commented 3 years ago

Das wirkt so unruhig. Findest nicht?

simonsymhoven commented 3 years ago

Joah schon^^ ich bau es später mal um

emka3 commented 3 years ago

Joah schon^^ ich bau es später mal um

Das wäre der letzte Punkt aus meiner Sicht. Wenn das durch ist, können wir die Tests angehen 😎.

simonsymhoven commented 3 years ago

@emka3 Sodele: Simulator Screen Shot - iPhone 12 Pro - 2021-02-18 at 12 47 30

emka3 commented 3 years ago

@emka3 Sodele:

Dann können wir den nun mergen!