Divested-Mobile / Mull-Fenix

Build scripts for a web browser built upon Mozilla technology
GNU Affero General Public License v3.0
566 stars 15 forks source link

generic unchangeable "Web site" label for some home screen shortcuts #233

Open lmamane opened 1 month ago

lmamane commented 1 month ago

Reproduction instructions:

Expected result:

Actual result:

Compare with many other websites, such as https://www.foo.be, https://www.debian.org, https://divestos.org where one gets the expected result.

SkewedZeppelin commented 1 month ago

are you able to reproduce this with upstream/official Firefox?

lmamane commented 1 month ago

F-Droid does not contain an app called "Firefox".

SkewedZeppelin commented 1 month ago

this is intended behavior

in the past it'd say "add to home screen" for adding a website and "install" for adding a PWA, but Mozilla merged them both to say "add to home screen"

when it is a PWA I guess they decided that you shouldn't be able to rename it so this is indeed an upstream issue you can report to them instead

lmamane commented 1 month ago

That's particularly unfortunate when the PWA does not give a title, and Firefox/Fennec/Mull replaces that with a generic "Web site".

SkewedZeppelin commented 1 month ago

Fallback string is here fwiw https://searchfox.org/mozilla-central/search?q=mozac_feature_pwa_default_shortcut_label&path=&case=false&regexp=false

I can't repro the fallback label on github.com however

lmamane commented 1 month ago

Reported upstream as https://bugzilla.mozilla.org/1908588 I reproduced on F-Droid's "Fennec" app whose description says it is actually Fenix.

I can't repro the fallback label on github.com however

Yes, you are right, with github.com one gets the label "GitHub", my bug report was not the most well written on that point.