Anatole: Current item highlighting in navbar is error-prone

[msfjarvis]

In the example site, when you select the About tab it will not be underlined in the manner Home and Posts is. I checked the generated HTML and can confirm the Hugo trick used to set class="current" seems to not work on standalone pages, as can be reproduced on my staging site at https://staging.msfjarvis.dev where all nav items except Posts are standalone pages.

[lxndrblz]

Hello Harsh, I was able to reproduce the issue and added my changes to a separate branch.
Would you mind verifying if this branch also resolves your issues on your staging site?

[msfjarvis]

Hello Harsh, I was able to reproduce the issue and added my changes to a separate branch.
Would you mind verifying if this branch also resolves your issues on your staging site?

On it!

[msfjarvis]

I don't see any changes in https://github.com/lxndrblz/anatole/commits/buggy-menu, is the branch not fully pushed?

[lxndrblz]

My bad. Try again. :)

[msfjarvis]

My bad. Try again. :)

Still doesn't seem to work for the About menu item, other two are working. Maybe this is a configuration issue on my site? The source is here if you wanna investigate: https://github.com/msfjarvis/msfjarvis.dev/tree/staging

[lxndrblz]

I'll check it out.

[lxndrblz]

Please notice that your menu items in the config.toml should have a trailing slash. Such as:

[[menu.main]]
name = 'Uses'
url = '/uses/'
weight = 3

Check the ReadMe as a reference.

Would you mind trying the newest update in the branch on your staging server? Your site executed just fine locally.

[msfjarvis]

Please notice that your menu items in the config.toml should have a trailing slash. Such as:

[[menu.main]]
name = 'Uses'
url = '/uses/'
weight = 3

Check the ReadMe as a reference.

Would you mind trying the newest update in the branch on your staging server? Your site executed just fine locally.

That fixed it!

[msfjarvis]

WARN 2020/05/23 22:02:15 Page.URL is deprecated and will be removed in a future release. Use .Permalink or .RelPermalink. If what you want is the front matter URL value, use .Params.url

Does spew out this warning though

[lxndrblz]

Yeah, I am aware of this. Before commiting this to the master I'll change it to .Permalink. Just wanted to make sure we are on the right track.

[lxndrblz]

RefPermalink seemed to do the trick. 👍