For any needed usage, be able to use tokens (like file extension) in download link classes would be helpfull.
Can this be done?
Thanks

edit: or add extension class in code like in attached patch.

Comments

PhilY created an issue. See original summary.

phily’s picture

Issue summary: View changes
StatusFileSize
new645 bytes
danflanagan8’s picture

Hi, PhilY,
Thanks for the issue and the preliminary patch. I think adding token support for the custom css classes is a good idea. I'll work on it!

danflanagan8’s picture

Hi again, PhilY,
Sorry for the delay. Here's a patch that brings support for tokens into the custom classes. I've updated and added test coverage, so I'm pretty sure it works. However, it would be great if you could test it out before I merge and add it to a release. Thanks!

danflanagan8’s picture

Status: Active » Needs review
phily’s picture

Hi danflanagan8,
I've tested your patch #4 and for the usages I have, it works perfectly.
Many thanks.

PS: you don't have to be sorry for a voluntary contribution delay ;-)
Stay safe.

danflanagan8’s picture

Fair enough! Thanks for testing. I'll get this out in the next release.

Cheers,
Dan

danflanagan8’s picture

Status: Needs review » Reviewed & tested by the community

  • danflanagan8 committed 25c6a1d on 8.x-1.x
    Issue #3119353 by danflanagan8, PhilY: Allow tokens in "Custom CSS...

  • danflanagan8 committed 78fcece on 8.x-1.x
    Issue #3119353 by danflanagan8, PhilY: Allow tokens in "Custom CSS...
danflanagan8’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.