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

Упрощает разметку фида #23

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Conversation

pepelsbey
Copy link
Member

@pepelsbey pepelsbey commented Nov 8, 2024

Вероятно, fix #8

Упрощаю фид, чтобы его лучше парсили платформы, включая Ютуб.

  • Заголовок заменяется на параграф
  • Обёртка списка заменяется на параграф
  • Элемент списка заменяется на буллит в начале и <br> в конце

До сих пор получилось всё, кроме последнего пункта:

Было

<ul>
  <li><a href="https://nodejs.org/en/blog/release/v22.11.0">LTS Node.js 22</a></li>
  <li><a href="https://survey.devographics.com/en-US/survey/state-of-react/2024">State of React</a></li>
  <li><a href="https://github.com/CSS-Next/css-next/issues/105#issuecomment-2445341089">New CSS logo?</a></li>
</ul>

Стало

<p>
  <a href="https://nodejs.org/en/blog/release/v22.11.0">LTS Node.js 22</a>
  <a href="https://survey.devographics.com/en-US/survey/state-of-react/2024">State of React</a>
  <a href="https://github.com/CSS-Next/css-next/issues/105#issuecomment-2445341089">New CSS logo?</a>
</p>

Хочется

<p><a href="https://nodejs.org/en/blog/release/v22.11.0">LTS Node.js 22</a><br><a href="https://survey.devographics.com/en-US/survey/state-of-react/2024">State of React</a><br><a href="https://github.com/CSS-Next/css-next/issues/105#issuecomment-2445341089">New CSS logo?</a>
</p>

Пробовал что-то вроде:

const br = new state.Token('html_inline', '', 0);
br.content = '<br>';
state.tokens.splice(state.tokens.indexOf(token) + 1, 0, br);

Но что-то markdown-it не хочет меня понимать.

@pepelsbey pepelsbey requested a review from monochromer November 8, 2024 17:52
@pepelsbey
Copy link
Member Author

@monochromer есть идеи, как это провернуть? :)

@pepelsbey
Copy link
Member Author

Решил быть проще и упростить только заголовки :) Они больше всего мешали, может этого хватит.

@pepelsbey pepelsbey merged commit 287bb2a into main Nov 8, 2024
2 checks passed
@pepelsbey pepelsbey deleted the simplify-feed branch November 8, 2024 18:14
@monochromer
Copy link
Member

@monochromer есть идеи, как это провернуть? :)

Я бы такие штуки через DomParser делал

@pepelsbey
Copy link
Member Author

Я бы такие штуки через DomParser делал

Я заметил, что все эти трансформации работают сильно медленнее, хотя и удобнее/понятнее.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants