doka-guide / platform

Платформа Доки: шаблоны, стили, скрипты и всё для сборки сайта
https://doka.guide
MIT License
237 stars 65 forks source link

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

Closed vitya-ne closed 1 month ago

vitya-ne commented 1 month ago

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

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

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

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

skorobaeus commented 1 month ago

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

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

      {% 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 commented 1 month ago

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

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

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

vitya-ne commented 1 month ago

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

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

github-actions[bot] commented 1 month ago

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

skorobaeus commented 1 month ago

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

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

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

TatianaFokina commented 1 month ago

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