OpenRTX / openrtx.github.io

12 stars 15 forks source link

Fix favicons #18

Closed turnrye closed 1 year ago

turnrye commented 1 year ago

What

In #17 I reported that the favicons were not loading, which is a minor UX impairment. The favicons existed, the index.html just had the wrong relative path. So fix that.

Why

With this, my naive link checker shows that there are no dead links on the openrtx.org page. Once this is done, I can explore integrating a GitHub Action to avoid introducing new dead links. So this change in itself is somewhat helpful, but it is also an incremental change towards adding some stability to this project.

How

Correct the relative path on the index.html file. This works because the files already exist and are being served properly.

➜  openrtx.github.io git:(fix/favicons) ✗ curl --remote-name-all http://localhost:3000/icons/\{apple-touch-icon.png,favicon-32x32.png,favicon-16x16.png,safari-pinned-tab.svg\} -s -w "%{http_code}\n"
200
200
200
200

So all that was needed was to add /icons to the hrefs.

Verification

  1. Load the index page in a Chrome desktop UA

Expected result: the OpenRTX logo appears as the favicon (in chrome, this is left of the omnibar) Actual result: the OpenRTX logo appears

Before After
Screen Shot 2023-08-04 at 8 12 50 AM Screen Shot 2023-08-04 at 8 13 57 AM
silseva commented 1 year ago

Thanks!