Skip to content

Conversation

@christoph-jerolimov
Copy link
Member

Hey, I just made a Pull Request!

When testing the global-header workspace I noticed this white background behind the app in the dark theme.

Before:

Screenshot From 2026-01-13 00-22-41

With this PR:

Screenshot From 2026-01-13 00-26-27

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Jan 12, 2026

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-global-header-test
  • @red-hat-developer-hub/backstage-plugin-global-header

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
app workspaces/global-header/packages/app none v0.0.0
@red-hat-developer-hub/backstage-plugin-global-header-test workspaces/global-header/plugins/global-header-test none v0.7.0
@red-hat-developer-hub/backstage-plugin-global-header workspaces/global-header/plugins/global-header none v1.20.1

Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

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

Does import '@backstage/ui/css/styles.css'; needs to be added also to dev/index.ts?

@christoph-jerolimov
Copy link
Member Author

@dzemanov the dev pages looked already good but I added it anyway. Please take a new look. Thanks.

Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@sonarqubecloud
Copy link

@christoph-jerolimov christoph-jerolimov merged commit 8a6734f into redhat-developer:main Jan 21, 2026
10 checks passed
@christoph-jerolimov christoph-jerolimov deleted the global-header/add-bui branch January 21, 2026 08:54
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.

2 participants