allow theme switching based on taxonomy:name property

CommentFileSizeAuthor
#1 0001-taxonomy-name-property.patch2.3 KByechuah
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yechuah’s picture

yechuah CreditAttribution: yechuah commented
Status: Active » Needs review
FileSize
0001-taxonomy-name-property.patch2.3 KB

created patch for this feature

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner commented
Status: Needs review » Needs work

Basically the patch looks good. But it lacks support for i18n.

I'm not sure, but probably we should add the translations to the name array. Or create a second property that includes the translations.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
CreditAttribution: mkalkbrenner commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)

I close this issue to clean up the queue.
Feel free to re-open it if you can provide a patch with i18n support.

  • mkalkbrenner committed aaaf739 on 8.x-1.x-playground authored by yechuah 
    Issue #1942444 by yechuah: Added taxonomy:name property.