Skip to content

fix: tweak custom colors#71

Merged
KaiVandivier merged 16 commits intomainfrom
custom-colors-fixes
Apr 13, 2026
Merged

fix: tweak custom colors#71
KaiVandivier merged 16 commits intomainfrom
custom-colors-fixes

Conversation

@kabaros
Copy link
Copy Markdown
Collaborator

@kabaros kabaros commented Apr 9, 2026

@kabaros
Copy link
Copy Markdown
Collaborator Author

kabaros commented Apr 9, 2026

how it looks now

  1. Text shadows for title: white behind black text; black behind white text, with 0.5 opacity
  2. Online Badge: background is black with light-opacity (0.25) - and border that is text-color with opacity 0.15
  3. Notification Badges: it is now text-color as background

Other opinions:

  1. We're better without the text shadows when using custom colors - they work sometimes better, but sometimes not [Mozafar]
  2. On light backgrounds, should we have black logo rather than the blue one [Kai]
  3. Should we have the notification badge as green with white text always [Kai]
    ..
image image image image image image image

notification with fixed green badge

This is how it would look like with green badge
image
image
image


text without shadow

This is how it would look like without the text shadow for the title (sliiiight difference)
image
image
image

@kabaros
Copy link
Copy Markdown
Collaborator Author

kabaros commented Apr 9, 2026

@KaiVandivier @cooper-joe I added some screenshots here for how the custom colors look like with the latest tweaks - there are some pending questions, let me know about them (and if I missed anything)

@KaiVandivier KaiVandivier requested a review from cooper-joe April 10, 2026 07:48
@cooper-joe
Copy link
Copy Markdown
Member

Thanks for the examples @kabaros. My suggestions:

Text shadows

I think the shadow only benefits white text. So if we can, conditionally apply the text-shadow in the dark-background-white-text scenario. If that isn't possible, remove the text-shadow for custom colours.

Online badge

I don't think the border is needed, the opacity background does the job of containing the element. The border looks too much like a button on some colour combinations. I'd also suggest reducing background opacity to 0.2.

Logo blue vs. black

Difficult to say what is best here. Blue is nice on desaturated lights, but can look bad and give a "shimmering" effect on some light colours. So, I'd vote black, as it has the least risk of contributing to a poor UX.

Notification badge colour

Agreed, keep as green-background-with-white-text always. (Probably a good time for us to consider improving these badges too, related ticket)

@kabaros kabaros marked this pull request as ready for review April 10, 2026 16:06
@kabaros kabaros requested a review from KaiVandivier April 10, 2026 16:06
@kabaros
Copy link
Copy Markdown
Collaborator Author

kabaros commented Apr 13, 2026

here are updated screenshots after the tweaks:

image image image image image image

Copy link
Copy Markdown
Member

@cooper-joe cooper-joe left a comment

Choose a reason for hiding this comment

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

Nice, those changes look good for design. Approved with the below fix.

Last issue I can see is the black DHIS2 logo version is larger than the standard white one, seems like the `svg { height: 20px; width: 22px; } isn't being applied.

Copy link
Copy Markdown
Collaborator

@KaiVandivier KaiVandivier left a comment

Choose a reason for hiding this comment

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

Something happened to the online badge's circle icon... looking into it

@kabaros kabaros force-pushed the custom-colors-fixes branch from e9cacc2 to 135a927 Compare April 13, 2026 14:02
Copy link
Copy Markdown
Collaborator

@KaiVandivier KaiVandivier left a comment

Choose a reason for hiding this comment

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

A couple comments added -- I think the SonarQube items are reasonable too

Still mainly waiting on the black logo size

Comment thread src/components/header-bar/title.jsx Outdated
Comment thread src/components/header-bar/online-status.styles.jsx Outdated
@sonarqubecloud
Copy link
Copy Markdown

@KaiVandivier
Copy link
Copy Markdown
Collaborator

  1. Fixed black icon size
  2. Handled "no color selected" case
  3. Handled SonarQube warnings
Screenshot 2026-04-14 at 01 11 50

@KaiVandivier KaiVandivier merged commit a5f3ff6 into main Apr 13, 2026
5 checks passed
dhis2-bot added a commit that referenced this pull request Apr 13, 2026
## [1.12.2](v1.12.1...v1.12.2) (2026-04-13)

### Bug Fixes

* tweak custom colors ([#71](#71)) ([a5f3ff6](a5f3ff6))
@dhis2-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.12.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants