Title: Invalid HTML output
Last modified: October 8, 2016

---

# Invalid HTML output

 *  Resolved [greatestview](https://wordpress.org/support/users/greatestview/)
 * (@greatestview)
 * [9 years, 8 months ago](https://wordpress.org/support/topic/invalid-html-output/)
 * First thanks for this plugin! I’m just getting used to IndieWeb related plugins,
   and I think the idea has great potential. I was just about to hack the plugin
   for my site, but I thought it would be much nicer to help improving the original
   plugin instead of doing my own thing. 🙂
 * I discovered that the HMTL output of the function _get\_syndication\_links_ is
   not valid, if one provides a _text\_before_ text, because an ul-element must 
   only contain li-elements, not even text. That may result in a strange browser
   behaviours, as in my current theme.
 * I have some suggestions to improve the HTML output:
 * 1. Replace line 238 `$synlinks = '<span class="relsyn"><ul>' . $options['text_before'];`
   with something like `$synlinks = '<span class="relsyn">' . $options['text_before'].'
   <ul>';`
 * 2. A minor thing: Currently the CSS class naming convention is not very consistent(.
   relsyn, .u-syndication, .syn-name), so I’d suggest namespacing the classes: .
   syndication-links, .syndication-links-name, etc. (or if that’s too long e.g. .
   synlinks, .synlinks-name, etc.)
 * 3. A minor thing: For individual styling it would help to apply a class to the
   ul-element (e.g. .syndication-links-list) and wrap $options[‘text_before] inside
   a span-element with a separate class (e.g. .syndication-links-text-before).
 * 4. Alternative/Addition 1: Moving the HTML to a **template file**, which can 
   be overridden by the theme.
 * 5. Alternative/Addition 2: Provide a **filter hook**, so that one can modify 
   the HTML output of the function _get\_syndication\_links_.
 * Besides that I have a feature question: Would it be possible to add an option
   to show the syndication links only on the full post, not the teaser on the homepage?
 * Thank you

Viewing 3 replies - 1 through 3 (of 3 total)

 *  Thread Starter [greatestview](https://wordpress.org/support/users/greatestview/)
 * (@greatestview)
 * [9 years, 7 months ago](https://wordpress.org/support/topic/invalid-html-output/#post-8405015)
 * _*Push*_
 *  Plugin Author [David Shanske](https://wordpress.org/support/users/dshanske/)
 * (@dshanske)
 * [9 years, 4 months ago](https://wordpress.org/support/topic/invalid-html-output/#post-8740795)
 * Sorry for the lack of response. I have to figure otu where I am supposed to get
   notifications of support issues here. I track issues on the Github repository.
   But I just fixed this one in the latest update.
 *  Thread Starter [greatestview](https://wordpress.org/support/users/greatestview/)
 * (@greatestview)
 * [9 years, 2 months ago](https://wordpress.org/support/topic/invalid-html-output/#post-9024705)
 * Thank you! Just tried it out, nice improvements.

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Invalid HTML output’ is closed to new replies.

 * ![](https://ps.w.org/syndication-links/assets/icon.svg?rev=3167284)
 * [Syndication Links](https://wordpress.org/plugins/syndication-links/)
 * [Frequently Asked Questions](https://wordpress.org/plugins/syndication-links/#faq)
 * [Support Threads](https://wordpress.org/support/plugin/syndication-links/)
 * [Active Topics](https://wordpress.org/support/plugin/syndication-links/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/syndication-links/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/syndication-links/reviews/)

 * 3 replies
 * 2 participants
 * Last reply from: [greatestview](https://wordpress.org/support/users/greatestview/)
 * Last activity: [9 years, 2 months ago](https://wordpress.org/support/topic/invalid-html-output/#post-9024705)
 * Status: resolved