Hi,
thanks for this module!
Is it possible to add a fontawesome watch symbol to dates next to the field label?
thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3156614-label-class-3.patch | 4.89 KB | gausarts |
Hi,
thanks for this module!
Is it possible to add a fontawesome watch symbol to dates next to the field label?
thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3156614-label-class-3.patch | 4.89 KB | gausarts |
Comments
Comment #2
gausarts commentedSorry for delay. I hope you can make one way or another.
Not via this module, I am afraid. Two options:
.field--NAME .field__label::before { /* regular icon rules, normally :*/ content: \CODE; }field.html.twig:field__label fa fa-watch, of course conditions must be applied accordingly.We'll keep it a Feature for a while in case anyone is feeling invited to patch. Thanks.
Comment #3
gausarts commentedThis will add the new option for field label.
Comment #4
gausarts commentedOn second thought, the above will add extra useless divities and might break existing theming.
But can be useful if you want to hook into field template. However this can be added as an option in the future, like what Slick has, if anyone needed one which I highly doubted since it will add complication to theming field.
We'd better put the label inside the existing template
linkicon.html.twiginstead.This way at future version we can even droplinkicon.html.twigfor coreitem-list.html.twigto save us another theme to maintain which also support custom title with classes. Since both have similarities.Comment #5
gausarts commentedHmm, apparently
item-list.html.twighas notitle_attributes, doh.No future for replacing
linkicon.html.twigwith this option in.So leaving us to decide whether to go with
field.html.twigorlinkicon.html.twig.Thoughts?
Comment #6
gausarts commentedOn third thought, I think it is fine since it is being optional you can always leave the option empty in case breaking for the recommended #2.
The obvious disadvantage is extra field divities if anyone can live with that :)
Comment #8
gausarts commentedWe have 20 modules to update for D10. Execution matters.
This will do for now. As always improvements or misses may follow behind the screen or bug reports.
Committed. Thank you for contribution.