Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
lmamane opened this issue Jul 18, 2024 · 6 comments
Open

Comments

@lmamane
Copy link

lmamane commented Jul 18, 2024

Reproduction instructions:

Expected result:

  • intermediate dialog that proposes as label for the shortcut the title of the web page, but allows one to change the label
  • after clicking "add" on the intermediate dialog, get final dialog (which may be from my launcher and not from Mull) with text "touch and hold the widget to move it around the home screen" and "cancel" and "Add to home screen" buttons.

Actual result:

  • no intermediate dialog, one gets directly the final dialog

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
Copy link
Member

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

@lmamane
Copy link
Author

lmamane commented Jul 18, 2024

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

@SkewedZeppelin
Copy link
Member

SkewedZeppelin commented Jul 18, 2024

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
Copy link
Author

lmamane commented Jul 18, 2024

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

@SkewedZeppelin
Copy link
Member

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
Copy link
Author

lmamane commented Jul 18, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants