Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

menu: let arrow navigation respect ordersectionsby configuration #170

Closed
LuisSucuc opened this issue Feb 7, 2022 · 0 comments
Closed

menu: let arrow navigation respect ordersectionsby configuration #170

LuisSucuc opened this issue Feb 7, 2022 · 0 comments
Assignees
@McShelby
Labels
bug Something isn't working
Milestone
2.9.5

Comments

@LuisSucuc
Copy link

If the setting is ordersectionsby="title" the variables relearnPrevPage and relearnNextPage keep the setting of ordersectionsby="weight" in the /layouts/parcials/footer.html file.

I tested with exampleSite and changing the setting to ordersectionsby="title" and using the prev and next buttons generates an undesired behavior.
@McShelby McShelby self-assigned this Feb 7, 2022
@McShelby McShelby added the bug Something isn't working label Feb 7, 2022
@McShelby McShelby added this to the 2.9.5 milestone Feb 7, 2022
@McShelby McShelby changed the title relearnPrevPage and relearnNextPage ignore ordersectionsby configuration menu: let arrow navigation respect ordersectionsby configuration Feb 7, 2022
McShelby added a commit that referenced this issue Feb 7, 2022
@McShelby
menu: let arrow navigation respect ordersectionsby configuration #170
6b943ae
@McShelby McShelby closed this as completed Feb 7, 2022
@McShelby McShelby mentioned this issue Feb 7, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@McShelby McShelby

Labels
bug Something isn't working
Projects
None yet
Milestone
2.9.5
Development

No branches or pull requests
2 participants
@McShelby @LuisSucuc