Skip to content
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

[FIX] theme_artists: add missing o_colored_level class #987

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chgo-odoo
Copy link
Contributor

This PR adds a missing o_colored_level class on the section.

With the themes revamp, we reviewed the design of a lot of snippets, which sometimes included colors adjustments using o_cc o_cc* classes.

While the visual result is okay on the page, the title of the snippet will be invisible in the modal as two o_cc o_cc* are nested without having an o_colored_level class on the section.

Master This PR
image image

task-4255088
related to task-4177975

This commit adds a missing `o_colored_level` class on the section.

With the themes revamp, we reviewed the design of a lot of snippets,
which sometimes included colors adjustments using `o_cc o_cc*` classes.

While the visual result is okay on the page, the title of the snippet will
be invisible in the modal as two `o_cc o_cc*` are nested without having
an `o_colored_level` class on the section.

task-4255088
related to task-4177975
@robodoo
Copy link
Collaborator

robodoo commented Oct 14, 2024

Pull request status dashboard

@chgo-odoo
Copy link
Contributor Author

@robodoo help

@robodoo
Copy link
Collaborator

robodoo commented Nov 20, 2024

Currently available commands for @chgo-odoo:

command
help displays this help
fw=no does not forward-port this PR
fw=default forward-ports this PR normally
up to <branch> only ports this PR forward to the specified branch (included)
check fetches or refreshes PR metadata, resets mergebot state

Note: this help text is dynamic and will change with the state of the PR.

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.

2 participants