WordPress.org

Forums

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

  1. reddo
    Member
    Posted 1 year 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. MW Delaney
    Member
    Plugin Author

    Posted 1 year 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.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.