Skip to content
This repository was archived by the owner on Oct 5, 2025. It is now read-only.

Conversation

@bowenjw
Copy link

@bowenjw bowenjw commented Aug 31, 2025

  • Updated Modal page for label and select menu components

I am not happy with how the select menu integration as it feels unwieldy and redundant to the existing select menu content. It is not a scalable layout should more modal components be added in the future

Dependent on


::: tip
You can have a maximum of five <DocsLink path="ActionRowBuilder:Class" />s per modal builder, and one <DocsLink path="TextInputBuilder:Class" /> within an <DocsLink path="ActionRowBuilder:Class" />. Currently, you can only use <DocsLink path="TextInputBuilder:Class" />s in modal action rows builders.
You can have a maximum of five <DocsLink path="LabelBuilder:Class" />s per modal builder, and one <DocsLink path="TextInputBuilder:Class" /> or <DocsLink path="StringSelectMenuBuilder:Class" /> within an <DocsLink path="LabelBuilder:Class" />.
Copy link

@thorminate thorminate Sep 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"... within an <DocsLink path="LabelBuilder:Class" /"

use "a" instead of "an" as "LabelBuilder"s first letter is a consonant.

.setDescription('The Water-type Tiny Turtle Pokémon.')
.setValue('squirtle'),
);
// An Label component only holds one interactive component,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use "a" instead of "an" here too.

@Jiralite
Copy link
Member

Jiralite commented Oct 5, 2025

Hey, this should be made over at our monorepo!

@Jiralite Jiralite closed this Oct 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants