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.

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

Bedrock: the page-index page has both page-index and index active #408

Open
Wolfr opened this issue Aug 24, 2021 · 1 comment
Open

Bedrock: the page-index page has both page-index and index active #408

Wolfr opened this issue Aug 24, 2021 · 1 comment

Comments

@Wolfr
Copy link
Contributor

Wolfr commented Aug 24, 2021

As reported in smartcoop/design#36

In the main menu (the one accessible with the ctrl-m shortcut), the active page is highlighted. If the page is page-index, it highlights both page-index and index. See screenshot:
screenshot

Can we make the selection on the page state a bit more strict?

Wolfr added a commit that referenced this issue Nov 16, 2021
@Wolfr
Copy link
Contributor Author

Wolfr commented Nov 16, 2021

I fixed it, and interestingly I also fixed a 5-year old bug by checking this code: #178

However I still don't think it is advisable to use index.html for page template names. For the server,/folder/ is the same as /folder/index.html. Needs more testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant