Skip to content

add header and styles to header#2751

Open
1Vladislav13 wants to merge 28 commits into
mate-academy:masterfrom
1Vladislav13:develop
Open

add header and styles to header#2751
1Vladislav13 wants to merge 28 commits into
mate-academy:masterfrom
1Vladislav13:develop

Conversation

@1Vladislav13

Copy link
Copy Markdown

@natalia-klonowska natalia-klonowska left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great start :D

Comment thread src/styles/blocks/title.scss Outdated
@@ -0,0 +1,6 @@
.title {
font-weight: 700px;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Image
Suggested change
font-weight: 700px;
font-weight: 700;

@natalia-klonowska natalia-klonowska left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

przestał działać ci link do demo:

Image

spróbuj jeszcze raz użyć komendy npm run deploy a jeśli to nie pomoże to sprawdź czy na pewno strona poprawnie się wyświetla przy pomocy komendy npm start

@danon321 danon321 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mobile

Image Tutaj twoj obrazek powinien byc równo z tekstami. Teraz masz tam na mobile dodany margin (na obrazku)

Tak to wszystko inne fajnie.

Czekam na gotową wersje Desktop :))

@1Vladislav13 1Vladislav13 requested a review from danon321 July 8, 2026 16:15
@1Vladislav13

Copy link
Copy Markdown
Author

może ktoś pomóc z sekciją 'heritage'? nie potrafię wyciągnąć tło na cały ekran, bez usunęcia padingów z 'content'. Być może jest jakiś hak? :)

@danon321 danon321 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Kawał dobrej roboty! Wszystko wyglada schludnie :))

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.

3 participants