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

add theme colors option #4172

Open
wants to merge 527 commits into
base: master-mysterious-egg
Choose a base branch
from

Conversation

emge-odoo
Copy link

No description provided.

FrancoisGe and others added 19 commits March 5, 2025 16:08
Rename masonry_item_option to vertical_justify_option to make it more
general and use it for the option with selector '.s_quadrant .o_grid_item'
The option was duplicated in the section of the carousel (in addition to be in the slide's section)
Before this commit, svg for the shadow option were invisible.
This commits restore the colors of the svg buttons.
Before this commit, the start and end of the input range
were not reachable due to its styling.

Now, this issue is resolved, improving usability.
Firefox records a mutation when the textContent of an element is
assigned to the same value as before, but Chrome doesn't. This leads to
the builder marking many elements as dirty as soon as a single
modification is done.
This commit is a quick fix for the observed errors. A deeper fix should
come with task-4629669.
This commit adds the option to move the order of an image in the image gallery
It removes the old "notify" system to handle the reorder command, but uses
dispatching and a name parameter to decide who should handle the reordering
The `AddSnippetDialo`g component is doing a `loadBundle` to fetch
the content to put in the snippet dialog.

Previously, when using the `AddSnippetDialog``, tests waited for
a arbitrary amount of time before the snippet dialog shows.
This resulted in tests failing when there is latency from the
server/network.

This commit adds an utility that allow us to await the loadBundle
used by `AddSnippetDialog`allowing the tests to not wait an arbitrary
amount of time.
@robodoo
Copy link

robodoo commented Mar 7, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@emge-odoo emge-odoo marked this pull request as ready for review March 7, 2025 15:38
@robodoo
Copy link

robodoo commented Mar 7, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@emge-odoo emge-odoo force-pushed the master-mysterious-egg-emge-3 branch from b7c5c58 to 1403a48 Compare March 7, 2025 15:43
we recommend you to try to use the new color system, which is
still customizable.
</we-alert>-->
<BuilderRow>

Choose a reason for hiding this comment

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

You probably don t need to use BuilderRow if you don t have a label

Copy link
Author

Choose a reason for hiding this comment

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

I used it to make sure that the padding is right.

<div class="d-flex flex-row">
<BuilderColorPicker
title="Primary"
action="'customizeWebsiteColor'"

Choose a reason for hiding this comment

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

Where did you define "customizeWebsiteColor"

Copy link
Author

Choose a reason for hiding this comment

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

it is in theme_tab_plugin.js

@emge-odoo emge-odoo force-pushed the master-mysterious-egg-emge-3 branch from f482e8e to 2446aac Compare March 7, 2025 20:06
@ged-odoo ged-odoo force-pushed the master-mysterious-egg branch from b651514 to 9236e22 Compare March 8, 2025 07:05
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.