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

[embedlite-components] Populate the technical details of the certificate page. JB#56370 OMP#JOLLA-541 #89

Merged
merged 1 commit into from Dec 22, 2021

Conversation

adenexter
Copy link
Contributor

@adenexter adenexter commented Dec 16, 2021

The description of the error is no longer encoded in the about:certerror
URL and needs to be constructed in the page itself.

Also fix the caret icon so it's apparent when the collapsed headings have
been expanded as with neither that nor content it wasn't apparent when
clicking worked.

@adenexter adenexter marked this pull request as draft December 16, 2021 09:42
@adenexter adenexter force-pushed the jb56370 branch 3 times, most recently from e704534 to 6812958 Compare December 21, 2021 01:18
@adenexter adenexter marked this pull request as ready for review December 21, 2021 01:20
Copy link
Member

@llewelld llewelld left a comment

Choose a reason for hiding this comment

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

These are some really nice changes.

The Finnish and Russian fluent files get installed in /usr/lib64/mozembedlite/chrome/chrome/content/localization/, but when I switch language to Finnish or Russian it doesn't pick up the changes (in the "Technical Details" section). Removing the ~/local/share/org.sailfishl/browser folder doesn't seem to help. Is this intentional behaviour?

overrides/aboutCertError.xhtml Outdated Show resolved Hide resolved
overrides/aboutCertError.xhtml Show resolved Hide resolved
@adenexter
Copy link
Contributor Author

The Finnish and Russian fluent files get installed in /usr/lib64/mozembedlite/chrome/chrome/content/localization/, but when I switch language to Finnish or Russian it doesn't pick up the changes (in the "Technical Details" section).

It appears that the locale file path substitution is filtered by known available translations. So either a packaged locale as identified by the name appearing in a multilocale.txt file in omni.ja, or an installed language pack.

Packaging the language packs would probably be the preferred way forward as it will cover translations for pages that are provided by gecko as well. But that's going to require new mirrors and a packaging repo.

@llewelld
Copy link
Member

It appears that the locale file path substitution is filtered by known available translations. So either a packaged locale as identified by the name appearing in a multilocale.txt file in omni.ja, or an installed language pack.

That's a real pain. It raises the question about whether it's worth shipping the fluent files for fi and ru at all. If I understand correctly, they'll not currently be used.

Could they be added at startup using L10nRegister.registerSource()?

@adenexter
Copy link
Contributor Author

Could they be added at startup using L10nRegister.registerSource()?

Yes, that's what the manifest entry did but doing it manually allows specifying the locales. And now the translations work.

Copy link
Member

@llewelld llewelld left a comment

Choose a reason for hiding this comment

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

Great, this is working nicely. Thank you. It'll also make things easier as more translations move to fluent in the future too, so really nice.

overrides/EmbedLiteOverrides.manifest Outdated Show resolved Hide resolved
@llewelld
Copy link
Member

Just a couple of small typos in the commit message, I think.

Also fix the caret icon so it's apparent when the collapsed headings have
been expanded as with neither that nor content it wasn't apparent when
clicking worked.

@rainemak
Copy link
Member

Rebased and fixed minor comment from @llewelld

…ate page. JB#56370 OMP#JOLLA-541

The description of the error is no longer encoded in the about:certerror
URL and needs to be constructed in the page itself.

Also fix the caret icon so it's apparent when the collapsed headings have
been expanded as with neither that nor content it wasn't apparent when
clicking worked.
@rainemak
Copy link
Member

Just a couple of small typos in the commit message, I think.

Also fix the caret icon so it's apparent when the collapsed headings have been expanded as with neither that nor content it wasn't apparent when clicking worked.

Fixed as well

@rainemak rainemak merged commit 08edfa8 into master Dec 22, 2021
@rainemak rainemak deleted the jb56370 branch December 22, 2021 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants