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.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | file_download_link--class-tokens--3119353--4.patch | 15.32 KB | danflanagan8 |
| #2 | file_download_link-3119353-file_extension.patch | 645 bytes | phily |
Comments
Comment #2
philyComment #3
danflanagan8Hi, 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!
Comment #4
danflanagan8Hi 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!
Comment #5
danflanagan8Comment #6
philyHi 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.
Comment #7
danflanagan8Fair enough! Thanks for testing. I'll get this out in the next release.
Cheers,
Dan
Comment #8
danflanagan8Comment #11
danflanagan8