Skip to content

feat: add french dictionary in firefox#165

Closed
hexaltation wants to merge 1 commit into
cloud-gouv:mainfrom
hexaltation:firefox-french-dictionnary
Closed

feat: add french dictionary in firefox#165
hexaltation wants to merge 1 commit into
cloud-gouv:mainfrom
hexaltation:firefox-french-dictionnary

Conversation

@hexaltation

Copy link
Copy Markdown

Securix installs by default french language pack

    programs.firefox = {
      enable = true;
      languagePacks = [
        "fr"
        "en-US"
      ];

But it lacks french dictionary to let the user to be able to choose french language orthographic corrections.

This PR proposes to fix this by adding the official french dictionary extension.

Comment thread modules/tools/firefox.nix
// (listToAttrs [
(extension "ublock-origin" "uBlock0@raymondhill.net")
(extension "bitwarden-password-manager" "{446900e4-71c2-419f-a6a7-df9c091e268b}")
(extension "dictionnaire_francais1" "fr-dicollecte@dictionaries.addons.mozilla.org")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting addition but we would really like to make Sécurix independent from France and have it as an EU option, could you think of a contribution that would move FR-specific customizations in its own module?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hexaltation did you see @rlahfa-dinum comment ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jdauphant-dinum We talked over DMs and @hexaltation explained that he could not do all the changes by himself. I will take care of doing this change.

@rlahfa-dinum

Copy link
Copy Markdown
Contributor

Done in #182.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants