Skip to content

Conversation

@iShelar
Copy link
Contributor

@iShelar iShelar commented Dec 9, 2025

- Add CSS variables for light and dark themes in style.css
- Create theme-toggle.js to manage theme switching and local storage
- Update base.html to include theme initialization script
- Enhance header.html with a theme toggle button for user interaction
- Refactor existing styles to utilize CSS variables for consistency
image image

    
    - Add CSS variables for light and dark themes in style.css
    - Create theme-toggle.js to manage theme switching and local storage
    - Update base.html to include theme initialization script
    - Enhance header.html with a theme toggle button for user interaction
    - Refactor existing styles to utilize CSS variables for consistency
@iShelar iShelar mentioned this pull request Dec 9, 2025
@iShelar
Copy link
Contributor Author

iShelar commented Dec 11, 2025

Hello. I’m excited about this small improvement to the site’s theme and styles. Whenever you have time, I’d really appreciate your review and any suggestions you have.

@ArcaneIRE @matrixise

@ArcaneIRE
Copy link
Member

Hey! The example image looks great! Thanks for your time and effort

Sorry to keep you waiting, unfortunately I've been ill since last Friday. I'll try to take the chance to review it this weekend.

I'm not sure about third party UI components, I'll also take a look at that if @matrixise doesn't get back to you by then


#header h1 #pyhead {
color: #336600;
color: var(--brand-primary);
Copy link
Member

Choose a reason for hiding this comment

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

Using variables is a nice refactor even outside of outside of the new theme!

<h1><img src="{% static 'img/pythonie.png' %}"
class="logo img-rounded"/>
<span id="pyhead">Python Ireland</span></h1></a>
<a class="header-brand" href="/">
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for formatting!

@ArcaneIRE ArcaneIRE merged commit dc7376d into PythonIreland:master Dec 13, 2025
1 check passed
@matrixise
Copy link
Contributor

Hey @iShelar and @ArcaneIRE,

Thank you so much for this fantastic contribution! The dark theme implementation looks amazing, and the refactoring to use CSS variables is a really nice improvement.

@iShelar - I really appreciate the time and effort you put into this feature. The screenshots look great, and it's wonderful to see such quality contributions to the project.

@ArcaneIRE - Thanks for the thorough review and for merging this. Your responsiveness is much appreciated.

It's genuinely exciting to see the project moving forward like this. I'm sincerely sorry for my unavailability lately - but seeing contributions like these makes me really happy. This is exactly the kind of progress we want to see!

Thanks again to both of you. Keep up the great work! 🎉

@iShelar
Copy link
Contributor Author

iShelar commented Dec 27, 2025

Thanks so much, @matrixise really appreciate the kind words and encouragement.

I’ve learned a lot about the project from these two PRs, and I’m looking forward to picking up more issues and contributing again soon.

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.

3 participants