WordPress.org

Forums

Bootstrap Shortcodes for WordPress
[resolved] [table-wrap] shortcode not working (2 posts)

  1. reddo
    Member
    Posted 11 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://wordpress.org/plugins/bootstrap-3-shortcodes/

  2. Michael W.
    Member
    Plugin Author

    Posted 11 months ago #

    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.

Reply

You must log in to post.

About this Plugin

About this Topic

Tags

No tags yet.