Skip to content

Theme improvements - #41

Merged
gionn merged 4 commits into
masterfrom
new-look
Oct 18, 2026
Merged

gionn merged 4 commits into
masterfrom
new-look

Conversation

@andychiare

Copy link
Copy Markdown
Contributor

Proposta di lieve miglioramento del look

image

Comment thread _layouts/default.html
<div class="shell header-inner">
<a class="brand" href="{{ '/' | relative_url }}" aria-label="Montacchiello.dev, home">
<img class="logo-light" src="{{ '/images/logo-light.png' | relative_url }}" alt="Montacchiello.dev">
<img class="logo-dark" src="{{ '/images/logo.png' | relative_url }}" alt="Montacchiello.dev">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

screenshottami anche il tema scuro please

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Deploying montacchiello-dev with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1168d11
Status: ✅  Deploy successful!
Preview URL: https://3f8f27ce.montacchiello-dev.pages.dev
Branch Preview URL: https://new-look.montacchiello-dev.pages.dev

View logs

gionn added 2 commits October 18, 2026 08:45
The new look replaced the dark theme colours and dropped the dark logo. Restore the original dark palette, add a hero background colour it needs, and swap the logo back per theme. Fix link hover, hero accent and card shadows so they stay readable on the dark background.
The dark theme link hover rule had a higher specificity than the nav button hover, so it recoloured the button text to the same coral as its hover background. Move the hover colour into a variable so component rules apply again; footer and breadcrumb hovers follow from it.
@gionn gionn changed the title New look and new event Theme improvements Oct 18, 2026
The brand box clips the logo with a border radius, and its background was hardcoded to the light logo's colour, showing pale wedges at the corners in dark mode. Drive the background from a variable that matches each logo's own background.
@gionn
gionn merged commit fa63de7 into master Oct 18, 2026
2 checks passed
@gionn
gionn deleted the new-look branch October 18, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants