Skip to content

add hero section#2753

Open
nikitinaanna47 wants to merge 5 commits into
mate-academy:masterfrom
nikitinaanna47:develop
Open

add hero section#2753
nikitinaanna47 wants to merge 5 commits into
mate-academy:masterfrom
nikitinaanna47:develop

Conversation

@nikitinaanna47

Copy link
Copy Markdown

Nothing Landing Page

A responsive landing page for the Nothing brand, built based on the provided Figma design. The project is implemented using modern HTML and SCSS with a mobile-first approach.

Technologies

  • HTML5
  • SCSS (Sass)
  • BEM methodology
  • JavaScript (ES6)
  • Vite

Preview

Design

  1. Clone the repository:
    git clone https://github.com/nikitinaanna47/layout_miami.git

@zibi-dev zibi-dev 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.

Hej. DEMO LINK nie dziala. Czy uruchomilas deploy?

@nikitinaanna47

Copy link
Copy Markdown
Author

Hej! Poprawiłam konfigurację ścieżki do obrazka i zrobiłam ponowny deploy. DEMO LINK powinien już działać poprawnie.

@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.

jest to dobry start więc powodzenia z dalszą pracą nad projektem i pamiętaj by oznaczać projekt do ponownego sprawdzenia ponieważ inaczej nie widzimy twoich komentarzy

Image

Comment on lines +13 to +24
&--secondary {
font-family: "Space Grotesk", sans-serif;
font-weight: 700;
font-size: 32px;
font-style: normal;
line-height: 32px;
letter-spacing: 0;

display: flex;
align-items: center;
color: #292929;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

modyfikatorowi wystarczą tylko te style które chcesz dodać lub zmienić względem oryginalnej klasy (w tym wypadku title) ponieważ element i tak będzie posiadał klasę na którą nakładasz modyfikator więc nie potrzeba powtarzać tych styli :) dodatkowo dobrze jest już teraz stworzyć zmienne na wszystkie potrzebne kolory w projekcie i to je wykorzystywać

Suggested change
&--secondary {
font-family: "Space Grotesk", sans-serif;
font-weight: 700;
font-size: 32px;
font-style: normal;
line-height: 32px;
letter-spacing: 0;
display: flex;
align-items: center;
color: #292929;
}
&--secondary {
font-size: 32px;
line-height: 32px;
}

Comment thread .gitignore

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nie zmieniaj plików konfiguracyjnych takich jak gitignore i eslintignore. możesz ponownie pobrać te pliki z brancha master i zastąpić je w swoim projekcie aby wróciły one do swojej pierwotnej wersji

Comment thread styles.css

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

usuń ten pusty plik. pamiętaj że nie potrzebujesz plików css ponieważ otwierając projekt w przeglądarce za pomocą komendy npm start to parcel będzie kompilować pliki scss

Comment thread burger-menu-hover.svg

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

dla lepszej organizacji projektu utwórz sobie osobny folder na wszystkie ikony najlepiej w tym samym miejscu co folder images albo wewnątrz niego

Comment on lines +16 to +19
&__icon {
width: 24px;
height: 24px;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

te style nie są używane ponieważ zagnieździłaś __icon wewnątrz __icons więc utworzył ci się selektor navigation__icons__icon. możesz to też zauważyć przy pomocy dev tools bo widać że element navigation__icon nie posiada tych styli

Image

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

powinnaś użyć pełnej wersji zdjęcia aby przy zmianie szerokości odkrywać większą część zdjęcia zamiast tylko całość powiększać

Image

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