Skip to content

fix: correct dropdown arrow icon path for production#196

Merged
Infi-Knight merged 3 commits intomainfrom
ravi/devport-32
Feb 4, 2026
Merged

fix: correct dropdown arrow icon path for production#196
Infi-Knight merged 3 commits intomainfrom
ravi/devport-32

Conversation

@Infi-Knight
Copy link
Contributor

@Infi-Knight Infi-Knight commented Feb 4, 2026

PR Checklist

  • Linked issue added (e.g., Fixes #123)
  • I have run bun run format to ensure code is properly formatted
  • I have verified that bun run lint passes without errors
  • If blog post was added:
    • Ensure images have been optimised
    • Update dates to reflect the actual publishing date when merged (file names, folder names, and frontmatter)

Summary

Fixes the dropdown arrow icon not displaying in production (404 error).

The navigation dropdown arrow icon was referencing an incorrect path in CSS: content: url(/public/img/arrow-menu.svg);

In Astro, the /public/ directory is a build-time concept. Files from public/ are served from the site's base path (/developers/) in production, not from /public/. This caused:

Chrome: Icon not visible
Safari: 404 error in console

Solution
Changed the path to use the correct production URL:

content: url(/developers/img/arrow-menu.svg);

@Infi-Knight Infi-Knight self-assigned this Feb 4, 2026
@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for developers-preview ready!

Name Link
🔨 Latest commit afd8ce9
🔍 Latest deploy log https://app.netlify.com/projects/developers-preview/deploys/6982f421bc3d410008e7f818
😎 Deploy Preview https://deploy-preview-196--developers-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Infi-Knight Infi-Knight changed the title fix: missing icons from the dropdown fix: correct dropdown arrow icon path for production Feb 4, 2026
@Infi-Knight Infi-Knight marked this pull request as ready for review February 4, 2026 07:32
Copy link
Contributor

@Anca2022 Anca2022 left a comment

Choose a reason for hiding this comment

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

LGTM :)
Thank you for the explanation 😄

@JoblersTune
Copy link
Contributor

Great @Infi-Knight, thanks. Please go ahead and merge.

@Infi-Knight Infi-Knight merged commit 85e5068 into main Feb 4, 2026
5 checks passed
@Infi-Knight Infi-Knight deleted the ravi/devport-32 branch February 4, 2026 08:38
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