[table-wrap] shortcode not working

  • Resolved reddo

    (@reddo)


    10 years, 9 months ago

    I’ve tried using the standard markup, i.e.: 

    [table-wrap bordered="true" striped="true"]
<tr><td></td></tr>
[/table-wrap]

    With 1 row there was no error, but no table either, instead inspecting the element (plain text with tags stripped away) i saw:

    <?xml version="1.0"?>
<tr class=" table table-bordered table-striped">
<td>Imputernicit medic sef</td>
<td>Dr. BOERIU Cristian – Marius</td>
<td>(UPU-SMURD Mures)</td>
</tr>

    With more than 1 rows the following error:

    Warning: DOMDocument::loadXML(): Extra content at the end of the document in Entity, line: 7 in [...]/wp-content/plugins/bootstrap-3-shortcodes/bootstrap-shortcodes.php on line 1049

Fatal error: Call to a member function setAttribute() on a non-object in [...]/wp-content/plugins/bootstrap-3-shortcodes/bootstrap-shortcodes.php on line 1050

    With the following markup

    [table-wrap bordered="true" striped="true"]
[tr][td][/td][/tr]
[/table-wrap]

    I got the following error:

    Warning: DOMDocument::loadXML(): Start tag expected, '<' not found in Entity, line: 2 in [...]/wp-content/plugins/bootstrap-3-shortcodes/bootstrap-shortcodes.php on line 1049

Fatal error: Call to a member function setAttribute() on a non-object in [...]wp-content/plugins/bootstrap-3-shortcodes/bootstrap-shortcodes.php on line 1050

    After I typed this all out, I noticed I should have actually added the whole table markup, including the <table> tag. Maybe it’s just me, but this should be more clear in the documentation, at least in the example markup.

    IMHO there practically is no use for this shortcode if you have to type the whole table markup anyway.

    https://www.ads-software.com/plugins/bootstrap-3-shortcodes/

Viewing 1 replies (of 1 total)
  • Plugin Author MWDelaney

    (@foolsrun)

    reddo,
    We’ve gone back and forth about keeping the table shortcodes in this plugin because tables aren’t really a feature Bootstrap, and ultimately settled on this best-and-worst-of-both-worlds approach. We’ll likely re-re-revisit the issue again in future releases.

    We appreciate the feedback. You’re correct that the <table> tag is necessary, and that [table-wrap] doesn’t have a whole lot of use. We’ll take another pass at the documentation for [table-wrap] in a future release.

Viewing 1 replies (of 1 total)
  • The topic ‘[table-wrap] shortcode not working’ is closed to new replies.