Bc. a Mgr. práce

Feedback k prvnímu odevzdání¶

• Kontakty
  - Styl hlavičky by měl být stejný jako u zprávy do rádia

• Zpráva do rádia
  - Navigace výběru typu zprávy do rádia může být trochu neintuitivní. Preferoval bych buď výběr způsobem jak bylo implementováno v aplikaci, nebo zpětnou navigaci implementovat skrze tlačítko "Zpět" v navigační hlavičce.
  - Nahrávání by se nemělo spouštět dokud uživatel nepovolí použití mikrofonu, taktéž pokud je již dříve zamítnuté. Uživatel by měl být upozorněn, že tato funkcionalita nefunguje bez povolení přístupu k mikrofonu v nastavení.
  - Dříve vytvořená nahrávka by se neměla přidávat ke zprávě, pro kterou nebyla vytvořena. Ideálně po odeslání soubor smazat.
  - Ukládání osobních údajů je v pořádku, ale ukládání textu zprávy do rádia je asi zbytečné.
  - Zobrazení náhledu vybrané fotky by mělo mít tlačítko "Zavřít", ne všichni mohou být zvyklí používat gesta.

• Homepage
  - Asi jsi zapomněl do projektu přidat barvy (assets), takže některým UI prvkům chybí správné barvy.
  - Uživatel by měl mít možnost vidět aktuální nastavení časovače a alarmu, také mít možnost zrušit aktuální nastavení.
  - Podobně jako u mikrofonu, uživateli by nemělo být umožněno pouze zdánlivě nastavit alarm bez požadovaného povolení notifikací. Měl by být upozorněn, že funkcionalita nebude fungovat bez autorizace.

• Novinky
  - Stejně jako u kontaktů, preferovaný styl hlavičky je takový, jaký je u zprávy do rádia.
  - V případě nedostupného internetu/pomalého připojení by bylo dobré zobrazovat indikaci načítání dat.
  - Při načtení dat novinek a dodatečného načítání obrázků se layout novinky rozbije (viz screenshot).

Jinak dobrá práce, projekt už funguje celý na základě SwiftUI.

... a ještě na tabletech se špatně zobrazuje program, výběr typu zprávy do rádia a načítací obrazovka při zapnutí aplikace. Textové pole by také mohlo být roztáhnuté na celou šířku.

Feedback k druhému odevzdání¶

• Zpráva do rádia
  - Stejný problém s navigací jako u odevzdání z minula. Pokud není jasné jak jsem to myslel, nebo jak se to liší vůči původní implementaci, můžeme probrat osobně.
  - Pokud uživatel přiložil audio zprávu, nemělo by být povinné zadávat i textovou zprávu.
  - Délka nahrávky by měla být omezena na 30 sekund. Zdá se, že se nahrávání při překročení limitu nezastaví a pokračuje dál dokud není manuálně zastaveno.
  - Zobrazení náhledu vybrané fotky by mělo mít tlačítko "Zavřít", ne všichni mohou být zvyklí používat gesta.
  - Nahrávání zprávy koliduje s poslechem rádia a trvale ztiší rádio. Nejlepší by asi bylo přehrávání pozastavit při vytváření nahrávky.
  - Font/odsazení v hlavičce je odlišné od ostatních stránek viz příloha.

• Homepage
  - Když nastavím časovač a chci ho rovnou přenastavit (ne nejprve zrušit), spinner na výběr se seká. Může souviset s překreslováním Menu elementu nastavení. Odpočet nutně nemusí být tak přesný, pokud by to dělalo problém, stačí když se zobrazí čas kdy doběhne.
  - Bylo by dobré zapracovat indikaci načítání rádia, případně pokud není vůbec internet k dispozici, upozornit uživatele. Aktuálně aplikace bez připojení k internetu při pokusu o spuštění přehrávání nic nedělá. (z osobní zkušenosti toto může být občas i důvod k zamítnutí uvolnění aplikace na App Store)
  - Logo metropole je nějaké malé.
  - Kvalita přenosu rádia by měla být uložená a zůstat stejná mezi vypnutím a zapnutím aplikace.
  - Nastavení alarmu zmizí z nastavení po ukončení aplikace, přestože je na pozadí aktivní.
  - Hláška ohledně nedostupné kvality je bez diakritiky v češtině a bez překladu do angličtiny. Také není jasné na první pohled co provede tlačítko "Potvrdit".
  - Aktuální program by měl mít uvedený i čas.
  - Šipka "Hrálo" má v tmavém režimu odlišnou barvu od ostatních prvků.

• Novinky
  - Obecně už vypadá v pořádku.

Jinak aplikace není přizpůsobena "landscape" orientaci na některých stránkách na mobilech (viz screenshoty). Tato orientace není požadovaná na mobilech, pokud tedy nechceš přizpůsobovat UI, změnu orientace prosím zruš. Na tabletech se textové pole pro zprávu do rádia neroztáhne na celou šířku v landscape orientaci. Tady je to ale zdá se jediný nedostatek, takže prosím uprav a pro tablety dále podporujme všechny orientace.

Feedback ke třetímu odevzdání¶

• Zpráva do rádia
  - Poslat zprávu do rádia by mělo být možné až po dokončení nahrávky. Aktuálně je možné kliknout na odeslat zprávu před dokončením nahrávání a způsobuje to hned několik problémů jako například předání chybného souboru e-mailovému klientovi a "zblbnutí" časovače při nové nahrávce.
  - Přehrávání rádia se správně pozastaví při spuštění nahrávání, ale ztišení rádia po vytvoření nahrávky stále přetrvává.
  - "Placeholder" v textovém poli zprávy do rádia naráží na nějaké problémy se zarovnáním (někdy je zarovnaný na střed, někdy je zbytečně ořezaný) viz screenshoty. Měl by být vždy roztáhnutý tak aby se vlezl a zarovnaný nahoře.
  - Autokorekci v poli pro e-mail, tel. číslo a jméno prosím zrušit, a pokud možno, nastavit odpovídající typ textových polí, aby je systém rozpoznával a mohl navrhovat vyplnění dle údajů uložených systémově.

• Homepage
  - Pokud spustím přehrávání, odejdu na jinou stránku a zase se vrátím, přehrávání by se nemělo zastavovat.
  - Indikace načítání je víceméně OK, ale reaguje na načtení programu, místo zvukové stopy. Bylo by ideální, kdyby reálnému stavu přehrávání odpovídalo ovládací tlačítko přehrávání.
  - Ověř prosím zasílání notifikací ohledně budíku. Po nastavení na daný čas mi přišly najednou dvě, jedna česky a jedna anglicky. Zobrazovat by se samozřejmě měla jenom jedna v závislosti na lokalizaci. Nevím jestli s tím souvisí, ale měnil jsem v nastavení lokalizaci pro konkrétní aplikaci, pokud by se ti to nedařilo nasimulovat, tak toto můžeš zkusit.
  - Pokud vyberu rádio Skyrock s nízkou kvalitou, upozornění se ukáže správně, ale aplikace nečeká a začne přehrávat. Toto je pravděpodobně zároveň způsobeno tím, že se zdá se zprovoznilo na backendu vysílání s nízkou kvalitou u Skyrock rádia. Prověřím s kolegou jestli budeme podporovat a dodatečně dám vědět, jak toto budeme řešit.

Ostatní vypadá v pořádku, včetně podpory tabletů.

Feedback ke čtvrtému odevzdání¶

• Zpráva do rádia
  - Poslat zprávu do rádia by mělo být možné až po dokončení nahrávky. Aktuálně je možné kliknout na odeslat zprávu před dokončením nahrávání a způsobuje to hned několik problémů jako například předání chybného souboru e-mailovému klientovi a "zblbnutí" časovače při nové nahrávce. Už se chová lépe, ale pokud přiložím nahrávku a odešlu zprávu a následně chci poslat další, tlačítko "odeslat" funguje i bez vytvoření nahrávky nebo přiložení textové zprávy, což opět způsobuje problémy jako např. možnost poslání nedokončené nahrávky a problém s časovačem.
  - Přehrávání rádia se správně pozastaví při spuštění nahrávání, ale ztišení rádia po vytvoření nahrávky stále přetrvává. Toto stále nefunguje, podívej se prosím na dokumentaci https://developer.apple.com/documentation/avfaudio/avaudiosession/category a https://developer.apple.com/documentation/avfaudio/avaudiosession/categoryoptions. Pokud bys něčemu nerozuměl nebo měl problém to nastavit, můžeme to projít společně.

Ostatní je OK.

Na základě posledního testování aplikace nyní splňuje domluvené požadavky ze zadání.

Dodatečně doporučuji za účelem vylepšení kvality a stylu kódu použít nástroj SwiftLint.