Skip to content

Conversation

@nhxe
Copy link

@nhxe nhxe commented Jun 27, 2025

Pull Request Template

Description:

  • Enhanced the Swagger UI by updating the color scheme, adding GoFr branding (logo, title, favicon), and introducing a "Back to Docs" link for improved navigation.
  • Implemented a dark mode toggle for the documentation pages, allowing users to switch between light and dark themes with their preference saved.
  • These changes improve the user experience and accessibility of both the API documentation and the main docs.

Breaking Changes (if applicable):

  • No breaking changes are introduced by this PR.
  • All updates are backward compatible and only affect the frontend appearance and usability.

Additional Information:

  • No new dependencies or external libraries were added.

Checklist:

  • I have formatted my code using goimport and golangci-lint.
  • All new code is covered by unit tests.
  • This PR does not decrease the overall code coverage.
  • I have reviewed the code comments and documentation for clarity.

@Umang01-hash
Copy link
Member

@nhxe The change related to Swagger UI is not required as of now. We are rendering it in the default format of swagger, GoFr doesn't have a much role to it. We can remove it.

Secondly the toggle button for the website is not working. Have you tested it? Maybe you need to make the changes in website repo and please share the video/image of the changes if tested locally.

@nhxe
Copy link
Author

nhxe commented Jul 2, 2025

Thanks for the update.

I've opened a PR in the website repo for the toggle button fix. I haven’t attached a screenshot since I ran into some setup issues with npm install. Let me know if a visual confirmation is needed — I’ll sort it out and share accordingly.

gofr-dev/website#178

@Umang01-hash
Copy link
Member

Sure @nhxe. Closing this PR as the main feature is realted to website.

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.

2 participants