Skip to content

Fix theme initilization issue#237

Open
saikiran09-ask wants to merge 2 commits into
imDarshanGK:mainfrom
saikiran09-ask:fix-theme-initialization
Open

Fix theme initilization issue#237
saikiran09-ask wants to merge 2 commits into
imDarshanGK:mainfrom
saikiran09-ask:fix-theme-initialization

Conversation

@saikiran09-ask
Copy link
Copy Markdown

Description

This PR fixes the theme initialization issue in "frontend/index.html" by improving how the theme state and icon are restored during page load.

What was changed

  • Fixed incorrect theme initialization behavior
  • Ensured the correct icon appears for dark/light mode
  • Improved consistency during theme switching
  • Cleaned up theme initialization logic for better maintainability

Related Issue

No linked issue.

Type of change

  • Bug fix

Checklist

  • I have read CONTRIBUTING.md
  • My branch is up to date with "main"
  • I have tested the changes locally
  • I have not introduced duplicate functionality
  • My PR title follows the required format

Testing

  • Verified theme toggle works correctly
  • Tested dark and light mode initialization
  • Confirmed icon updates properly after page reload

Additional Notes

This update improves reliability of the frontend theme system without affecting existing functionality.

Screenshot (33) Screenshot (32)

Test evidence

  • Manual frontend testing completed

  • Verified theme initializes correctly on page load

  • Tested dark mode and light mode switching

  • Confirmed theme icon updates properly

  • Reloaded page to ensure theme persistence works correctly

@saikiran09-ask
Copy link
Copy Markdown
Author

Implemented the requested fix for theme initialization and verified dark/light mode behavior manually. Kindly review the PR. Thank you!

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.

1 participant