Viewing 9 replies - 1 through 9 (of 9 total)
  • It will ONLY leave the tables because of the columns (we have an issue in GitHub on how to approach this). Aside from that, nothing else ??

    GitHub issue: https://github.com/gambitph/Page-Builder-Sandwich/issues/46

    Thread Starter cruisin_long

    (@cruisin_long)

    Are you saying this is a proposed enhancement in the future?
    That is that it will not leave changed code??

    Yes that will definitely be included in the future.

    It will leave a few but not a lot. Shortcodes for example are not at all touched right now. Only the columns/rows create the existing clutter, in the future that would change and only leave a few elements/divs just to keep the layout (and so that it can be brought back to Sandwich’s layout when re-installed)

    Thread Starter cruisin_long

    (@cruisin_long)

    I have been playing with Beaver Builder and I think it does not leave any rubbish behind if deactivated
    Are you suggesting to leave any changes to original content is acceptable?

    You misunderstand me.

    What I’m saying is.. one of the main features is the creation of columns (1, 2, 3, etc columns).. let’s say you created a 2 column area and added text to it. When you deactivate Sandwich, our goal is that that 2 column area would still be 2 columns. Right now the result in the front end is a table, so we want to make that handling better. That’s it.

    Thread Starter cruisin_long

    (@cruisin_long)

    No you misunderstand me perhaps and I wish this had not evolved to this point

    Latest question from me was:
    Are you suggesting to leave any changes to original content is acceptable?

    As after setting up a three column set up with my text in each column after deactivating and removing Page Builder Sandwich and my original text this was left:

    <table class=”pbsandwich_column” style=”width: 100%; height: auto; border: none;” border=”0″>
    <tbody class=”row”>
    <tr>
    <td class=”col-sm-4″ style=”width: 33.3333333333333%;” data-wp-columnselect=”1″>
     
    </td>
    <td class=”col-sm-4″ style=”width: 33.3333333333333%;”></td>
    <td class=”col-sm-4″ style=”width: 33.3333333333333%;”></td>
    </tr>
    </tbody>
    </table>

    I know other page builders etc leave rubbish behind if you deactivate or uninstall them.
    Others may see this as being ok and acceptable – I do not.

    I wonder if it is possible for this plugin if deactivated or removed to leave just original text rather than the table structure?

    I am not trying to give you a hard time there is lots I like about it and if it left clean unaltered text or even say Bootstrap Columns behind if deactivated or uninstalled I would pay for it

    It’s okay. We’re actually agreeing on the same thing ??

    What you’re pointing out is what I want changed too. The tables should be gone, but the layout should remain the same. What I’m thinking is that the table, tbody, tr, td, etc should get replaced with only the necessary divs upon uninstallation so that at least their structure and the front end would still remain somewhat the same. If Sandwich was turned back on (installed & activated), those divs can be brought back into table form.

    From your point of view, would it be better to just remove them all, don’t replace them with anything and just leave the content as paragraphs? This would actually be easier to accomplish instead of the one I suggested above, but installing & re-activating Sandwich will not bring back the previous layout.

    This is in the todo: https://github.com/gambitph/Page-Builder-Sandwich/issues/46

    Thread Starter cruisin_long

    (@cruisin_long)

    Cool
    I think Beaver Builder just leaves the original text and perhaps if you can do the same …
    One of the criticisms Chris Lema had a while ago about VC was the stuff it left behind:
    https://chrislema.com/wordpress-page-builders/
    If you can get yours right the way it works from within the WordPress Text Editor is I think what a lot are looking for

    +1 ??

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Code Left Behind’ is closed to new replies.