Skip to content

[18.0][IMP] web_company_color: allow to set bottom border color of navbar#3484

Merged
OCA-git-bot merged 3 commits intoOCA:18.0from
hbrunn:18.0-web_company_color-navbar-bottom
Mar 25, 2026
Merged

[18.0][IMP] web_company_color: allow to set bottom border color of navbar#3484
OCA-git-bot merged 3 commits intoOCA:18.0from
hbrunn:18.0-web_company_color-navbar-bottom

Conversation

@hbrunn
Copy link
Copy Markdown
Member

@hbrunn hbrunn commented Mar 17, 2026

Odoo adds a 1px border below the nav bar, which is not visible below navbar buttons. Usually it's hard to notice because the colors are darkish, but when using web_company_color to set a bright color, it looks like this on current runbot:

image

but with this patch and setting the border color, it looks like this:

image

I took the liberty to simplify the code I had to touch anyways

@pedrobaeza pedrobaeza added this to the 18.0 milestone Mar 19, 2026
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Thanks for this. I think the first commit should be [REF], not [FIX].

@hbrunn hbrunn force-pushed the 18.0-web_company_color-navbar-bottom branch from 7a944b6 to d1b75f3 Compare March 24, 2026 14:07
@hbrunn
Copy link
Copy Markdown
Member Author

hbrunn commented Mar 24, 2026

@pedrobaeza thanks for the headsup, fixed

Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-3484-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 525dc0a into OCA:18.0 Mar 25, 2026
7 of 9 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at cdd0e55. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants