lovasoa / SQLpage

SQL-only webapp builder, empowering data analysts to build websites and applications quickly
https://sql.ophir.dev
MIT License
883 stars 64 forks source link

Support custom favicon location and manifest.json #317

Closed djyotta closed 1 month ago

djyotta commented 1 month ago

I like to provide a PWA for mobile devices.

Currently, no manifest.json is served at all (requires a meta link in HTML header as far as I know).

Similarly, the favicon is served, but the location is not configurable (relies on browser looking for domain.com/favicon.ico).

I'm not able to provide a PWA without the meta link in the shell component.

I guess I can work around with a custom shell component, but for now I forego the PWA.

The favicon I am also not serving, as it affects my entire domain root favicon.

Describe the solution you'd like

We should:

Describe alternatives you've considered

I tried routing the favicon via my reverse proxy, but it affects the whole domain. I need the meta link to tell the browser where to look other than the domain root.

These commits show what I would like (the tag in the HTML ):

But we need to implement it through the shell component.

lovasoa commented 1 month ago

Would you be ok with just adding a favicon and a manifest properties to the shell component ? Could you make a pr with just these changes ? I'm not sure testing and refreshing the existence of favicon.ico and manifest.json is worth it or even desired.

djyotta commented 1 month ago

Properties on the shell component is fine. I'll take a look when I get a moment.