Skip to content

Develop - #668

Open
Namart wants to merge 3 commits into
mate-academy:masterfrom
Namart:develop
Open

Develop#668
Namart wants to merge 3 commits into
mate-academy:masterfrom
Namart:develop

Conversation

@Namart

@Namart Namart commented Mar 23, 2021

Copy link
Copy Markdown

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

Nice!

Comment thread src/index.html Outdated
<!-- <h1>Enclosures</h1> -->
<div id="qa-block" class ="outer">
<div class="med">
<div class="inner">In programming languages, a closure is a technique for implementing lexically scoped name binding in a language with first-class functions. Operationally, a closure is a record storing a function together with an environment.</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

don't make lines longer than 80 symbols

Comment thread src/style.css
}

.outer {
box-sizing: border-box;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

set box-sizing: border-box to all the elements with "*" selector

Comment thread src/index.html Outdated
</div >
<div class ="outer">
<div class="med">
<div class="inner">У програмуванні замиканням називають підпрограму, що виконується в середовищі, яке містить одну або більше зв’язаних змінних. Підпрограма має доступ до цих змінних під час виконання.</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
<div class="inner">У програмуванні замиканням називають підпрограму, що виконується в середовищі, яке містить одну або більше зв’язаних змінних. Підпрограма має доступ до цих змінних під час виконання.</div>
<div class="inner">
У програмуванні замиканням називають підпрограму, що виконується в середовищі, яке містить одну
або більше зв’язаних змінних. Підпрограма має доступ до цих змінних під час виконання.
</div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

fix it like this everywhere

Comment thread src/style.css
padding: 10px;
margin: 40px 20px;
background-color: #0090c2;
border: 1px solid #e95d49;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

if border is the same for all the containers, then why not create another class for all the containers and write border there?

@aionova aionova 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!

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