Pr/mainpage #5

2022-07-31T18:59:04Z

Ota-Prokopec commented

2022-07-31 18:59:04 +00:00

(Migrated from github.com)

Nevěděl jsem jak mám přesně pojmenovat všechny komponenty, které jsem vytvořil, tak jsem je uložil do src/lib/Ota/. V routes jsem nechal 2 složky, jedna game.svelte je komponenta a try.svelte je ukázka jak by mohla jednotlivá stránka fungovat. Na poslední zobrazenou fotku můžete kliknout a zobrazí se vám všechny zbívající. Jesliže budete koukat na zařízení s malým width tak se vám zobrazí jen 2 fotky, myslím si, že to není nejhezčí a řešení by bylo to udělat, že by se za nimi zobrazila ještě jedna, která by byla malinko ořízlá a celý wrap (imageSlide) by byl overflow-x: auto.

Nevěděl jsem jak mám přesně pojmenovat všechny komponenty, které jsem vytvořil, tak jsem je uložil do src/lib/Ota/. V routes jsem nechal 2 složky, jedna **game.svelte** je komponenta a **try.svelte** je ukázka jak by mohla jednotlivá stránka fungovat. Na poslední zobrazenou fotku můžete kliknout a zobrazí se vám všechny zbívající. Jesliže budete koukat na zařízení s malým width tak se vám zobrazí jen 2 fotky, myslím si, že to není nejhezčí a řešení by bylo to udělat, že by se za nimi zobrazila ještě jedna, která by byla malinko ořízlá a celý wrap (imageSlide) by byl overflow-x: auto.

lewis-wow (Migrated from github.com) reviewed 2022-08-01 19:44:30 +00:00

lewis-wow (Migrated from github.com) left a comment

Dost hezky vypadající stránka a i přehledný kód. Super 😇

src/lib/Image.svelte

lewis-wow (Migrated from github.com) commented

2022-08-01 19:37:06 +00:00

Není potřeba dávat min-width kde je nastaveno width.

src/lib/Ota/Bubble.svelte

lewis-wow (Migrated from github.com) commented

2022-08-01 19:37:29 +00:00

Element bubble neexistuje, buď div nebo section.

src/lib/Ota/ImageSlider.svelte

lewis-wow (Migrated from github.com) commented

2022-08-01 19:38:42 +00:00

styl normálně v attributu

style="overflow-y: {overfow}"

styl normálně v attributu ``` style="overflow-y: {overfow}" ```

src/lib/Ota/ImageSlider.svelte

													
				@ -0,0 +24,4 @@

					{/each}

				</div>

				<style>

lewis-wow (Migrated from github.com) commented

2022-08-01 19:41:28 +00:00

Není to vyžadováno, ale bude se ti lépe psát kód. Koukni na SCSS preprocessor.

<style lang="scss">

Je možné "nestit" css selectory

.imageSlider {
    // styles...
   .imageCounter {
      // styles...
   }
}

Není to vyžadováno, ale bude se ti lépe psát kód. Koukni na SCSS preprocessor. ``` <style lang="scss"> ``` Je možné "nestit" css selectory ``` .imageSlider { // styles... .imageCounter { // styles... } }

src/lib/Ota/Sharebutton.svelte

													
				@ -0,0 +12,4 @@

				</script>

				<div on:click={share}>

						<IconShare />

lewis-wow (Migrated from github.com) commented

2022-08-01 19:41:52 +00:00

Ikonka není uprostřed tlačítka.

src/routes/try.svelte

lewis-wow (Migrated from github.com) commented

2022-08-01 19:43:36 +00:00

Zkus udělat aby při skrolování byl obrázek a share button fixed (aby se nehýbaly) pouze ten overlay. Možná ten Overlay chceme spíš tak jako jsem dělal já v ./routes/game.svelte, dovnitř by se vkládal text a další elementy.

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:34:19 +00:00

src/lib/Image.svelte

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:34:19 +00:00

Už jsem ho vymazal

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:34:29 +00:00

src/lib/Ota/Bubble.svelte

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:34:28 +00:00

Opravil jsem ho na div

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:34:36 +00:00

src/lib/Ota/ImageSlider.svelte

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:34:36 +00:00

Opravil jsem

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:35:12 +00:00

src/lib/Ota/ImageSlider.svelte

													
				@ -0,0 +24,4 @@

					{/each}

				</div>

				<style>

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:35:12 +00:00

Kouknu na to, vypadá to hodně dobře pro jednoduchost a přehlednost kódu, děkuji.

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:35:52 +00:00

src/lib/Ota/Sharebutton.svelte

													
				@ -0,0 +12,4 @@

				</script>

				<div on:click={share}>

						<IconShare />

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:35:52 +00:00

Opravil jsem to vymazáním , který neměl žádný styl. Div ikonku vycentruje

Opravil jsem to vymazáním <span>, který neměl žádný styl. Div ikonku vycentruje

Ota-Prokopec (Migrated from github.com) reviewed 2022-08-04 13:39:39 +00:00

src/routes/try.svelte

Ota-Prokopec (Migrated from github.com) commented

2022-08-04 13:39:39 +00:00

Skrolování jsem změnil a jak si to udělal ty, tak takový podobný jsem udělal v game.svelte, ale try.svelte je jen pro to jak by to mohlo být dále ulehčené, jak by to mohlo pro jednotlivé lokality být, aby to bylo jednoduší pro toho kdo tam bude cpát proměnné ze serveru. Ale jestli chcete tak to udělám v game.svelte více s proměnnými a nebude to jako svelte komponenta, aby se to dalo více editovat.