Skip to content

[FIX] developer/tutorials: Button example without deprecated attrs#18017

Closed
Megaaaaaa wants to merge 1 commit into
17.0from
17.0-button_exemple_without_deprecated_attrs-thbe
Closed

[FIX] developer/tutorials: Button example without deprecated attrs#18017
Megaaaaaa wants to merge 1 commit into
17.0from
17.0-button_exemple_without_deprecated_attrs-thbe

Conversation

@Megaaaaaa
Copy link
Copy Markdown
Contributor

@Megaaaaaa Megaaaaaa commented May 20, 2026

Using attrs to make a button invisible is deprecated since 17.0 but the button example hyperlink sends to a commit that used it so it induces people following the tutorial to use attrs which shouldn't be done anymore.

This commit replaces the link with the commit that replaced the attrs="" to the plain invisible="" use on the exact same button. (And adjusts the size of the delimiter to respect documentation guidelines.)

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented May 20, 2026

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team May 20, 2026 14:21
Using attrs to make a button invisible is deprecated since 17.0 but the button
example hyperlink sends to a commit that used it so it induces people following
the tutorial to use attrs which shouldn't be done anymore.

This commit replaces the link with the commit that replaced the attrs="" to the
plain invisible="" use on the exact same button. (And adjusts the size of the
heading delimiter to respect documentation guidelines.)
@Megaaaaaa Megaaaaaa force-pushed the 17.0-button_exemple_without_deprecated_attrs-thbe branch from 19519ce to b20dfd9 Compare May 20, 2026 14:25
@Megaaaaaa Megaaaaaa requested a review from AntoineVDV May 21, 2026 07:11
Copy link
Copy Markdown
Collaborator

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

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

robodoo pushed a commit that referenced this pull request May 21, 2026
Using attrs to make a button invisible is deprecated since 17.0 but the button
example hyperlink sends to a commit that used it so it induces people following
the tutorial to use attrs which shouldn't be done anymore.

This commit replaces the link with the commit that replaced the attrs="" to the
plain invisible="" use on the exact same button. (And adjusts the size of the
heading delimiter to respect documentation guidelines.)

closes #18017

Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@robodoo robodoo closed this May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants