caisin/typst
1
0
Fork 0
mirror of https://github.com/typst/typst synced 2025-08-15 15:38:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

Use element specific terminology in warning

Browse Source
This commit is contained in:
mkorje 2025-03-25 23:48:54 +11:00
parent b89cdbbfad
commit c6536797d2
No known key found for this signature in database
3 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
crates/typst-layout/src/math/mat.rs
View File

@ -35,6 +35,7 @@ pub fn layout_vec(
None, None,
Axes::with_y(elem.gap(styles)), Axes::with_y(elem.gap(styles)),
span, span,
"elements",
)?; )?;
let delim = elem.delim(styles); let delim = elem.delim(styles);
@ -60,6 +61,7 @@ pub fn layout_cases(
None, None,
Axes::with_y(elem.gap(styles)), Axes::with_y(elem.gap(styles)),
span, span,
"branches",
)?; )?;
let delim = elem.delim(styles); let delim = elem.delim(styles);
@ -119,6 +121,7 @@ pub fn layout_mat(
augment, augment,
Axes::new(elem.column_gap(styles), elem.row_gap(styles)), Axes::new(elem.column_gap(styles), elem.row_gap(styles)),
span, span,
"cells",
)?; )?;
let delim = elem.delim(styles); let delim = elem.delim(styles);
@ -136,6 +139,7 @@ fn layout_body(
augment: Option<Augment<Abs>>, augment: Option<Augment<Abs>>,
gap: Axes<Rel<Abs>>, gap: Axes<Rel<Abs>>,
span: Span, span: Span,
children: &str,
) -> SourceResult<Frame> { ) -> SourceResult<Frame> {
let nrows = columns.first().map_or(0, |col| col.len()); let nrows = columns.first().map_or(0, |col| col.len());
let ncols = columns.len(); let ncols = columns.len();
@ -193,7 +197,7 @@ fn layout_body(
if cell.is_multiline() { if cell.is_multiline() {
ctx.engine.sink.warn(warning!( ctx.engine.sink.warn(warning!(
cell_span, cell_span,
"linebreaks are ignored in cells"; "linebreaks are ignored in {}", children;
hint: "use commas instead to separate each line" hint: "use commas instead to separate each line"
)); ));
} }

2
tests/suite/math/cases.typ
View File

@ -17,6 +17,6 @@ $ x = cases(1, 2) $
$ cases(a, b, c) $ $ cases(a, b, c) $
--- math-cases-linebreaks --- --- math-cases-linebreaks ---
// Warning: 40-49 linebreaks are ignored in cells // Warning: 40-49 linebreaks are ignored in branches
// Hint: 40-49 use commas instead to separate each line // Hint: 40-49 use commas instead to separate each line
$ cases(a, b, c) cases(reverse: #true, a \ b \ c) $ $ cases(a, b, c) cases(reverse: #true, a \ b \ c) $

2
tests/suite/math/vec.typ
View File

@ -51,6 +51,6 @@ $ vec(1, 2) $
#set math.vec(delim: (none, "%")) #set math.vec(delim: (none, "%"))
--- math-vec-linebreaks --- --- math-vec-linebreaks ---
// Warning: 20-29 linebreaks are ignored in cells // Warning: 20-29 linebreaks are ignored in elements
// Hint: 20-29 use commas instead to separate each line // Hint: 20-29 use commas instead to separate each line
$ vec(a, b, c) vec(a \ b \ c) $ $ vec(a, b, c) vec(a \ b \ c) $