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

feat: add footer #40

Closed
wants to merge 74 commits into from
Closed

feat: add footer #40

wants to merge 74 commits into from

Conversation

jedryczkatymon
Copy link
Member

No description provided.

@jedryczkatymon
Copy link
Member Author

gadałem z Konradem i doszedłem do wniosku, że zrobię przysługę sobie i innym rozdzielając to wszystko na osobne komponenty bo przyznaję się bez bicia stylowanie kodu z pierwszego commita mogłoby być trochę... uciążliwe to say the least

@kguzek
Copy link
Member

kguzek commented Dec 14, 2024

List nie musi być zagnieżdżony w Footer, więc sobie przenieś do góry - inaczej funkcja jest definiowana ponownie za każdym razem gdy footer jest renderowany.

Dodaj też export do footera a tak na marginesie to na dc była dyskusja o tym że wolimy function od () => 👀👀

dobra nie widziałem że na dole masz export { Footer }

@kguzek
Copy link
Member

kguzek commented Dec 14, 2024

możesz sobie też do ListItem dać domyślną wartość argumentu target jako "_blank" żeby nie musieć podawać za każdym razem

@jedryczkatymon
Copy link
Member Author

oo okej wsm słuszne uwagi dzięki

src/components/Footer.tsx Outdated Show resolved Hide resolved
Octopus4344
Octopus4344 previously approved these changes Dec 15, 2024
Copy link
Collaborator

@Octopus4344 Octopus4344 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spoko

src/components/Footer.tsx Outdated Show resolved Hide resolved
src/components/Footer.tsx Outdated Show resolved Hide resolved
src/components/Footer.tsx Outdated Show resolved Hide resolved
@jedryczkatymon
Copy link
Member Author

już chyba wszystko

kguzek
kguzek previously approved these changes Dec 16, 2024
Copy link
Member

@kguzek kguzek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jak dla mnie git, wszystkie uwagi od Poli uwzględnione więc chyba też dałaby approve. Jedyne co to label do linka do TikToka ma też napis "Instagram" ale to i tak tylko placeholdery więc kiedyś się i tak zmieni

@jedryczkatymon
Copy link
Member Author

jak dla mnie git, wszystkie uwagi od Poli uwzględnione więc chyba też dałaby approve. Jedyne co to label do linka do TikToka ma też napis "Instagram" ale to i tak tylko placeholdery więc kiedyś się i tak zmieni

jak już ma być porządnie to niech będzie porządnie XD

@Octopus4344
Copy link
Collaborator

Gap dla mobilki troche moglby byc wiekszy jeszcze bo srednio to wyglada poki co. Poza tym wszystko git

fix: adapt files to solvro config

fix: button

fix: delete redundant components

fix: delete redundant components
@jedryczkatymon jedryczkatymon deleted the feat/footer branch December 22, 2024 17:19
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.

4 participants