barrio_path_images required to be configured when creating a subtheme

2023-09-01 15:19:51 Error xx.xx.xx.xx 404 GET /themes/custom/contrib/bootstrap_barrio/images/icons/text-plain.png HTTP/1.0 https://mywebsite/article/getting-started-ddev-and-apache-solr-service-addon Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/115.0.0.0 Safari/537.36 919 Apache SSL/TLS access 2023-09-01 15:19:51 Error xx.xx.xx.xx 404 GET /themes/custom/contrib/bootstrap_barrio/images/icons/text-html.png HTTP/1.0 https://mywebsite/article/getting-started-ddev-and-apache-solr-service-addon Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/115.0.0.0 Safari/537.36 918 Apache SSL/TLS access 2023-09-01 15:19:51 Error xx.xx.xx.xx 404 GET /themes/custom/contrib/bootstrap_barrio/images/required.svg HTTP/1.0 https://mywebsite/article/getting-started-ddev-and-apache-solr-service-addon Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/115.0.0.0 Safari/537.36 35.1 K Apache SSL/TLS access 2023-09-01 15:19:51 Error xx.xx.xx.xx 404 GET /themes/custom/contrib/bootstrap_barrio/images/icons/text-x-generic.png HTTP/1.0

Comments & Activities

Issue created by @2dareis2do
First commit to issue fork.
@sim_1 opened merge request.
Status changed to Needs review about 1 year ago10:31pm 6 September 2023
Comment about 1 year ago →
🇺🇸United States sim_1
I accidentally committed the change with the default commit message, but there is now a MR ready for review. I added $barrio_path_images: "../images/"; to the _variables.scss file like the issue description suggested. This fixed the build issue when I tried it in my sub theme.
Comment about 1 year ago →
🇺🇸United States sim_1
The other thing I'll say though, is that this is also fixed in the barrio project in version 5.5.11. So it might not make sense to add this variable, but instead mark this as an issue that is fixed upstream.
Comment about 1 year ago →
🇬🇧United Kingdom 2dareis2do
Hi @sim_1

Thanks for looking into this.

I think you are correct in your analysis.

I will raise an issue on the barrio project and reference this here. If I understand correctly I think your suggestion that $barrio_path_images: "../images/" should go in the parent theme is correct.

https://git.drupalcode.org/project/bootstrap_barrio/-/commit/e6fa5dc458d...
Comment about 1 year ago →
🇬🇧United Kingdom 2dareis2do
Comment 9 months ago →
🇺🇸United States tommyk
The fix of adding $barrio_path_images: "../images/"; to the subtheme SCSS works for me. The parent theme works on its own, but using this theme does not without that variable set.