If you install this module because you want pop-up warnings, but no icons, it appears to break the module. A note to the user might help.
This is a slightly unusual use case for this module, but:
I actually installed it on a site where I wanted the pop-up warnings, but no icons.
It wasn't immediately obvious to me that, if I deselected both "Add icon" checkboxes, the module basically stops working.
Once I realized that, it was pretty easy to activate "Add icon to external links" and then override the CSS to remove the icon.
Would it be possible to add a note to the Configuration page letting users know that you have to add the icon in order to get the pop-up warnings to work?
Thanks!
Comments
Comment #1
elachlan CreditAttribution: elachlan commentedChanged title to be more relevant. Not sure if this is a bug or not. Marking as bug for now.
We could change the behaviour so that you can enable popups separately from adding the icons.
Maybe wrap the "a" tag in a span, then apply classes depending on settings?
Comment #2
elachlan CreditAttribution: elachlan commentedI have thought about this quite a bit.
I think it is best to leave the functionality as is and change the interface to make it more obvious that external links will only work when an icon is applied (which is its core function).
I suggest hiding or disabling the check box for popups when both the Add Icon check boxes are unchecked. Similar to the change in #1247644: a few language and UI improvements (patch).
Comment #3
elachlan CreditAttribution: elachlan commentedThe code I was referring to from #1247644: a few language and UI improvements (patch). We can add this too all the fields.
https://api.drupal.org/api/drupal/includes%21common.inc/function/drupal_process_states/7
Comment #4
elachlan CreditAttribution: elachlan commentedComment #5
elachlan CreditAttribution: elachlan commentedI am really unsure about this one. I feel like it could be a great addition to allow a warning message without the icon. But I also feel like it is outside the scope of the project.
We can change the design of the administration page to make it more obvious that you must have it enabled for the warnings to work. At the moment is is implied by the project as a whole.
Some more discussion on this is needed.
Comment #6
elachlan CreditAttribution: elachlan commented