Closed thedemonsid closed 4 weeks ago
The changes made in this pull request involve updating the srcDark
property of the logo configuration in the docusaurus.config.js
file for the Docusaurus documentation site. The logo's source URL has been changed from a relative path to an absolute URL, ensuring that the dark mode logo is consistently fetched from a specified external source. No other significant modifications to logic or functionality were made.
File | Change Summary |
---|---|
docs/docusaurus.config.js | Updated srcDark in navbar.logo from a relative path to an absolute URL for dark mode logo. |
Objective | Addressed | Explanation |
---|---|---|
Implement a version of the logo optimized for dark mode (#189) | β |
π° In the dark, our logo shines bright,
A new URL brings it to light.
From shadows it leaps, clear and bold,
A tale of contrast, now retold.
Hooray for the change, letβs hop with glee,
A logo that stands out, for all to see! π
docs/docusaurus.config.js (1)
`38-38`: **Approved: Logo visibility issue addressed. Consider improvements for reliability.** The change successfully addresses the logo visibility issue in dark mode by using the light mode logo URL for both `src` and `srcDark`. This ensures the logo remains visible in both modes. However, consider the following suggestions for improvement: 1. Host the logo image within the project repository to reduce dependency on external servers. 2. If available, use a dark mode specific logo for better aesthetics. Please verify the logo visibility in dark mode with this change. You can use the following script to check the configuration:β Verification successful
**Logo configuration verified successfully.** The `logo` settings in `docs/docusaurus.config.js` have been verified. Both `src` and `srcDark` are set to the same absolute URL, ensuring consistency across light and dark modes.π Scripts executed
The following scripts were executed for the analysis: Script: ```shell #!/bin/bash # Description: Verify the logo configuration in docusaurus.config.js # Test: Check if src and srcDark are the same and use an absolute URL rg -A 5 'logo: \{' docs/docusaurus.config.js ``` Length of output: 332
@thedemonsid Can you show the before and after for this pr?
Fixes #189
Update the dark mode logo configuration in
docs/docusaurus.config.js
.srcDark
property to point to the light mode logo URL.For more details, open the Copilot Workspace session.
Summary by CodeRabbit
New Features
Bug Fixes