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

Исправляет разметку списка ответов на странице участника #1293

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

vitya-ne
Copy link
Contributor

Ошибка:
Допустим вклад участника в какой-либо раздел (HTML, CSS, и тп) это только ответы на вопросы собеседования (см. isOnlyWithAnswer() в файле person.11tydata.js)
В этом случае разметка списка для этой категории включает цикл по ответам участника (answersInArticles) состоящим из <ul>. Так как сами <ul> образуют список, требуется родительский элемент для установки отступов между ними.

Та же ситуация произойдёт если вклад участника в какую-либо категорию это только советы (см. isOnlyWithPractice() в файле person.11tydata.js)

Исправляет: #1234

Доп. тестовая страница для примера: /people/vitya-ne/ (см. вклад в раздел "Веб-платформа")

@skorobaeus
Copy link
Member

О-о-о, ты нашёл, где собака зарыта, спасибо тебе огромное!

Но подожди-подожди, а что если мы сделаем вот так?

      {% if isOnlyWithPractice %}
        {% for category in categoriesOnlyWithPractice %}
          <section class="person-page__group articles-group articles-group--accent articles-group--title-static" style="--accent-color: var(--color-{{ category }})">
            <h2 class="articles-group__title">
              {{ articleIndexesMap[category].data.name }}
            </h2>
            <h3 class="articles-group__sub-title font-theme font-theme--code">{{ practicesPersonRole }}</h3>
              <ul class="articles-group__list base-list">
                {% for practice in practicesIndex[category] | sort %}
                <li>
                  <a class="articles-group__link link" href="/{{ category }}/{{ practice.fileSlug }}/#na-praktike">
                    {{ practice.data.title | descriptionMarkdown | safe }}
                  </a>
                </li>
                {% endfor %}
              </ul>
          </section>
        {% endfor %}
      {% endif %}
      {% if isOnlyWithAnswer %}
        {% for category in categoriesOnlyWithAnswers %}
          <section class="person-page__group articles-group articles-group--accent articles-group--title-static" style="--accent-color: var(--color-{{ category }})">
            <h2 class="articles-group__title">
              {{ articleIndexesMap[category].data.name }}
            </h2>
            <h3 class="articles-group__sub-title font-theme font-theme--code">{{ answersPersonRole }}</h3>
              <ul class="articles-group__list base-list">
                {% for articleList in answersInArticles[category] %}
                  {% for article in articleList | sort %}
                    <li>
                      <a class="articles-group__link link" href="/{{ category }}/{{ article.fileSlug }}/#na-sobesedovanii">
                        {{ article.data.title | descriptionMarkdown | safe }}
                      </a>
                    </li>
                  {% endfor %}
                {% endfor %}
              </ul>
          </section>
        {% endfor %}
      {% endif %}

Я перенесла конструкции for in внутрь <ul> и... кажется... работает???

@vitya-ne
Copy link
Contributor Author

Но подожди-подожди, а что если мы сделаем вот так?

надо потестить )

у меня вот какое сомнение:
есть ответы/вопросы кторые привязаны к нескольким статьям. Сейчас кол-во ссылок === кол-ву статей с ответом.
Тоесть, если один вопрос виден в двух статьях, то и на странице участника в списке ответов мы увидем две ссылки, хотя фактически, ответ один. Это ок ?

@vitya-ne
Copy link
Contributor Author

Но подожди-подожди, а что если мы сделаем вот так?

@skorobaeus, ты права, исправил

Copy link

Превью контента из 0c1da9b опубликовано.

@skorobaeus
Copy link
Member

Ур-р-ра, я рада, что работает! Спасибо, что нашёл отвечающий за всё это код ❤

есть ответы/вопросы кторые привязаны к нескольким статьям. Сейчас кол-во ссылок === кол-ву статей с ответом.
Тоесть, если один вопрос виден в двух статьях, то и на странице участника в списке ответов мы увидем две ссылки, хотя фактически, ответ один. Это ок ?

Мне кажется, да. Если нам ок выводить один и тот же вопрос на нескольких страницах, то почему не ок выводить ссылки на все эти страницы? Есть ощущение, что пока контрибьюты на странице участника не конвертируются напрямую в докакоины, это не слишком принципиальный вопрос :) я была бы самая богатая, хаха

@TatianaFokina
Copy link
Member

Е-е-е, наконец-то больше не будет списка в списке в списке. Спасибо большое тебе!

@TatianaFokina TatianaFokina added ошибка Что-то критичное и мешает вёрстка Задачи по вёрстке labels Sep 2, 2024
@TatianaFokina TatianaFokina merged commit 2a73321 into doka-guide:main Sep 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
вёрстка Задачи по вёрстке ошибка Что-то критичное и мешает
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants