modalman
Forum Replies Created
-
I tested 1.6.7-beta.2 and am able to change the HTML tag to an anchor element and add a custom CSS class, but unfortunately without an href attribute on the anchor tag, it still isn’t focusable.
I suppose we could use JavaScript on our site to add href=”#” to the a tags, but that would involve editing our site’s theme which we are trying to avoid.
The ability to change the HTML tag and add custom CSS classes is nice, but it seems this has to be done for each popup trigger. It would instead be nice to have a site-wide setting in the plugin to make it output anchor tags for popup triggers by default (instead of spans), as well as a site-wide setting to add href=”#” to the anchor tags. These defaults could then be overridden for an individual popup trigger.
The reason for this logic is that we have a lot of editors who might like to add popups to their pages, and they are probably not going to change the HTML Tag or adjust other options for each popup trigger that they add, so it would be good to be able to set these options as site-wide defaults.
Might your team be able to implement such a feature?
- This reply was modified 8 years, 4 months ago by modalman.
Here is the URL:
http://aarsi.ccsd.net/accountability/nspf/It seems the links aren’t anchor tags, but <span>’s, so they aren’t focusable, meaning the :focus pseudo-element has no effect. Is there a way to make them anchor tags?
If not, maybe there is a way to add the tabindex attribute to the <span>’s to make them tab-focusable, however the plugin does not expose this option. It seems the right way to do it would require parsing the HTML WordPress generates for a page and either adding the tabindex attribute to the spans or replacing them with anchor tags. Is it possible to write a sort of PHP middleware function for WordPress to run all pages through to do this, or (feature request) could this be added to the plugin?
- This reply was modified 8 years, 5 months ago by modalman.
- This reply was modified 8 years, 5 months ago by modalman. Reason: Fix formatting
- This reply was modified 8 years, 5 months ago by modalman. Reason: Fix formatting 2
- This reply was modified 8 years, 5 months ago by modalman. Reason: Fix formatting 3