- Issue created by @marcoscano
- last update
over 1 year ago 61 pass - ๐ช๐ธSpain marcoscano Barcelona, Spain
Turns out I was wrong suspecting from the long paths, that was my IDE playing a trick with me. Turns out the keys were fine and not trimmed at all.
On the other hand, I did find that sometimes (I'm guessing when the namespace pointed to a custom site-specific (in a multi-site) theme directory) we had a trailing slash, which made the
str_replace()
not do the job it was supposed to do.Attached patch that seems to solve the issue for us.
- Status changed to Needs review
over 1 year ago 8:57am 4 July 2023 - Status changed to RTBC
over 1 year ago 8:39am 7 July 2023 - e0ipso Can Picafort
This is working good in our install. Thanks for the patch @marcoscano!
- ๐ง๐ทBrazil julio_retkwa Balneรกrio Camboriรบ
#3 working fine for me as I had to follow the new standard described here: https://www.drupal.org/docs/contributed-modules/components/registering-t... โ
TL:DR
I had to replace my theme.info.yml section from this:component-libraries: base: paths: - components/00-base atoms: paths: - components/01-atoms molecules: paths: - components/02-molecules organisms: paths: - components/03-organisms
To something like:
components: namespaces: base: components/00-base atoms: components/01-atoms molecules: components/02-molecules organisms: components/03-organisms templates: components/04-templates pages: components/05-pages
- last update
about 1 year ago 61 pass - ๐บ๐ธUnited States joycehutch Kansas City, MO
Thanks @julio_retkwa! I also had to change the theme.info.yml with the same, because of pattern lab configuration. This was helpful!
- ๐จ๐ฆCanada joelpittet Vancouver
I wonder if a BC layer can be added to help ease people off of 2.x?
Thanks @julio_retkwa for the TL;DR
- ๐ฆ๐บAustralia pasan.gamage
Thanks @julio_retkwa
Saved me a heap of time! - last update
about 1 year ago 61 pass - last update
about 1 year ago 61 pass -
RobLoach โ
committed 0b46c285 on 3.x
Issue #3368589 by marcoscano, RobLoach: Trim the right-hand slash on...
-
RobLoach โ
committed 0b46c285 on 3.x
- Status changed to Fixed
11 months ago 7:04pm 25 January 2024 - ๐จ๐ฆCanada RobLoach Earth
Small fix to switch the string to use single-quotes instead of double quotes. We're not processing any variables in the string so ' is what we want. Thanks all!
Automatically closed - issue fixed for 2 weeks with no activity.