further validation of cookie table issue

  • Resolved davidebabylonia

    (@davidebabylonia)


    9 years, 7 months ago

    your HTML output misses of the tags <tr></tr> on lines (if you remember I gave you a post on this 3 weeks ago):

    $ret .= '<table class="cookielawinfo-' . $style . '"><thead><tr>';
$ret .= '<th class="cookielawinfo-column-1">Cookie</th>';
$ret .= '<th class="cookielawinfo-column-2">Type</th>';
$ret .= '<th class="cookielawinfo-column-3">Duration</th>';
$ret .= '<th class="cookielawinfo-column-4">Description</th>';
$ret .= '</tr></thead><tbody>';

    Please fix it in next release
    Thanks

Viewing 4 replies - 1 through 4 (of 4 total)
  • Richard

    (@richardashby)

    Hi Davide, that is something I will get to though my previous answer still stands- this specific shortcode is being upgraded so your suggestion will become a moot point, unless you choose to use the original shortcode which you will need to do manually as the [cookie_audit] shortcode will use the upgraded code.

    Remember though that this is a free plugin, it cost you nothing, and is not my full time job. I will get to this when I have time, I don’t ever promise release dates.

    Thread Starter davidebabylonia

    (@davidebabylonia)

    I understand your point, but first, giving a plugin for free is your choice, second I was relying on your previous reply when you said this fix would be included in the release 1.5 and it’s not, that’s why I reminded you.

    Anyway, I’ll wait the new release with different code, I have modified the table for solve the validation problem until you will release the new 1.6.

    In my point of view, error must be corrected asap and new functionalities can take their time until done.

    Richard

    (@richardashby)

    Davide- it is on the backlog to look into but is unlikely to be in the next release and I’m not guaranteeing any specific version number. Updates are always in the changelog so you can see what’s new before updating the plugin.

    Richard

    (@richardashby)

    Hi- all done. Thanks for finding and posting the solution.

    The shortcode will be updated in a future version which will generate different HTML. I’ll provide a fallback to the previous code so you’ll still be able to use the “original” version should you wish. That change is on its way though I can’t confirm a release date / version number yet.

    Cheers, Richard

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘further validation of cookie table issue’ is closed to new replies.