Copilot commented on code in PR #143:
URL: https://github.com/apache/sedona-db/pull/143#discussion_r2374560097
##########
docs/stylesheets/extra.css:
##########
@@ -107,18 +107,22 @@
Swap Logo ONLY in Mobile Navigation
==========================================================================
*/
-/* 1. Hide the default logo image inside the mobile nav */
-.md-nav--primary .md-nav__title .md-logo img {
- display: none;
-}
-
-/* 2. Apply the new mobile logo as a background image */
-.md-nav--primary .md-nav__title .md-logo {
- display: block;
- height: 48px; /* Adjust size as needed */
- width: 100%;
- background-image: url('../image/sedona_logo_symbol_white.svg'); /*
IMPORTANT: Update this path */
+/*
+ Target the logo link (<a> tag) directly and apply the new logo
+ as a background image, overriding the theme's default icon.
+*/
+.md-sidebar--primary .md-nav__title a.md-nav__button.md-logo {
+ background-image: url('/docs/image/sedona_logo_symbol.svg');
Review Comment:
The absolute path `/docs/image/sedona_logo_symbol.svg` may not work
correctly in all deployment environments. Consider using a relative path like
`../image/sedona_logo_symbol.svg` to ensure the logo displays properly across
different hosting configurations.
```suggestion
background-image: url('../image/sedona_logo_symbol.svg');
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
