search

backstage / community-plugins

Community plugins for Backstage
Apache License 2.0
148 stars 157 forks source link

🐛 copilot: Overall theme is not applied correctly to plugin pages #1259

Open ScottGuymer opened 1 week ago

ScottGuymer commented 1 week ago

Plugin Name

copliot

📜 Description

The plugin seems to use its own theme and styling.

This means that it does not take on the theme from the instance its installed in so it looks out of place.

👍 Expected behavior

It should match the theme of the surrounding backstage app.

👎 Actual Behavior with Screenshots

It should match the theme of the surrounding backstage app.

image

Some of the styling also bleeds over into other non-related components on other pages outside of this plugin.

👟 Reproduction steps

Install plugin within a themed backstage app.

📃 Provide the context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

🏢 Have you read the Code of Conduct?

Are you willing to submit PR?

Yes I am willing to submit a PR!

awanlin commented 1 week ago

Hi @ScottGuymer, I did the initial review of this plugin when it was submitted and I tottaly missed the fact there was a ThemeProvider wrapping the entire page:

https://github.com/backstage/community-plugins/blob/7c68b0002c895f547b8304e404ca2bd83a6f345a/workspaces/copilot/plugins/copilot/src/components/Pages/CopilotPage.tsx#L56

The downside of these large reviews is that it's hard to catch everything. Hard to avoid with a net new plugin though!

I'd also mention that during the review they moved everything to MUI v5 seeing as that's really where we want to be heading which may also have some impact on this.

esw-afabiano commented 1 week ago

At the time, I encountered some issues with the theme not updating internally, though I don’t recall the specifics anymore. There is a PR already addressing this, but it may be separated into a new PR for those adjustments. In any case, I’ll attach it here as soon as there’s a resolution.