n-ce / ytify

Opus Audio Streaming Web App for YouTube. Stop watching, Listen & Save Data.
https://ytify.netlify.app
GNU General Public License v3.0
145 stars 48 forks source link

Fix the paths for the woff2 files in main.css #118

Closed JBLorincz closed 1 year ago

JBLorincz commented 1 year ago

Hi, I found this repository by chance on Github and I thought it was interesting. When I ran it for the first time, I saw this:

image

As you can see, the icons are not showing properly. I looked in the console and saw this: image

I saw that the path in main.css for these Woff2 files needed modification, as the old configuration was attempting to look into an assets folder that resided below main.css. With these changes the icons are now working:

image

netlify[bot] commented 1 year ago

Deploy Preview for ytify ready!

Name Link
Latest commit a979f7ef16f4ffcc0e787cd04d71523e7acf8847
Latest deploy log https://app.netlify.com/sites/ytify/deploys/64bb66bb60acb90008004cfc
Deploy Preview https://deploy-preview-118--ytify.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 site configuration.

n-ce commented 1 year ago

It works fine on my local as well as production though. I also see that in your fixed case, the site isn't still loading to its entirety. So I wonder what's going on

JBLorincz commented 1 year ago

It works fine on my local as well as production though. I also see that in your fixed case, the site isn't still loading to its entirety. So I wonder what's going on

Hi, looking at it again it appears to that "assets/NotoSans.woff2" is served by your web server, thereby letting the path be accessible globally. So this is not an issue.