Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Опечатка в закрывающем теге #327

Open
NtTel opened this Issue Mar 20, 2019 · 13 comments

Comments

Projects
None yet
4 participants
@NtTel
Copy link

commented Mar 20, 2019

echo '<h3>' . tf('Карта сайта (архив)') . '</h1>';

@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 20, 2019

Спасибо, исправлю.

@Sergeysem

This comment has been minimized.

Copy link

commented Mar 27, 2019

Максим, я об этом писал уже.. #322 (comment), но вы наверно не смотрели на скрины.
И вопрос, почему было принято решение заменить h1 на h3, ссылаясь на то, что бы размер был везде одинаковый?
Для этого есть стили ( я вам говорил в одном из вопросов, что нужно добавить br в код, вы мне сказали что есть стили, а тут вон оно как) и негоже удалять тег h1 со страницы.

Сделайте тег h1 c классом и кому не нравится размер, подправят его в стилях. Я не прав?

@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 28, 2019

Это служебные страницы. Изначально для них и был H3, но на некоторых я упустил этот момент и получилось, то на одних H1, как на полноценных страницах, а где-то H3, как на служебных. Поэтому я просто привёл всё к одному виду. И чисто визуально там не нужен большой заголовок. Пользы от H1 там нет никакой.

@Sergeysem

This comment has been minimized.

Copy link

commented Mar 28, 2019

Ну хорошо, тогда к h3 на этих страницах, можете добавить класс?

@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 28, 2019

Зачем? Используйте каскадность CSS.

@Sergeysem

This comment has been minimized.

Copy link

commented Mar 28, 2019

Нет!!! Придется менять h3 на h1 при каждом обновлении, и это только из за того, что автору понадобилось подгонять под какие то размеры шрифтов, таким непонятным способом.

@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 28, 2019

Хотите свою разметку? Копируйте файл в свой шаблон и там делайте что душе угодно. Автор же привёл вёрстку к одному виду. Жаль, что вы этого не хотите понять.

@akaguny

This comment has been minimized.

Copy link

commented Mar 30, 2019

с точки зрения семантики h1 на странице таки должен быть.
опционально h1 в html5 секции может быть в секции свой.
применение hN для стилизации в корне не верно, hN применяется для структуризации документов.
назначение страницы не имеет значения в вопросе разметки если она будет показана пользователю.

@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 30, 2019

Ну, ну... Приведите-ка ссылку на документацию, где написано, что H1 обязательный заголовок и «должен быть».

@akaguny

This comment has been minimized.

Copy link

commented Mar 30, 2019

с точки зрения семантики h1 на странице таки должен быть.

Речь не про обязательность)

Если нужна ссылка на документацию, то https://www.w3.org/TR/2011/WD-html5-author-20110705/the-h1-h2-h3-h4-h5-and-h6-elements.html. Однако я уверен что ты более чем знаком с ней)
Могу также скинуть ссылку на MDN в статье явно говорится:

* Heading information may be used by user agents, for example, to construct a table of contents for a document automatically.
* Не используйте низкие уровни чтобы снизить размер шрифта: используйте CSS font-size вместо.
Избегайте пропуск уровней заголовков: всегда начинайте с <h1>, потом используйте <h2>, и так далее.
* Вам следует рассмотреть избегание использования <h1> более раза на страницу. Смотрите Defining sections в Using HTML sections and outlines.
@maxsite

This comment has been minimized.

Copy link
Owner

commented Mar 30, 2019

Как видите, в документации HTML5 (MDN — это не стандарт) нет ни слова про обязательность H1 (N) или их очерёдность. Есть только «ранг заголовка» и в данных страницах он и должен быть понижен, чтобы дать возможность нижнему тексту получить более высокий приоритет. Если вы не согласны, то никто вам не мешает сделать свой вариант.

@akaguny

This comment has been minimized.

Copy link

commented Apr 8, 2019

Как видите, в документации HTML5 (MDN — это не стандарт) нет ни слова про обязательность H1 (N) или их очерёдность.

Очень даже есть:
@maxsite Если походить по ссылкам из спецификации, точнее по одной, где описана семантическое значение заголовков то там есть не прозрачный намёк на то как надо использовать и примеры по использованию:

Sections may contain headings of any rank, but authors are strongly encouraged to either use only h1 elements, or to use elements of the appropriate rank for the section's nesting level.

Authors are also encouraged to explicitly wrap sections in elements of sectioning content, instead of relying on the implicit sections generated by having multiple headings in one element of sectioning content.

https://www.w3.org/TR/2011/WD-html5-author-20110705/headings-and-sections.html#headings-and-sections

Есть только «ранг заголовка» и в данных страницах он и должен быть понижен

на самом деле при отсутствии других заголовков более высокого ранга по спецификации ваш hN+1 будет самый высокий не уступающий h1 в случае его наличия.

@maxsite

This comment has been minimized.

Copy link
Owner

commented Apr 8, 2019

Вообще-то здесь речь идёт об Headings and sections — размещении заголовках в секциях. Рекомендации для тех случаев, когда заголовки разбивают тексты и когда каждая секция не отмечена специально. Вам же приведены примеры разметки, где смешаны заголовки и секции. Посмотрите, поизучайте. Я абсолютно уверен, что вы даже не потрудились посмотреть какая в sitemap генерируется html-разметка и почему будет правильным понизить заголовок страницы до уровня остальных заголовков этой страницы.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.