Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello, I was wondering if there's a way to make the icon display before the link, not after. I want it to appear more like a bullet. I tried messing with the css file and it didn't work :/
LOVE THIS MODULE, tia
Comment | File | Size | Author |
---|---|---|---|
#17 | IconPlacement-1672650-17.patch | 5.94 KB | elachlan |
| |||
#15 | IconPlacement-1672650-15.patch | 5.93 KB | elachlan |
#13 | IconPlacement-1672650-13.patch | 5.94 KB | elachlan |
#6 | extlink_icon_placement-1672650-6.patch | 3.91 KB | DrColossos |
Comments
Comment #1
lahode CreditAttribution: lahode commentedJust change in extlink.js $(this).after by $(this).before everywhere
Something nicer would be to include the option in extlink_admin_settings, like:
Then add line 24 after drupal_add_js(array('extlink' => array( :
And in your extlink.js file, change the line:
by:
Cheers
Comment #2
elachlan CreditAttribution: elachlan commentedCould be a possible inclusion. Submit a patch and we will see about including it.
Comment #3
DrColossos CreditAttribution: DrColossos commentedSince we needed this functionallity, I created a patch from the above. I slightly changed some of the wordings and changed the way the JavaScript is handled. Created against the latest dev
Comment #4
DrColossos CreditAttribution: DrColossos commentedThis patch has been around for quite some time, also with the refactored Javascript. We would love to have this feature available without patching the module. Please check the patch and let me know if anything needs changing.
Comment #5
elachlan CreditAttribution: elachlan commentedCould you re-roll it to the latest version?
Also I may need to get the OK from quicksketch before a release, but we should be able to include it in dev for now.
Comment #6
DrColossos CreditAttribution: DrColossos commentedSo, i finally managed to update the little patch. It's against the latest 7.x-1.x branch. I also rephrased some of the old wordings from my last patch.
Comment #8
elachlan CreditAttribution: elachlan commentedI have pushed the change through.
Thanks you!
Comment #9
DamienMcKennaNeeds an update to hook_uninstall() to remove the new variable.
Comment #10
osopolarThis works for me. As this was already fixed in the dev version I created a new issue for #9: #2508509: Delete only variables set by extlink on uninstall.
Comment #12
elachlan CreditAttribution: elachlan commentedComment #13
elachlan CreditAttribution: elachlan commentedComment #15
elachlan CreditAttribution: elachlan commentedComment #17
elachlan CreditAttribution: elachlan commentedComment #19
elachlan CreditAttribution: elachlan commented