znai: Collapsible API-Parameters breaks on tabs

If you try to add a collapsible api-parameters inside of a tab, the api-parameter gets eaten and, instead, the title for said api-parameter is added as a tab.

Example:

# /badges
Shows a list of your own badges or a list of the badges from the given user. A full list of the badges can be found [here](currency/badges).

````tabs
Subcommands:
* `Info`: Shows info about a badge.
* `List`: Lists all the obtainable badges.
* `Show`: Show your badge list, or someone else's badge list. 

Options:
```api-parameters {title: "Hello", collapsible: true}
Info, Click me!, "Options for the `Info` Subcommand"
Info.badge, "Required", "The badge to check info for"
Show, Click me!, "Options for the `Show` Subcommand"
Show.user, "Optional", "The user to check. If none, you."
Show.brief, "Optional", "Whether to show this in brief format. Default is false."
```

Requirements:
* This command requires [**Embed**, **Emotes** and **Reactions** permissions](commands/permissions#intro).

Examples:
* /badges show
* /badges show `user:`@Kodehawa#3457
* /badges list
* /badges info `badge:`Donator
````

Result: image image

If the collapsible: true option is removed however, it works fine. image

About this issue

  • Original URL
  • State: closed
  • Created 2 years ago
  • Comments: 15 (8 by maintainers)

Most upvoted comments

Thanks for trying! you can ignore sync in IDE. as for the test failures looks like it has been awhile I built it using windows.

Don’t want to use more of your time. But just in case, if you ever want to PR windows fix I will be more than happy to merge it.

Meanwhile I will work on releasing a patch today or tomorrow.

oh, Options {title: "a"}: is an interesting cue. Thanks. PR is being built right now. Should fix that. @MrLar BTW you adding title to tab name, is it by accident or are you trying to achieve something with this?

It’s simply something I noticed there was no real intention behind it, just messing about trying to find a cause.