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

[FW][FIX] base_import: fix background color in dark mode #159336

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 26, 2024

This commit adapts base_import's background that was too dark compared to other modules. This was caused by the use of a bg-white class that is not very "dark-mode friendly" as it will display as a pure black, which is not visually appealing and not wanted.

This commit also adapts the background color of the sidebar to make it more consistent with other modules.

We also add a bg-view class to the table, so it stands out more in light mode and stays the same in dark mode.

task-3814508

Before After
Screenshot 2024-03-22 at 08 32 26 Screenshot 2024-03-22 at 08 34 08
Screenshot 2024-03-22 at 08 32 42 Screenshot 2024-03-22 at 08 34 19
Screenshot 2024-03-22 at 08 59 52 Screenshot 2024-03-22 at 08 57 41
Screenshot 2024-03-22 at 08 33 08 Screenshot 2024-03-22 at 08 33 59

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #158822
Forward-Port-Of: #158019

This commit adapts `base_import`'s background that was too dark
compared to other modules. This was caused by the use of a `bg-white`
class that is not very "dark-mode friendly" as it will display as a
pure black, which is not visually appealing and not wanted.

This commit also adapts the background color of the sidebar to make it
more consistent with other modules.

We also add a `bg-view` class to the table, so it stands out more in
light mode and stays the same in dark mode.

task-3814508

X-original-commit: e7d0b3b
@robodoo
Copy link
Contributor

robodoo commented Mar 26, 2024

@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 26, 2024

@anso-odoo @pparidans this PR targets master and is the last of the forward-port chain containing:

To merge the full chain, use

@fw-bot r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Mar 26, 2024
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 26, 2024
@pparidans
Copy link
Contributor

@fw-bot r+

robodoo pushed a commit that referenced this pull request Mar 28, 2024
This commit adapts `base_import`'s background that was too dark
compared to other modules. This was caused by the use of a `bg-white`
class that is not very "dark-mode friendly" as it will display as a
pure black, which is not visually appealing and not wanted.

This commit also adapts the background color of the sidebar to make it
more consistent with other modules.

We also add a `bg-view` class to the table, so it stands out more in
light mode and stays the same in dark mode.

task-3814508

closes #159336

X-original-commit: e7d0b3b
Signed-off-by: Pierre Paridans (app) <app@odoo.com>
@robodoo robodoo closed this Mar 28, 2024
@robodoo robodoo added the 17.3 label Mar 28, 2024
@fw-bot fw-bot deleted the master-saas-16.3-base-import-dark-mode-fix-anso-i7WK-fw branch April 11, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
17.3 forwardport This PR was created by @fw-bot RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants