Re: python-sphinx_rtd_theme update: comments requested
Le 2020-07-02 10:51, Miro Hrončok a écrit : On 01. 07. 20 19:34, Nicolas Mailhot via devel wrote: Le mercredi 01 juillet 2020 à 18:35 +0200, Miro Hrončok a écrit : Given the /usr/share font links in CSS won't work when the documentation is exposed via a webserver, I assume the docs are mostly intended to be browsed (possibly offline) from within Fedora anyway. It’s not that hard to expose /usr/share/fonts in a webserver (and pretty risk-free). Let's consider this scenario. The documentation is exposed on 1.2.3.4:8000 from machine A. On machine B, I open http://1.2.3.4:8000/ in my webbrowser. The HTML is loaded, it contains a CSS file reference. The CSS is loaded, it contains an /usr/share/... font link. Now I don't know if having the font /usr/share/... font installed on machine B makes it work, but certainly, th font from machine A won't be served this way. You’d need to declare the font files you want to serve as a relative path, in your CSS files, and that relative path matches the path your webserver serves /usr/share/fonts at. Not sure it is worth the effort but is is certainly doable Regards, -- Nicolas Mailhot ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
On 01. 07. 20 19:34, Nicolas Mailhot via devel wrote: Le mercredi 01 juillet 2020 à 18:35 +0200, Miro Hrončok a écrit : Given the /usr/share font links in CSS won't work when the documentation is exposed via a webserver, I assume the docs are mostly intended to be browsed (possibly offline) from within Fedora anyway. It’s not that hard to expose /usr/share/fonts in a webserver (and pretty risk-free). Let's consider this scenario. The documentation is exposed on 1.2.3.4:8000 from machine A. On machine B, I open http://1.2.3.4:8000/ in my webbrowser. The HTML is loaded, it contains a CSS file reference. The CSS is loaded, it contains an /usr/share/... font link. Now I don't know if having the font /usr/share/... font installed on machine B makes it work, but certainly, th font from machine A won't be served this way. That makes me thing. Wouldn't it be better to reference the fonts by name instead of full path? That way, if the font is installed on machine B (not Fedora machine necessarily) in any way, it works. -- Miro Hrončok -- Phone: +420777974800 IRC: mhroncok ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
Le mercredi 01 juillet 2020 à 18:35 +0200, Miro Hrončok a écrit : > > Given the /usr/share font links in CSS won't work when the > documentation is > exposed via a webserver, I assume the docs are mostly intended to be > browsed > (possibly offline) from within Fedora anyway. It’s not that hard to expose /usr/share/fonts in a webserver (and pretty risk-free). However old IE browsers are a plague security-wise, it’s much better to remove IE workarounds than help users keep an insecure browser that does not work all that well on free software webapps. Also, making the browser walk a remote location is not good for performance and will have all kind of interesting effects in restricted networks Regards, -- Nicolas Mailhot ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
On 01. 07. 20 17:37, Jerry James wrote: On Wed, Jul 1, 2020 at 5:21 AM Miro Hrončok wrote: I've worked on this in https://src.fedoraproject.org/fork/churchyard/rpms/python-sphinx_rtd_theme/commits/generator However, there is "tiny little problem" that makes it not work: https://github.com/rpm-software-management/rpm/issues/1297 Nice! Thank you, Miro, I like what you've done. I'll wait to see the resolution of your "tiny little problem" before doing anything more. Well I think that before this is resolved, we can only add the requirement manually to all the affected Fedora packages and check newly added packages regularly. Should the %{?python_provide...} line be removed? It seems to be redundant with the new python dependency generator. Yes, if this will not be backported (I assume it won't, my generator for example only targets rawhide+). I'm also thinking of purging all traces of html5shiv from the spec file, and rewriting the reference to it in layout.html to something like this: {# JAVASCRIPTS #} {%- block scripts %} What do you think? Given the /usr/share font links in CSS won't work when the documentation is exposed via a webserver, I assume the docs are mostly intended to be browsed (possibly offline) from within Fedora anyway. As such, I would be inclined to simply drop the html5shiv part entirely, rather than pointing to a code that's outside of Fedora's area of influence. Whoever wants to browse the docs online will do it on readthedocs anyway. -- Miro Hrončok -- Phone: +420777974800 IRC: mhroncok ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
On Wed, Jul 1, 2020 at 5:21 AM Miro Hrončok wrote: > I've worked on this in > > https://src.fedoraproject.org/fork/churchyard/rpms/python-sphinx_rtd_theme/commits/generator > > However, there is "tiny little problem" that makes it not work: > > https://github.com/rpm-software-management/rpm/issues/1297 Nice! Thank you, Miro, I like what you've done. I'll wait to see the resolution of your "tiny little problem" before doing anything more. Should the %{?python_provide...} line be removed? It seems to be redundant with the new python dependency generator. I'm also thinking of purging all traces of html5shiv from the spec file, and rewriting the reference to it in layout.html to something like this: {# JAVASCRIPTS #} {%- block scripts %} What do you think? Regards, -- Jerry James http://www.jamezone.org/ ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
On 01. 07. 20 2:07, Miro Hrončok wrote: On 01. 07. 20 0:47, Jerry James wrote: Third, any package that includes documentation generated from this package will need this: Requires: font(fontawesome) Requires: font(lato) Requires: font(robotoslab) Quick idea: Package an RPM dependency generator into python3-sphinx_rtd_theme that matches *.css files and when it find the "sphinx_rtd_theme" comment in them, it adds the requires. I can help with that. I've worked on this in https://src.fedoraproject.org/fork/churchyard/rpms/python-sphinx_rtd_theme/commits/generator However, there is "tiny little problem" that makes it not work: https://github.com/rpm-software-management/rpm/issues/1297 -- Miro Hrončok -- Phone: +420777974800 IRC: mhroncok ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Re: python-sphinx_rtd_theme update: comments requested
On 01. 07. 20 0:47, Jerry James wrote: Third, any package that includes documentation generated from this package will need this: Requires: font(fontawesome) Requires: font(lato) Requires: font(robotoslab) Quick idea: Package an RPM dependency generator into python3-sphinx_rtd_theme that matches *.css files and when it find the "sphinx_rtd_theme" comment in them, it adds the requires. I can help with that. If we want to get fancy, we can add a more general CSS-font requirements generator used for all packaged *.css files in Fedora, but frankly, I don't have the energy to spare for this. -- Miro Hrončok -- Phone: +420777974800 IRC: mhroncok ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org