-
Notifications
You must be signed in to change notification settings - Fork 25
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
More fonts are needed to avoid breaking rendering #49
Comments
so the issue there is that some of the /etc/fonts/conf.d files are not available in the snap, the directory has some entries which symlinks to ../conf.avail, which are there in the snap but some other are pointing to /usr, example /etc/fonts/conf.d/61-urw-nimbus-roman.conf -> /usr/share/fontconfig/conf.avail/urw-nimbus-roman.conf could snapd perhaps make /usr/share/fontconfig/conf.avail/ also available in the snap env? |
I've also reported a snapd issue about it on https://bugs.launchpad.net/snapd/+bug/1982577 |
So what would be a legit fix, modifying the |
Since snapd already gives access to the conf.d/conf.avail it would probably make sense to include now the new directory in /usr/share/fontconfig, the snapd team seemed ok on principle with a such change. |
We've avoided using the host system fontconfig configuration because it was prone to triggering incompatibilities. In particular, the config file parser treats any unknown element as a hard error. This last popped up when fontconfig added I think either of these options would be appropriate:
|
We need at least the URW fonts configuration for correct PDF rendering as reported against firefox on ubuntu#49 After discussion we decided to include the configurations provided in a default Ubuntu desktop installation for 3.38. It's not ideal but should help with consistency with non snaps apps and be an improvement. Going forward we should work with fontconfig upstream on a solution which allows use to import the system configuration instead.
After discussion we decided to bundled the fonts configuration from a default installation which should give a consistent behaviour with non snapped applications, submitted for review as #72 now. Going forward we want to work with the fontconfig upstream in a way to tag the configuration version in a way which would allow us to import the compatible system configuration |
We should have a build in the candidate channel in some hours, then it takes usually a bit longer to validate those and move them to stable, probably next week since we usually avoid to push stable updates on a friday |
the new snap is available in candidate now |
Thanks, I've been able to locally verify that it is properly the issue. |
We need at least the URW fonts configuration for correct PDF rendering as reported against firefox on ubuntu#49 After discussion we decided to include the configurations provided in a default Ubuntu desktop installation for 42. It's not ideal but should help with consistency with non snaps apps and be an improvement. Going forward we should work with fontconfig upstream on a solution which allows use to import the system configuration instead.
As documented on https://bugzilla.mozilla.org/show_bug.cgi?id=1767925 the fontconfig is missing some bits. This results in Firefox Snap package showing poor rendering both of webpages as well as PDF documents.
Copying everything using the following ensures the same rendering on both sides:
The text was updated successfully, but these errors were encountered: