Support » Plugin: ACF Merge Group Tabs » New class names

  • Hi there,

    Awesome you created this feature, I’m very happy with it. But this plugin is broken with the new ACF version I think?

    The class name of the tab field is different. You use ‘.field_type-tab’ but it’s changed to ‘.acf-field-tab’.

    Also when you hide the default content_editor the first field is not within the second postbox-container (#postbox-container-2).

    When I change

    var $boxes = jQuery("#postbox-container-2 .postbox .field_type-tab").parent(".inside");

    to

    var $boxes = jQuery(".postbox .acf-field-tab").parent(".inside");

    It works fine with my WordPress installation 🙂

    Oh and btw I use ACF PRO, maybe that is the difference?

    https://wordpress.org/plugins/acf-merge-group-tabs/

Viewing 2 replies - 1 through 2 (of 2 total)
  • @patrick Leijser … thanks for this fix. I can confirm that the plugin worked with previous versions of ACF Pro, but fails with the most recent version.

    @yarcheek … any chance for an update? Thanks for all your great work!

    enshrined

    (@enshrined)

    This seems to work for me in the new ACF Pro. Although it could probably use some checking over

    add_action('admin_footer', function() {
    
    	$screen = get_current_screen();
    	if ( $screen->base == 'post' ) {
    		echo '
    		<!-- ACF Merge Tabs -->
    		<script>
    		var $firstBox;
    			jQuery(document).ready(function(){
    				var $boxes = jQuery("#postbox-container-2 .postbox .acf-tab-group").closest(".inside");
    
    				if ( $boxes.length > 1 ) {
    
    				    $firstBox = $boxes.first();
    
    				    $boxes.not($firstBox).each(function(){
    					    jQuery(this).children(".acf-field:not(.acf-tab-wrap)").addClass("hidden-by-tab").appendTo($firstBox);
    					    jQuery(this).children(".acf-tab-wrap").children("ul").children("li").removeClass("active").appendTo($firstBox.children(".acf-tab-wrap").children("ul"));
    					    jQuery(this).closest(".postbox").remove();
    				    });
    
    				}
    			});
    
    		</script>';
    	}
    
    });
Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘New class names’ is closed to new replies.