fix: inconsistent background in dark mode #59

Merged
klaipras merged 2 commits from dark-mode-fix into ui_change 2026-10-04 14:54:30 +00:00
Member

Description

remove uneccessary css and fix duplicate classes causing the issues.
There are duplicate declaration on the same class at different part of css file

#45

Closes #45

Type of change

Type:

Testing

  • I added/updated tests.
  • I tested my changes locally.
    • I tested my changes with a laptop-size screen.
    • I tested my changes with a phone-size screen.
    • I tested my changes with a tablet-size screen.
  • I tested on the kennel preview deployment for this pull request.

Screenshots/Recordings

Checklist

  • My code follows the project's existing style.
  • I have reviewed my changes.
  • Only the necessary files and lines have been changed.
  • This PR is ready for review.
## Description remove uneccessary css and fix duplicate classes causing the issues. There are duplicate declaration on the same class at different part of css file ## Related Issue #45 Closes #45 ## Type of change **Type:** <!-- Bug fix / New feature / Refactor / Test coverage / Documentation --> ## Testing - [x] I added/updated tests. - [x] I tested my changes locally. - [x] I tested my changes with a laptop-size screen. - [ ] I tested my changes with a phone-size screen. - [ ] I tested my changes with a tablet-size screen. - [ ] I tested on the kennel preview deployment for this pull request. ## Screenshots/Recordings <!-- If this includes a UI change, attach screenshots on both laptop and mobile. --> ## Checklist - [x] My code follows the project's existing style. - [x] I have reviewed my changes. - [x] Only the necessary files and lines have been changed. - [x] This PR is ready for review.
fix: inconsistent background in dark mode
All checks were successful
kennel/build skipped: no services or static sites defined
Test / Unit tests (push) Successful in 1m23s
Test / Integration tests (push) Successful in 1m47s
Test / Deploy to Production (push) Has been skipped
Test / Deploy to Dev (push) Successful in 2m8s
082c9ca6a8
remove uneccessary css and fix duplicate classes causing the issues.
klaipras changed title from fix: inconsistent background in dark mode to WIP: fix: inconsistent background in dark mode 2026-10-03 20:29:38 +00:00
klaipras changed title from WIP: fix: inconsistent background in dark mode to fix: inconsistent background in dark mode 2026-10-03 20:29:39 +00:00
Member

Since this is a UI change, can you add a screenshot of the change? A "before" screenshot would also be useful for comparison (you can just run ui_change locally to get that).

Since this is a UI change, can you add a screenshot of the change? A "before" screenshot would also be useful for comparison (you can just run ui_change locally to get that).
Author
Member

@annadavi Added Screenshots

@annadavi Added Screenshots
annadavi approved these changes 2026-10-04 03:31:39 +00:00
annadavi left a comment

Be sure to update the branch with merge/rebase before creating a merge commit!

Be sure to update the branch with merge/rebase before creating a merge commit!
Merge branch 'ui_change' into dark-mode-fix
All checks were successful
kennel/build skipped: no services or static sites defined
Test / Unit tests (push) Successful in 1m25s
Test / Integration tests (push) Successful in 1m44s
Test / Deploy to Production (push) Has been skipped
Test / Deploy to Dev (push) Successful in 1m50s
a29a410ac1
klaipras merged commit 5b99a43a5d into ui_change 2026-10-04 14:54:30 +00:00
Sign in to join this conversation.
No description provided.