Skip to content

Comments

Fix ui.log background color being tinted by inner element#5831

Merged
falkoschindler merged 1 commit intomainfrom
log-background
Feb 22, 2026
Merged

Fix ui.log background color being tinted by inner element#5831
falkoschindler merged 1 commit intomainfrom
log-background

Conversation

@falkoschindler
Copy link
Contributor

Motivation

Fixes #5828. Applying Tailwind bg-* classes to ui.log results in "dirty" colors (e.g. bg-white shows as #F8FAFC instead of pure white) because a semi-transparent background-color: rgba(127, 159, 191, 0.05) on the inner .q-scrollarea__container blends on top of the user-specified background.

Implementation

Move the default background-color from .nicegui-log .q-scrollarea__container to .nicegui-log itself. This way, user-applied bg-* classes on the log element win via CSS specificity, cleanly overriding the default tint instead of being layered underneath it.

Progress

  • I chose a meaningful title that completes the sentence: "If applied, this PR will..."
  • The implementation is complete.
  • This is not a security issue.
  • Pytests are not necessary.
  • Documentation is not necessary.

@falkoschindler falkoschindler added this to the 3.8 milestone Feb 22, 2026
@falkoschindler falkoschindler added bug Type/scope: Incorrect behavior in existing functionality review Status: PR is open and needs review labels Feb 22, 2026
Copy link
Collaborator

@evnchn evnchn left a comment

Choose a reason for hiding this comment

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

Almost exactly the same as the solution in #5828 (comment) but reuses the block above. Great!

@falkoschindler falkoschindler added this pull request to the merge queue Feb 22, 2026
Merged via the queue into main with commit b662dca Feb 22, 2026
7 checks passed
@falkoschindler falkoschindler deleted the log-background branch February 22, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Type/scope: Incorrect behavior in existing functionality review Status: PR is open and needs review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ui.log: Wrong background color

2 participants