Custom top navigation link component seems to be broken on mobile

@akanora mentioned in another topic that the component didn’t work for them on mobile:

I can reproduce that with the preview on theme creator:

On my laptop it looks fine:

But on my phone there is no navigation at the top:


I also wonder if it’s intentional that hiding the category breadcrumb seems to work only while I am logged in. They were visible when I was logged out:

2 Likes

There’s another issue that should be fixed as well.

When using New;topic created or replied to in the last few days;/new, the navigation item doesn’t display the number of new topics in parentheses, unlike the default implementation. For example, instead of showing New (5), it only shows New.

1 Like

Thanks for reporting it! I’ve got some refactoring on the way that will fix this issue (CSS hiding too much) and also includes a migration to an objects type setting, which is a bit nicer for configuration

I don’t think it’s a bug really, more like a lack of a feature… it’s not really meant to replace the default link functionality entirely, only supplement them

2 Likes

I get that it’s meant to be a supplement, not a replacement but without the count, it kind of loses its usefulness. The number is what tells me at a glance whether there’s anything worth checking, and how much. Without it, I have to click in every time just to find out, which defeats the point of having a quick nav item at all. Would be great if the count could be added, even as an optional setting.

I’ve just made a pull request to fix this issue. It’s disabled by default @awesomerobot