Skip to content

исправил размеры картинок, текс, и др. - #1

Open
kirill-sotnikov wants to merge 4 commits into
masterfrom
Kirill
Open

исправил размеры картинок, текс, и др.#1
kirill-sotnikov wants to merge 4 commits into
masterfrom
Kirill

Conversation

@kirill-sotnikov

Copy link
Copy Markdown
Member

No description provided.

@sotnikov-link sotnikov-link left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  1. Визуально вёрстка кривая
  2. Бардак в файлах: до сих пор indeXX.html, ненужных фалов куча
  3. Код бездумный
  4. Мокап не соответсвует макету: размеры не те; качество изображения ужасное

Какой-то подгон сплошной. Проще заново сделать, чем разбираться в этом. Готов помочь лично.

Нужно сначала навести порядок в файлах, затем сделать правильный мокап, а потом приступать к перевёрстке.

Comment thread index.html Outdated
<td style="font: normal 16px/1.25em Arial, sans-serif; padding:9px 20px 6px 20px; vertical-align: top;border-spacing: 0;">
<p style="padding:22px 0 0 0;margin:0;">Привет,друзья!</p><p style="margin:0;padding: 12px 0 0 0;">Cредний возраст основателей стартапов в Акселераторе 35+. Сегодня
расскажу подробнее о типах основателей, которые к нам приходят,
их <br> запросах, и как программа помогает решить их проблемы в бизнесе.</p>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

1.) убрать все br
2.) Сделать правильные переносы текста

Comment thread index.html Outdated
<p style="padding:22px 0 0 0;margin:0;">Привет,друзья!</p><p style="margin:0;padding: 12px 0 0 0;">Cредний возраст основателей стартапов в Акселераторе 35+. Сегодня
расскажу подробнее о типах основателей, которые к нам приходят,
их <br> запросах, и как программа помогает решить их проблемы в бизнесе.</p>
<p style="padding:10px 0 0 0;margin: 0;">Для проектов из сферы VR/AR собрали цифры по объему рынка

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Если здесь педдинг 10 пикс., то и в параграфе выше должно быть столько же

Comment thread index.html
<br>виртуальной и дополненной реальности
</td>
</tr>
<tr>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image
Кирилл:
1.) внимательно посмотри на структуру этого блока, сделай свой по аналогии
2.) Два последующих блока (Вакансии от Ильи...., Фонд русской экономики....) сделай по аналогии с этим блоком.
Примечание: Обрати внимание, высота пикселей над картинкой рассчитана не в пикселях а в емах от основного шрифта текста в блоке. На эту тему было видео, попробую найти.
3.) Разберись с ШИРИНОЙ ячеек (td) в таблице. суммарно они должны соответствовать общей ширине таблицы не учитывая педдинги (540 пикс.)

Comment thread index.html Outdated
с высоким потенциалом, целями и результатами. Это
возможность реализовать свой проект и войти
в сообщество ведущих предпринимателей и руководителей,
<br>а также приобреcти единомышленников, друзей

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Опять br. Удаляй. Для того чтобы сцепить слова используй Ctrl+Alt+Space (неразрывный пробел)

Comment thread index.html
</td>
</tr>
<tr>
<td style="padding: 0 20px 0 20px;border-spacing: 0">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image
Нарушена общая структура. Нет педдинга справа

Comment thread index.html
</tr>
</table>
</body>
</html> No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Где пустая строка?

Comment thread index.html
</tr>
<tr>
<td style="padding: 0;">
<table

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image
Опять нарушена структура

Comment thread index.html
<td style="padding:0px 24px 0px 0px ">
<table style="border-spacing: 0;">
<tr>
<td style="padding: 0">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image
Размер иконки 23 пикс., размер ячейки таблицы 25 пикс. Надо приравнять к 23!

@sotnikov-link sotnikov-link left a comment

Copy link
Copy Markdown
Member

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.

3 participants