WordPress.org

Ready to get started?Download WordPress

Forums

Tabify Edit Screen
[resolved] ACF fix (9 posts)

  1. elliotcondon
    Member
    Posted 1 year ago #

    Hi Marko

    Loving the plugin. I'm using it on a clients site and I realy like your interface for creating the tabs.

    To integrate with ACF, I had to edit your css/tabs.css: line 23 and change

    .tabifybox-hide { display: none; }

    to
    #poststuff .tabifybox-hide { display: none; }

    This hides the acf postboxes on first load.

    Just an idea. Instead of using jquery to hide / show, I would use classes to do the same thing, but your classes can then use the !important rule to override other plugins display. This will give you better compatibility.

    Cheers
    Elliot

    http://wordpress.org/extend/plugins/tabify-edit-screen/

  2. Marko Heijnen
    Member
    Plugin Author

    Posted 1 year ago #

    The reason that I used javascript there was that it only is needed when javascript is enabled. Obviously you can use the .js CSS class for that.

    For me it's really important that my plugin works as best as possible without javascript. So I will look into this improvements this month :)

  3. elliotcondon
    Member
    Posted 1 year ago #

    Hi

    I think you mY have misunderstood me. Definitely use javascript but think Bout using javascript to add / remove classes instead of using hide / show functions.

    That way, your css classes can use the !important attribute and override ACF.

  4. Marko Heijnen
    Member
    Plugin Author

    Posted 1 year ago #

    Ah okay that way. Did misunderstood that. But is this only a problem on load or also on switching tabs? Hopefully I got some time next week to play and fix it

  5. elliotcondon
    Member
    Posted 1 year ago #

    Only a problem on load.

    ACF has css to show / hide the metaboxes and unfortunately the css gets a higher priority than yours. So on load, all the acf metaboxes are shown.

    But with the fix above (which won't ruin anything), it works 100% with ACF.

    Also, it might be nice to place the tabs underneath the post title input field. As that field is always present, it might be good to take it out of the tab content area.

  6. Marko Heijnen
    Member
    Plugin Author

    Posted 1 year ago #

    Thanks for the info.

    The placing of tabs is not going to change. I had it under the title before but changed it because for me above it makes more sense how it is right now since it gets the extra width from the right column. Did however create a ticket that it should be changeable through a filter.

  7. elliotcondon
    Member
    Posted 1 year ago #

    Nice, a filter would be great.

    Good work all round mate.

  8. Marko Heijnen
    Member
    Plugin Author

    Posted 1 year ago #

    Been working today for this fixes and way more: https://github.com/markoheijnen/tabify-edit-screen.

    Will be releasing an update tomorrow. First need to check all edge cases really well before ending up with a javascript error again.

  9. Marko Heijnen
    Member
    Plugin Author

    Posted 1 year ago #

    Just released version 0.5 with all your demands ;)

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.