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

Update frame's background #172

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

iris-garcia
Copy link

Fixes #148.

Dash 2.18.0 subsumed dash-functional, and company-box no longer
loads dash-functional, but it still lists old versions of dash and
dash-functional as dependencies, which can cause issues.

While at it, require the latest version 2.19.0 of Dash.

Re: sebastiencs#152, sebastiencs#153, sebastiencs#156, sebastiencs#157, sebastiencs#161, sebastiencs#168, sebastiencs#170.
@jthaman
Copy link

jthaman commented Aug 13, 2022

I'd love to see this merged.

sebastiencs added a commit that referenced this pull request Sep 9, 2022
Delete existing frames to rebuild them from scratch.
They will use the colors of the new theme.
@sebastiencs
Copy link
Owner

Hi @iris-garcia , thanks for your PR !

The call to company-box--update-frame-background is a bit expensive and I would prefer to avoid it on every run of company-box--display.

In 82c1a30 I made an alternate solution: it will delete the frames (of company-box) every time the theme is changing.
This will force to use the colors of the new theme.

Could you, or @jthaman , confirm that it works ?

Thanks !

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.

Weird face colors when changing themes
4 participants