legolijntje

Navbar links are incorrect when on API page

Recommended Posts

Small bug; when you're on the API information page, all the top-level navbar links except for Build and Forum, link to the API page.

Share this post


Link to post
Share on other sites
On 2/3/2017 at 2:52 AM, legolijntje said:

Small bug; when you're on the API information page, all the top-level navbar links except for Build and Forum, link to the API page.

It looks fine to me?

Share this post


Link to post
Share on other sites
10 hours ago, Nathan said:

It looks fine to me?

It doesn't for me... :huh:

When I'm on http://rebrickable.com/api/, all top-level navbar links (except for Build and Forum) link to that same adress. When I'm on http://rebrickable.com/api/v3/docs/?key=xxxxxxxxxx (with my key obviously), the top-level links link to that adres.

Not a lot more I can find I'm afraid.

Wait a minute....... After I typed the above, I think I found the 'problem'. It's not tied to the API page at all. It doesn't matter which page you're on, all top-level navbar links (except for build and forum) link to the current page you're on, but I never noticed. I don't think it's really a problem, but it's a bit weird nonetheless.

Share this post


Link to post
Share on other sites
On 2/4/2017 at 6:38 PM, legolijntje said:

It doesn't for me... :huh:

When I'm on http://rebrickable.com/api/, all top-level navbar links (except for Build and Forum) link to that same adress. When I'm on http://rebrickable.com/api/v3/docs/?key=xxxxxxxxxx (with my key obviously), the top-level links link to that adres.

Not a lot more I can find I'm afraid.

Wait a minute....... After I typed the above, I think I found the 'problem'. It's not tied to the API page at all. It doesn't matter which page you're on, all top-level navbar links (except for build and forum) link to the current page you're on, but I never noticed. I don't think it's really a problem, but it's a bit weird nonetheless.

If I put links on the top level menus, you can't open the menus on touch screen devices.

Share this post


Link to post
Share on other sites