mozilla / multi-account-containers

Firefox Multi-Account Containers lets you keep parts of your online life separated into color-coded tabs that preserve your privacy. Cookies are separated by container, allowing you to use the web with multiple identities or accounts simultaneously.
https://addons.mozilla.org/firefox/addon/multi-account-containers/
Mozilla Public License 2.0
2.64k stars 323 forks source link

Show error message and troubleshooting steps on contextual identities error #2598

Open brizental opened 7 months ago

brizental commented 7 months ago

Before submitting your pull request

Description

This is just a suggestion. See screenshot of what the error looks like:

Screenshot 2023-11-17 at 18 56 41

Two things:

  1. That is not a content suggestion. I couldn't come up with something good so I just added a sentiment there for now 🆘
  2. I had no idea how to add new strings. I added something directly in the l10n submodule. Here it is, just for illustration purposes: https://github.com/brizental/multi-account-containers-l10n/pull/1/files

Type of change

Tag issues related to this pull request:

dannycolin commented 7 months ago

I had no idea how to add new strings. I added something directly in the l10n submodule. Here it is, just for illustration purposes: https://github.com/brizental/multi-account-containers-l10n/pull/1/files

You can open a PR on the upstream l10n repository and simply mention that it depends on this PR to be merged.

That is not a content suggestion. I couldn't come up with something good so I just added a sentiment there for now 🆘

Looks good to me.