fix(code): Add ability to disable SEO optimization for titles #555
Conversation
| const linkTokens = createLinkTokens(state, customId, anchorTitle, setId, href); | ||
|
|
||
| inlineToken.children?.unshift(...linkTokens); | ||
| if (!disableSEOFixForTitles) { |
There was a problem hiding this comment.
Думаю ты хотел не этого.
В данном случае ты полностью отключил генерацию ссылок на якорях.
СЕО оптимизация лежит внутри createLinkTokens и она просто добавляет текст. Стоит точечно отключить именно ее.
Либо, если все-таки нужно отключить генерацию ссылок, обрати внимание, что ниже в
if (supportGithubAnchors) {
генерируется еще одна
There was a problem hiding this comment.
Да, нужно именно ссылки удалить (они не имеет смысла без контента), там просто тогда пустая ссылка будет.
А вот по поводу supportGithubAnchors я думал что
supportGithubAnchors != SEOAnchorFix
Могу вырезать supportGithubAnchors при флаге disableSEOFixForTitles. Тут просто как скажешь
There was a problem hiding this comment.
Ну по сути тогда это логика disableAnchors.
Но имей в виду, что пустые ссылки имеют смысл.
На этих ссылках генерируется параметр id, за счет чего к одному и тому же месту в документе можно подскроллиться по разным якорям. (Один якорь на h1, остальные здесь же на невидимых ссылках)
There was a problem hiding this comment.
Я за то чтобы флаг disableAnchorsHiddenContent (удалял anchor и контент) только для первой ветки
Если я захочу удалить anchors для Github я выставлю флаг supportGithubAnchors: false
| @@ -76,12 +76,14 @@ const removeCustomIds = (token: Token) => { | |||
| interface Options { | |||
| extractTitle?: boolean; | |||
| supportGithubAnchors?: boolean; | |||
| disableSEOFixForTitles?: boolean; | |||
In out service we use YFM for email content. To be exact we compile YFM to HTML and then send it to email. The problem is that we don't inline or send CSS with email so hiddenDesc for title is visible to user and user sees 2 titles.
This PR allowed us to disable this SEO optimisation with
disableSEOFixForTitlesoption.disableSEOFixForTitles: false (default)
disableSEOFixForTitles: true