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

Provide a static way of navigating the site #5

Open
etopian opened this issue May 23, 2015 · 9 comments
Open

Provide a static way of navigating the site #5

etopian opened this issue May 23, 2015 · 9 comments

Comments

@etopian
Copy link

etopian commented May 23, 2015

It seems the menu system is totally dynamic which means that any bot wishing to index the site needs Javascript... Googlebot by default does not have Javascript enabled, Google has bots that can index Javascript but not all of its bots do so. So there needs to be a static way to allow the site to be indexed. Either we need a sitemap or we need to do something about the menu system and generate some static html for it... Thoughts?

@ChrisZieba
Copy link
Owner

Generating a sitemap would be a pretty simple addition. The menu could also be added to each static page when it's saved, then replaced with JavaScript when the page is loaded. Adding a sitemap may be enough, not too sure.

@etopian
Copy link
Author

etopian commented May 23, 2015

Is there any actual need for the dynamic menu? What are the benefits of having a dynamic menu on the published site?

@ChrisZieba
Copy link
Owner

Let's say you delete an entry. If you go to a different entry in your system on the front end, the menu would still show the one you deleted. If the menu were static, every page would need to be regenerated each time the uri structure of any entry were edited or deleted.

@etopian
Copy link
Author

etopian commented May 23, 2015

Ok. So you don't have to regenerate the entire site structure because of this. Interesting.

@etopian
Copy link
Author

etopian commented May 23, 2015

... and what about the breadcrumbs, is there a good reason to keep them dynamic as well?

@ChrisZieba
Copy link
Owner

It's the same reasoning. For example, let's say this is your URL structure.

root
│   index
└───folder1 (Docs)
    │   install
    │   help

Docs is the label for folder1. The breadcrumbs for the pages /folder1/install and /folder1/help would be Home >> Docs >> Install and Home >> Docs >> Help.

If I change the label for folder1 to Documentation, the breadcrumbs would be incorrectly shown, unless I recompiled each static page in that folder with the new information.

This is the reason it's dynamic - to prevent each page from having to be regenerated each time entries/folders are edited, added and removed.

@etopian
Copy link
Author

etopian commented May 23, 2015

Ok. So let's give users a choice of incorporating a static version of the breadcrumbs and menu that is then removed and replaced with a dynamic version. Which obviously will be outdated, but will allow the site to get indexed by bots that don't parse dynamic content. And I guess also add a sitemap.

@ChrisZieba
Copy link
Owner

A static version would be a good. I've been thinking of adding a button in the backend that goes through each entry and updates its html and pushes it up to s3.

@etopian
Copy link
Author

etopian commented May 23, 2015

That would probably be ideal. To have a way to regenerate all the pages that is not mandatory. The only concern would be the amount of time it takes to perform the operation. Any concerns with timeouts, etc.

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

2 participants