-
Notifications
You must be signed in to change notification settings - Fork 308
[IMP] theme_*: improve SEO by correcting heading tag hierarchy #1043
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
chgo-odoo
wants to merge
1
commit into
odoo:master
from
odoo-dev:master-website-finetune-snippets-heading-chgo
Closed
[IMP] theme_*: improve SEO by correcting heading tag hierarchy #1043
chgo-odoo
wants to merge
1
commit into
odoo:master
from
odoo-dev:master-website-finetune-snippets-heading-chgo
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bb5de82 to
1e15873
Compare
e20a929 to
52b8cd3
Compare
8b28770 to
012a09e
Compare
Drose1971
approved these changes
Apr 30, 2025
572f1f3 to
6c50db1
Compare
6c50db1 to
2386fe2
Compare
a11e6cf to
db0c38a
Compare
4fd527f to
6e9c041
Compare
6e9c041 to
f0073ac
Compare
s_popup xpath* anelusia, artists, avantgarde, aviato, beauty, bewise, bistro, bookstore, buzzy, clean, cobalt, enark, graphene, kea, kiddo, loftspace, monglia, nano, notes, odoo_experts, orchid, paptic, real estate, treehouse, vehicle, yes, zap This commit adapts all the `xpath` expressions inside design themes to target the new heading tags that were changed within the snippet files. task-4349019
f0073ac to
fe78996
Compare
qsm-odoo
approved these changes
Sep 8, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That must have been an annoying work 😬 Good job ;)
@robodoo r+
robodoo
pushed a commit
that referenced
this pull request
Sep 8, 2025
* anelusia, artists, avantgarde, aviato, beauty, bewise, bistro, bookstore, buzzy, clean, cobalt, enark, graphene, kea, kiddo, loftspace, monglia, nano, notes, odoo_experts, orchid, paptic, real estate, treehouse, vehicle, yes, zap This commit adapts all the `xpath` expressions inside design themes to target the new heading tags that were changed within the snippet files. task-4349019 closes #1043 Related: odoo/odoo#191300 Related: odoo/enterprise#92978 Signed-off-by: Quentin Smetz (qsm) <[email protected]>
robodoo
pushed a commit
to odoo/odoo
that referenced
this pull request
Sep 8, 2025
Complete list of snippets : s_attributes_horizontal, s_call_to_action, s_card, s_card_offset, s_card_offset, s_cards_grid, s_cards_soft, s_carousel, s_carousel_cards, s_company_team, s_company_team_basic, s_company_team_detail, s_company_team_spotlight, s_comparisons, s_cta_card, s_cta_mobile, s_cta_mockups, s_faq_collapse, s_faq_list, s_features, s_features_grid, s_features_wave, s_image_frame, s_image_text_box, s_masonry_block, s_numbers, s_numbers_charts, s_numbers_list, s_showcase, s_table_of_content, s_text_highlight, s_three_columns, s_title Some snippets in website were using inconsistent heading levels (e.g. h2 directly followed by h6), which harmed both accessibility and SEO. This commit standardizes the heading hierarchy inside snippets to ensure a logical structure. This improves: 1. Natural SEO ranking by providing clearer semantic signals to search engines. 2. Accessibility for screen readers that rely on a consistent heading order. 3. Overall readability and maintainability of the themes. Note that the we preserved the design by applying a `.h*-fs` class that matches the old heading tag in use. We also fixed selectors inside tests and adapted the new page templates task-4349019 closes #191300 Related: odoo/enterprise#92978 Related: odoo/design-themes#1043 Signed-off-by: Quentin Smetz (qsm) <[email protected]>
robodoo
pushed a commit
that referenced
this pull request
Sep 8, 2025
* anelusia, artists, avantgarde, aviato, beauty, bewise, bistro, bookstore, buzzy, clean, cobalt, enark, graphene, kea, kiddo, loftspace, monglia, nano, notes, odoo_experts, orchid, paptic, real estate, treehouse, vehicle, yes, zap This commit adapts all the `xpath` expressions inside design themes to target the new heading tags that were changed within the snippet files. task-4349019 closes #1043 Related: odoo/odoo#191300 Related: odoo/enterprise#92978 Signed-off-by: Quentin Smetz (qsm) <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Requires :
*: anelusia, artists, avantgarde, aviato, beauty, bewise, bistro, bookstore, buzzy, clean, cobalt, enark, graphene, kea, kiddo, loftspace, monglia, nano, notes, odoo_experts, orchid, paptic, real estate, treehouse, vehicle, yes, zap
This PR adapts all the
xpathexpressions inside design themes to target the new heading tags that were changed within the snippet files.task-4349019