Breadcrumbs on Detail on Mobile

This topic contains 5 replies, has 2 voices, and was last updated by  Paolo 8 years, 6 months ago.

We have moved to a support ticketing system and our forums are now closed.

Open Support Ticket

Tagged: , ,

  • Author
    Posts
  • November 17, 2015 at 2:19 pm #60554

    Jen Thomson
    Buyer
    Post count: 145

    Hi there,

    I’ve noticed that on the listing detail page the breadcrumbs do not have the correct < li > structure as on the other pages.

    This is an issue on mobile as everything after ‘Home’ drops down to the next level and this is hard to fix in css.

    I’ve also checked your demo and the same issue appears.

    November 17, 2015 at 2:21 pm #60556

    Jen Thomson
    Buyer
    Post count: 145
    This reply has been marked as private.
    November 17, 2015 at 6:51 pm #60609

    Paolo
    Site Admin
    Post count: 31206

    HI,

    you should wrap any html with code tags, otherwise the page will parse the html (and break our templates).

    I’ve flagged this for developers. I couldn’t find any easy fix with css too.

    Thanks for spotting this.

    November 19, 2015 at 3:17 pm #60870

    Paolo
    Site Admin
    Post count: 31206

    HI,

    this has been fixed and it will be in the version.

    You should be able to patch the bug with this in your theme functions.php:

    

    function geodir_strip_breadcrumb_li_wrappers($breadcrumb, $separator) {
$breadcrumb = str_replace(array(‘</li><li>’, ‘</li> <li>’), ”, $breadcrumb);

return $breadcrumb;
}
add_filter(‘geodir_breadcrumb’, ‘geodir_strip_breadcrumb_li_wrappers’, 999, 2);
    

    However, you’d have to remove the code upon next update or having the function twice will cause a PHP fatal error.
    

    Or you could test the latest version that includes the fix by downloading it from https://github.com/wp-plugins/geodirectory and manually udpating plugin files via FTP. 
    

    Thanks
    

		
	
    


    

			
				

    

	
    

		November 19, 2015 at 4:30 pm

		
		#60884

		
		
		
	
    


    


    

	
    

		
		
    Jen Thomson
    Buyer
    
		
		
     Post count: 145
    
	
    

	
    

		
		
    Thanks Paolo. I’ll just apply the fix for now. Cheers 🙂
    

		
	
    


    

			
				

    

	
    

		November 19, 2015 at 6:25 pm

		
		#60895

		
		
		
	
    


    


    

	
    

		
		
    Paolo
    Site Admin
    
		
		
     Post count: 31206
    
	
    

	
    

		
		
    You are welcome 🙂
    

		
	
    


    

			
		
	
    • 

	
  • 

		
    Author
    

		
    

			
				Posts
			
		
    

	
    • 





			



	


		Viewing 6 posts - 1 through 6 (of 6 total)
	


	


		
	





		
				

			

				

					
We have moved to a support ticketing system and our forums are now closed.

					Open Support Ticket