Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Artykuł o łaziku #23

Merged
merged 4 commits into from
Sep 15, 2023
Merged

Artykuł o łaziku #23

merged 4 commits into from
Sep 15, 2023

Conversation

AdamGetBackToWork
Copy link
Collaborator

5 - Czy wymiary sa ok?
6,7 - Czy svg da rade wrzucic? wiadomo fajnie ze wektorowe wiec na plus ale jak nie da sie to zrobie z tego jpg'a i tez bedzie

5 - Czy wymiary sa ok?
6,7 - Czy svg da rade wrzucic? wiadomo fajnie ze wektorowe wiec na plus ale jak nie da sie to zrobie z tego jpg'a i tez bedzie
@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

@AdamGetBackToWork
5 wydaje się mieć dobre wymiary
Co do 6 i 7 to svg powinno działać, tylko pamiętaj, że musisz jeszcze dodać gdzieś wyświetlanie tych obrazków, jeśli chcesz, aby były widoczne :)
Nie wiem, gdzie chcesz aby się pojawiły, może pod samym opisem łazika.

@AdamGetBackToWork
Copy link
Collaborator Author

No tak tak, myślałem ze je zawrę w artykule na wzór selfie, tak pomiędzy tekstem itp. Dlatego chce je najpierw dodać by później móc do nich path wskazać :)

@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

Svg na 98% będzie działać.
tak btw to planujemy potem dodać naszą stronkę do jakiegoś contant generatora, aby można było edytować w przeglądarce z pogdglądem: #3 , ale to będzie dopiero potem.
Na ten moment można to robić tylko lokalnie jeśli zainstalujesz sobie apkę hugo, która generuje stronkę na bieżąco

@pktiuk pktiuk marked this pull request as draft September 14, 2023 08:28
@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

Zmieniłem też status na draft. Kiedy twoim zdaniem będzie to wszystko gotowe do zmergowania to kliknij Ready for Review

@AdamGetBackToWork
Copy link
Collaborator Author

O to extra z tym generatorem!
A z tym draftem to do tego pulla też później dodać tą zmienioną stronę łazika?

@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

Wszystko dodawaj w tym jednym pullu, jako jedną całość. Będzie wygodniej niż robić po kawałku.

@AdamGetBackToWork
Copy link
Collaborator Author

OK :)

Powinno być wszystko ok, korzystałem z Hugo - fajne narzędzie :). 
Wiadomo, jeśli jakieś uwagi, merytoryczne lub też nie, to śmiało!
mala pomylka
@AdamGetBackToWork AdamGetBackToWork marked this pull request as ready for review September 14, 2023 10:06
@AdamGetBackToWork
Copy link
Collaborator Author

Jak coś bym źle robił/robił bałagan z tym mergowaniem commitami etc. to mów proszę, to nieintencjonalnie. Pierwszy raz pracuje nad tego typu wspólnym projektem na Githubie (dużym i tak ustrukturyzowanym), też Gitlab to jednak coś ciut innego

@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

Po południu spojrzę, ale na pierwszy rzuto oka wszystko wygląda dobrze

content/project/hal.md Outdated Show resolved Hide resolved
content/project/hal.md Outdated Show resolved Hide resolved
@pktiuk
Copy link
Contributor

pktiuk commented Sep 14, 2023

Technicznie wszystko dobrze pododawane, ale dodałem dwa komentarze co do treści. :)

@pktiuk pktiuk changed the title Rover new jpg's Artykuł o łaziku Sep 14, 2023
@AdamGetBackToWork
Copy link
Collaborator Author

Dobra, pozmieniam co wskazałeś, uważam ze masz racje, dzieks

@pktiuk pktiuk merged commit e898baa into main Sep 15, 2023
@pktiuk pktiuk deleted the AdamGetBackToWork-patch-1 branch September 15, 2023 07:24
@pktiuk
Copy link
Contributor

pktiuk commented Sep 15, 2023

Za kilka minut pojawi się na stronie

@pktiuk
Copy link
Contributor

pktiuk commented Sep 15, 2023

Jak będziesz miał jakieś poprawki to po prostu zrób nowego pull requesta

@AdamGetBackToWork
Copy link
Collaborator Author

Pewnie, dzieki :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants