Skip to content

feat: Practice HTML and CSS Basics#215

Open
zanetasochon wants to merge 10 commits intodevmentor-pl:masterfrom
zanetasochon:master
Open

feat: Practice HTML and CSS Basics#215
zanetasochon wants to merge 10 commits intodevmentor-pl:masterfrom
zanetasochon:master

Conversation

@zanetasochon
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Owner

@devmentor-pl devmentor-pl left a comment

Choose a reason for hiding this comment

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

Żaneto,

Zadanka są jak najbardziej ok 👍
Zostawiłem parę uwag w komentarzach, wszystko wyjaśnimy na spotkaniu :)

Comment thread 01/index.html
</footer>
</main>
</body>
</html>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Comment thread 02/styles/index.css
height: 230px;
} No newline at end of file
height: 230px;
}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Comment thread 03/styles/fonts.css
src: url(./fonts/source-sans-pro/SourceSansPro-It.otf) format('truetype');
font-style: italic;
font-weight: 400;
}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Comment thread 03/styles/index.css
font-family: 'Source Sans PRO';
}

.child--first {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Brakuje odstępów, o których mowa w zadaniu tj. np. margin-right: 10px, wtedy szerokość elementy można ustawić jako width: calc(30% - 10px);

Comment thread 03/index.html
Lorem ipsum dolor sit amet consectetur adipisicing elit. Nobis praesentium in possimus!</div>
<div class="child child--last">
Lorem ipsum dolor sit amet consectetur adipisicing elit. Delectus, iure?</div>
<div class="child child--first">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Trzeba pamiętać, że w "inline", również entery wliczają się w odstępy więc mamy 100% + entery, co jest więcej niż 100% i to powoduje, że elementy się nie mieszczą.

Comment thread 04/index.html
<nav class="container">
<ul class="nav">
<li class="nav__item">HOME</li>
<li class="nav__item nav__item--focused">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Bardziej chodziło, aby był efekt :hover po najechaniu :)

Comment thread 04/index.html
<li class="first-level__item__one">Item 1</li>
<li class="first-level__item__two">
Item 2
<ul class="nav__item__second-level">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

W BEM nie stosujemy wielokrotnych podkreśleń maksymalnie co możemy zrobić to Block__Element--modyfikator.
Proponuję zapoznać się z tym artykułem: https://devmentor.pl/b/metodologia-bem-w-css-i-sass - myślę, że pomoże ;)

Comment thread 05/index.html
</div>
</footer>
</body>
</html>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Comment thread 05/styles/index.css

.orange-container__footer-item {
width: 150px;
}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

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