_break and _opt call grp instead of _grp: compilation error #17

New Issue

just-some-bugs · 2025-07-27T14:26:28Z

just-some-bugs commented

2025-07-27 14:26:28 +00:00

unknown variable: grp
   ┌─ @preview/chronos:0.2.1/src/group.typ:41:25
   │
41 │ #let _opt(desc, elmts) = grp("opt", desc: desc, type: "opt", elmts)

Edit: I also just noticed that the opt block doesn't expand the width based on the title length nor its content. It does work if I put another grp inside the opt (in my case a loop inside of opt). But not if I use _seq.

``` unknown variable: grp ┌─ @preview/chronos:0.2.1/src/group.typ:41:25 │ 41 │ #let _opt(desc, elmts) = grp("opt", desc: desc, type: "opt", elmts) ``` __Edit:__ I also just noticed that the opt block doesn't expand the width based on the title length nor its content. It does work if I put another grp inside the opt (in my case a loop inside of opt). But not if I use _seq. ![image.png](/attachments/257064d0-3330-46fb-bb2b-2ff566b4fd61)

image.png

24 KiB

HEL added the

bug

label 2025-07-27 14:28:18 +00:00

HEL commented

2025-07-27 14:34:03 +00:00

I noticed this when refactoring the package for styling in d6aeb33fa5
I don't know if this is worth a patch release or not though, especially since it is only an alias

I noticed this when refactoring the package for styling in d6aeb33fa5 I don't know if this is worth a patch release or not though, especially since it is only an alias

HEL commented

2025-07-27 14:37:01 +00:00

Edit: I also just noticed that the opt block doesn't expand the width based on the title length nor its content

Yes this was also a know bug that should be fixed in the dev branch (see #16)

> **Edit:** I also just noticed that the opt block doesn't expand the width based on the title length nor its content Yes this was also a know bug that should be fixed in the dev branch (see #16)

👍 1

just-some-bugs commented

2025-07-27 14:39:23 +00:00

Edit: I also just noticed that the opt block doesn't expand the width based on the title length nor its content

Yes this was also a know bug that should be fixed in the dev branch (see #16)

Can I use the dev branch somehow? Or do I have to clone the repo and include the file manually?

I noticed this when refactoring the package for styling in d6aeb33fa5
I don't know if this is worth a patch release or not though, especially since it is only an alias

At the end of the day, that's what patch releases are for :D

> > **Edit:** I also just noticed that the opt block doesn't expand the width based on the title length nor its content > > Yes this was also a know bug that should be fixed in the dev branch (see #16) Can I use the dev branch somehow? Or do I have to clone the repo and include the file manually? > I noticed this when refactoring the package for styling in d6aeb33fa5 > I don't know if this is worth a patch release or not though, especially since it is only an alias At the end of the day, that's what patch releases are for :D

just-some-bugs commented

2025-07-27 17:48:13 +00:00

Edit: I also just noticed that the opt block doesn't expand the width based on the title length nor its content

Yes this was also a know bug that should be fixed in the dev branch (see #16)

Hey! I figured out how to use the dev branch (@local packages). However, the above code doesn't work with self-calls. The MRE is this:

#import "@local/chronos:0.2.2"

#chronos.diagram({
  import chronos: *

  _par("A")
  _par("B")

  _opt("Test", {
    _seq("A", "A")
  })
})

Here is a screenshot with a comparison between the self-call and a sequence between A and B:

> > **Edit:** I also just noticed that the opt block doesn't expand the width based on the title length nor its content > > Yes this was also a know bug that should be fixed in the dev branch (see #16) Hey! I figured out how to use the dev branch (@local packages). However, the above code doesn't work with self-calls. The MRE is this: ```typ #import "@local/chronos:0.2.2" #chronos.diagram({ import chronos: * _par("A") _par("B") _opt("Test", { _seq("A", "A") }) }) ``` Here is a screenshot with a comparison between the self-call and a sequence between A and B: ![image.png](/attachments/63165335-aa14-41fc-a097-44b3c2cf3a27)

image.png

47 KiB

HEL commented

2025-07-27 19:38:12 +00:00

Indeed, this should affect all groups :(
Edit: I opened a separate issue for this

Indeed, this should affect all groups :( **Edit:** I opened a [separate issue](https://git.kb28.ch/HEL/chronos/issues/18) for this

HEL referenced this issue from a commit

2025-07-28 20:36:04 +00:00

fixed #17

HEL referenced this issue

2025-07-28 21:04:20 +00:00

Groups don't expand to contain name, description or self sequences #18