feat: components v2 & `View` improvements #2707

NeloBlivion · 2025-02-06T04:42:20Z

Summary

Soon

Edit by Lala:
Temporary reference: https://github.com/Lulalaby/discord-api-docs/blob/comp_v2/docs/interactions/Message_Components.md#v2-components

Yet another edit by Lala:
Official PR for the docs is available now at Components v2 discord/discord-api-docs#7487

Information

This PR fixes an issue.
This PR adds something new (e.g. new method or parameters).
This PR is a breaking change (e.g. methods or parameters removed/renamed).
This PR is not a code change (e.g. documentation, README, typehinting,
examples, ...).

Checklist

I have searched the open pull requests for duplicates.
If code changes were made then they have been tested.
- I have updated the documentation to reflect the changes.
If type: ignore comments were used, a comment is also left explaining why.
I have updated the changelog to include these changes.

Reference:

discord/flags.py

discord/types/components.py

Co-authored-by: plun1331 <plun1331@gmail.com> Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com> Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

Co-authored-by: plun1331 <plun1331@gmail.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

Paillat-dev · 2025-07-02T18:18:11Z

@NeloBlivion I think something broke with your recent pushes:

if payload["content"] or payload["embeds"]

in ``discord/webhooks/_async.py`

Is causing:

KeyError: 'content'
  File "discord/commands/core.py", line 138, in wrapped
    ret = await coro(arg)
  File "discord/commands/core.py", line 1078, in _invoke
    await self.callback(self.cog, ctx, **kwargs)
  File "src/extensions/chatcop/cogs/admin/__init__.py", line 260, in view
    await ctx.respond(view=view, ephemeral=True, allowed_mentions=discord.AllowedMentions.none())
  File "discord/ext/bridge/context.py", line 92, in respond
    return await self._respond(*args, **kwargs)
  File "discord/ext/bridge/context.py", line 145, in _respond
    return await self._get_super("respond")(*args, **kwargs)
  File "discord/interactions.py", line 694, in respond
    return await self.followup.send(*args, **kwargs)
  File "discord/webhook/async_.py", line 1822, in send
    params = handle_message_parameters(
  File "discord/webhook/async_.py", line 673, in handle_message_parameters
    if payload["content"] or payload["embeds"]:

Icebluewolf · 2025-07-09T23:22:46Z

discord/components.py

@@ -69,12 +105,15 @@ class Component:
    ----------
    type: :class:`ComponentType`
        The type of component.
+    id: :class:`int`
+        The component's ID. If not provided by the user, it is automatically incremented.


Suggested change

The component's ID. If not provided by the user, it is automatically incremented.

The component's ID. If not provided by the user, it is set sequentially by Discord.

The ID `0` is treated as if no ID was provided.

Icebluewolf · 2025-07-09T23:32:01Z

discord/types/components.py

+    id: NotRequired[int]
+
+
+class ActionRow(BaseComponent):
    type: Literal[1]
    components: list[Component]


Suggested change

components: list[Component]

components: list[ButtonComponent, InputText, SelectMenu]

Or make an AllowedActionRowComponent class like done with container.

Icebluewolf · 2025-07-09T23:34:47Z

discord/components.py

        self.children: list[Component] = [
            _component_factory(d) for d in data.get("components", [])
        ]

+    @property
+    def width(self):
+        """Return the total item width that this action row uses."""


Suggested change

"""Return the total item width that this action row uses."""

"""Return the sum of the children's widths."""

Icebluewolf · 2025-07-10T21:34:03Z

discord/components.py

+class Section(Component):
+    """Represents a Section from Components V2.
+
+    This is a component that groups other components together.


Suggested change

This is a component that groups other components together.

This is a component that groups other components together with an additional component to the right as the accessory.

Icebluewolf · 2025-07-10T21:36:07Z

discord/components.py

+    description: Optional[:class:`str`]
+        The thumbnail's description, up to 1024 characters.
+    spoiler: Optional[:class:`bool`]
+        Whether the thumbnail is a spoiler.


Suggested change

Whether the thumbnail is a spoiler.

Whether the thumbnail has the spoiler overlay.

Icebluewolf · 2025-07-10T22:08:01Z

discord/ui/media_gallery.py

+        """
+
+        if len(self.items) >= 10:
+            raise ValueError("maximum number of children exceeded")


Suggested change

raise ValueError("maximum number of children exceeded")

raise ValueError("maximum number of items exceeded")

Icebluewolf · 2025-07-10T22:10:22Z

discord/ui/section.py

+        description: Optional[:class:`str`]
+            The thumbnail's description, up to 1024 characters.
+        spoiler: Optional[:class:`bool`]
+            Whether the thumbnail is a spoiler. Defaults to ``False``.


Suggested change

Whether the thumbnail is a spoiler. Defaults to ``False``.

Whether the thumbnail has the spoiler overlay. Defaults to ``False``.

Icebluewolf · 2025-07-10T22:10:56Z

discord/ui/section.py

+
+    def enable_all_items(self, *, exclusions: list[Item] | None = None) -> Self:
+        """
+        Enables all buttons and select menus in the container.


Suggested change

Enables all buttons and select menus in the container.

Enables all buttons and select menus in the section.

Icebluewolf · 2025-07-10T22:12:34Z

discord/ui/thumbnail.py

+    description: Optional[:class:`str`]
+        The thumbnail's description, up to 1024 characters.
+    spoiler: Optional[:class:`bool`]
+        Whether the thumbnail is a spoiler. Defaults to ``False``.


Suggested change

Whether the thumbnail is a spoiler. Defaults to ``False``.

Whether the thumbnail has the spoiler overlay. Defaults to ``False``.

Icebluewolf · 2025-07-10T22:12:57Z

discord/ui/thumbnail.py

+
+    @property
+    def spoiler(self) -> bool:
+        """Whether the thumbnail is a spoiler. Defaults to ``False``."""


Suggested change

"""Whether the thumbnail is a spoiler. Defaults to ``False``."""

"""Whether the thumbnail has the spoiler overlay. Defaults to ``False``."""

Paillat-dev · 2025-07-13T12:04:24Z

discord/client.py

+        This only fires for a view if you did not define its :func:`~discord.ui.View.on_error`.
+        """
+
+        print(


Should consider using something else than print

Paillat-dev · 2025-07-13T12:04:36Z

discord/client.py

+        This only fires for a modal if you did not define its :func:`~discord.ui.Modal.on_error`.
+        """
+
+        print(f"Ignoring exception in modal {interaction.modal}:", file=sys.stderr)


Paillat-dev · 2025-07-13T12:24:05Z

discord/components.py

+        r._state = state
+        return r
+
+    def to_dict(self):


Suggested change

def to_dict(self):

def to_dict(self) -> dict[str, str]:

Paillat-dev · 2025-07-13T12:26:08Z

discord/components.py

+        r.media = media
+        return r
+
+    def to_dict(self):


Suggested change

def to_dict(self):

def to_dict(self) -> dict[str, str | dict[str, str] | bool]:

or just dict[str, Any]

Icebluewolf

I cannot for the life of me figure out how to suggest a change to this line.
https://github.com/Pycord-Development/pycord/pull/2707/files#diff-5894d281c8e052e9224fdf3643f8260c0c0a5fa59acbb808ba7faf54bede6ee9R1773
But just add "dispatchable" before "view"
Line 1773 of webhook/async_.py

Lulalaby · 2025-07-14T19:20:27Z

yeah that line is not part of the actual diff so u cant suggest there

NeloBlivion and others added 4 commits February 6, 2025 05:40

base types and flags

714b2f6

style(pre-commit): auto fixes from pre-commit.com hooks

468f996

textdisplayComponent

905b9ff

more

49080e7

Lulalaby assigned Lulalaby and NeloBlivion Feb 6, 2025

Lulalaby added discord limitation Limitation imposed by discord priority: medium Medium Priority status: in progress Work in Progess feature Implements a feature upcoming discord feature Involves a feature not yet fully released by Discord labels Feb 6, 2025

Lulalaby added this to the v2.7 milestone Feb 6, 2025

Lulalaby requested review from Lulalaby, plun1331 and JustaSqu1d February 6, 2025 05:35

JustaSqu1d reviewed Feb 6, 2025

View reviewed changes

discord/flags.py Outdated Show resolved Hide resolved

NeloBlivion and others added 7 commits February 6, 2025 06:57

Section, TextDisplay

e3e7aba

style(pre-commit): auto fixes from pre-commit.com hooks

e961db5

remaining classes

947890d

style(pre-commit): auto fixes from pre-commit.com hooks

6e7dde9

basic view support start

d18a9c4

style(pre-commit): auto fixes from pre-commit.com hooks

5ab45fd

flag clarification

fb8d13d

plun1331 reviewed Feb 6, 2025

View reviewed changes

discord/types/components.py Outdated Show resolved Hide resolved

NeloBlivion and others added 6 commits February 7, 2025 05:53

complete models

65dc63d

style(pre-commit): auto fixes from pre-commit.com hooks

f86f707

Merge branch 'master' into components-v2

1cced5a

fix

3e03e84

fix2

0f8c20a

update loading_state

bf60ffc

NeloBlivion and others added 21 commits June 29, 2025 14:35

Merge branch 'master' into components-v2

9b1aa43

raise on content/embeds

d92fd92

style(pre-commit): auto fixes from pre-commit.com hooks

d26dba5

updated docs, streamlined unfurledmedia

72234ff

fix with_components

b7d5040

style(pre-commit): auto fixes from pre-commit.com hooks

969600d

typing.Self

0c8c719

style(pre-commit): auto fixes from pre-commit.com hooks

c29b967

raise again

ca3f1ce

style(pre-commit): auto fixes from pre-commit.com hooks

5490f90

Apply suggestions from code review

2fdf147

Co-authored-by: plun1331 <plun1331@gmail.com> Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

Apply suggestions from code review

310ecf9

Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com> Co-authored-by: JustaSqu1d <89910983+JustaSqu1d@users.noreply.github.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

style(pre-commit): auto fixes from pre-commit.com hooks

80ff839

Update examples/views/new_components.py

a54223d

Co-authored-by: plun1331 <plun1331@gmail.com> Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>

style(pre-commit): auto fixes from pre-commit.com hooks

3e5fec8

typ

2930396

item docstring

bc88140

i hate this

37da37a

add repr

c66b668

custom_id

32f149c

Update section.py

66bbf2a

NeloBlivion added 3 commits July 2, 2025 22:06

keyerror

f6de681

add warning for Button.row

b470a36

Merge branch 'master' into components-v2

b0d4ced

Icebluewolf suggested changes Jul 10, 2025

View reviewed changes

InputText.id

bcf7aa1

Paillat-dev reviewed Jul 13, 2025

View reviewed changes

Icebluewolf suggested changes Jul 14, 2025

View reviewed changes

	The component's ID. If not provided by the user, it is automatically incremented.
	The component's ID. If not provided by the user, it is set sequentially by Discord.
	The ID `0` is treated as if no ID was provided.

	components: list[Component]
	components: list[ButtonComponent, InputText, SelectMenu]

	"""Return the total item width that this action row uses."""
	"""Return the sum of the children's widths."""

	This is a component that groups other components together.
	This is a component that groups other components together with an additional component to the right as the accessory.

	Whether the thumbnail is a spoiler.
	Whether the thumbnail has the spoiler overlay.

	raise ValueError("maximum number of children exceeded")
	raise ValueError("maximum number of items exceeded")

	Whether the thumbnail is a spoiler. Defaults to ``False``.
	Whether the thumbnail has the spoiler overlay. Defaults to ``False``.

	Enables all buttons and select menus in the container.
	Enables all buttons and select menus in the section.

	"""Whether the thumbnail is a spoiler. Defaults to ``False``."""
	"""Whether the thumbnail has the spoiler overlay. Defaults to ``False``."""

	def to_dict(self):
	def to_dict(self) -> dict[str, str \| dict[str, str] \| bool]:

Uh oh!

feat: components v2 & View improvements #2707

Are you sure you want to change the base?

feat: components v2 & View improvements #2707

Uh oh!

Conversation

NeloBlivion commented Feb 6, 2025 • edited by Lulalaby Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Information

Checklist

Reference:

Uh oh!

Uh oh!

Uh oh!

Paillat-dev commented Jul 2, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Icebluewolf left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Lulalaby commented Jul 14, 2025

Uh oh!

Uh oh!

feat: components v2 & `View` improvements #2707

feat: components v2 & `View` improvements #2707

NeloBlivion commented Feb 6, 2025 •

edited by Lulalaby

Loading

Icebluewolf left a comment •

edited

Loading