NO BUG: Query Loop for Nested Menu

Bugs
1

Feedback for Nestable Menu

image
image304×510 10.4 KB

I noticed that the query loop is enabled at the UL tag when I enable the query loop for dynamic menu items.

When I have the query loop enabled, it generates multiple UL <ul> tags.

image
image1003×275 23.5 KB

I attempted to change the tag (Nav items) to <li> from <ul>, then change the tag to <ul> for the Nav (Nestable). But the Nav link already has the <li> tag inside it, which cannot change. So it won’t work as it will cause an error due to <li> inside a <li>.
image

It should be enabling the query loop at the Nav link instead of Nav items. We want the query loop for the li items. Not the <UL>

P/S: I’m not sure if I should submit this as a feature improvement or a bug.
I changed to bug.

2

Hi Jornes,
This is neither a bug, nor a feature request, since you can already do this.

If you apply the loop to the nav items ul, it is clear that exactly this element is repeated - as in any other loop.

Accordingly, as you know, you have to apply it to the nav-link. But this is not a nestable element and therefore has no loop setting.

What you can do is to wrap the nav link with a div or block and set its tag to li, and everything will work as it should.

image
image2518×1084 169 KB

Best,
timmse

3

Hi @timmse
Thank you for your reply!

Okay. I’ve followed your suggestion, and it really worked as it should. I didn’t think of this type of setup previously. Anyway, I think this setup could be a little confusing sometimes if I don’t know. Then the Use Query Loop should not be made available for Nav Items to reduce confusion, or maybe I still don’t know the other use case?

4

Hey Jornes,
In Bricks 1.9.2 we removed the query setting in places where it really doesn’t make sense. Get in touch if you encounter any problems.

1 Like
5

Hey @timmse
Thank you for your update!

Yes. This will reduce confusion. :wink: :+1: