document.first_line_indentation падает

[danilasar]

В этом документе
report.zip
из-за зависимости

#import "@preview/codelst:2.0.2": sourcecode

происходит ошибка в conf.typ, а именно на 454 строчке:

for (i, elem) in doc.children.enumerate() {

Компилятор ожидает, что doc имеет тип text, и у данного типа нет свойства children. У меня очень большие подозрения на 474-488 строки, где происходит рекурсивный вызов функции first_line_indentation:

else if element == sequence {
	(self.document.first_line_indentation)(
		self,
		elem,
		last-is-heading: last-is-heading,
		indent-already-added: indent-already-added,
	)
} else if element == styled {
	styled((self.document.first_line_indentation)(
		self,
		elem.child,
		last-is-heading: last-is-heading,
		indent-already-added: indent-already-added,
	), elem.styles)
}

first_line_indentation писал не я и я не особо понимаю в целом, почему здесь происходит рекурсивный вызов, но очень похоже, что elem (или elem.child) является text и, соответственно, его скармливание в качестве doc ни к чему хорошему не приводит. Здесь точно нужна рекурсия?

[danilasar]

@aragami3070 думаю, ты в этой функции более компетентен

[aragami3070]

Если честно я ее тупа с обсуждения на гите взял, как отчет доделаю, завтра чекну

[danilasar]

Пупупу, людям для отчёта как раз и нужно. Я тогда тоже сейчас гляну

[aragami3070]

капец
А зачем они этот модуль юзают?

[aragami3070]

мб я знаю как это по другому сделать, чтобы конфиг не править сейчас
@danilasar

[danilasar]

На будущее, я, спасибо всемогущему осинту, нашёл автора этого кода: typst/typst#311 (comment)

[danilasar]

капец А зачем они этот модуль юзают?

Чтобы сделать нумерацию строк кода в листингах

[aragami3070]

ща вроде видел норм вариант от самого typst

[aragami3070]

вроде нашел сейчас тестовый вариант накидаю, скину, если норм

[aragami3070]

https://typst.app/universe/package/algo/

попробуй algo, ту херню которую я начал тестить ты походу тоже видел, ибо я тоже на codelst при попытке загуглить вышел

[aragami3070]

@danilasar

[danilasar]

Я решил пойти внаглую и задал вопрос напрямую автору) В любом случае никто лучше него не объяснит назначение этих строк и не исключит в них ошибку. Кстати, параллельно гуглил решение по поводу красных строк и наткнулся на https://github.com/flaribbit/indenta, выглядит аппетитно

[danilasar]

https://typst.app/universe/package/algo/

попробуй algo, ту херню которую я начал тестить ты походу тоже видел, ибо я тоже на codelst при попытке загуглить вышел

Ща посмотрим

[aragami3070]

Я решил пойти внаглую и задал вопрос напрямую автору) В любом случае никто лучше него не объяснит назначение этих строк и не исключит в них ошибку. Кстати, параллельно гуглил решение по поводу красных строк и наткнулся на https://github.com/flaribbit/indenta, выглядит аппетитно

мб мб, посмотрим завтра, сейчас сложно параллельно два дела делать)

[danilasar]

наткнулся на https://github.com/flaribbit/indenta, выглядит аппетитно

@preview/indenta выглядит сырой и заставить работать с ходу не получилось. Есть 3 стула:

• Работает только на титульном листе
• Не работает в принципе
• Не компилируется и тоже падает на styled

в зависимости от конкретного расположения set par: перед import, перед show, конкретно в документе (не в conf.typ). Возможно, я чего-то просто не понимаю

[danilasar]

https://forum.typst.app/t/how-to-indent-every-paragraph-excluding-headings/2137/2

[danilasar]

https://forum.typst.app/t/how-to-indent-every-paragraph-excluding-headings/2137/2

На всё том же report.zip работает во всех случаях кроме абзаца после выровненной по левому краю таблицы. Это странно, ведь в примерах с форума решение работает железобетонно.

Также показалось, будто бы он игнорирует размер красной строки и вне зависимости от обстоятельств она будет 1.25em. Возможно, мне показалось и я от усталости уже не соображаю. В любом случае, нам всё равно вроде как столько и нужно, а, значит, годится как временное решение.

[danilasar]

После таблиц тоже заработало. Я думаю, стоит закоммитить это решение. Рассматриваю его как временное, потому что оно не выглядит универсальным. Но зато не ломает совместимость и не падает на styled)

[aragami3070]

Недавно обсуждали этот фикс, вроде сошлись на том, что оно не работает (как минимум у меня).
Думаю стоит открыть issue обратно. Оно либо билдится без красных строк, либо вообще не билдится (typst preview) и это печалит. Я бы вообще пока реверт фикса сделал (имхо)
Надеюсь у меня руки дойдут это пофиксить

[aragami3070]

Кста, вот че нашел, мб typst нормальный фикс красных строк мерджнули, думаю имеет смысл завтра глянуть.
typst/typst#5768

[aragami3070]

Почекал. Вот этот вариант работает (ниже), но есть НО

#set par(
  first-line-indent: (amount: 12pt, all: true),
  spacing: 5pt,
  leading: 5pt,
)

Те НО:

• lsp для typst пока еще не обновили и оно орет на это решение (не критично)
• typst-preview, тоже пока не обновили => не работает typst-preview (обидно досадно, но никто не мешает стили в самом конце подрубать)

[aragami3070]

#set par(
first-line-indent: (amount: 12pt, all: true),
spacing: 5pt,
leading: 5pt,
)

Значения рандомные брал, нужно будет поиграться с точными значениями, но это уже не ко мне))))
А, и с той либой для кода не тестил (надеюсь официальное решение работает с либами)