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

Document cache invalidation opportunity #1377

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yhoiseth
Copy link
Contributor

It took me a while to discover cachebust, so it might be useful to others to have it in the Sass docs.

Sanity check:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@@ -40,3 +40,11 @@ Files with the `scss` extension use "Sassy CSS" syntax,
while files with the `sass` extension use the "indented" syntax: <https://sass-lang.com/documentation/syntax>.
Zola will return an error if `scss` and `sass` files with the same
base name exist in the same folder to avoid confusion -- see the example above.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part of the docs is about the content itself, not the templates. None of the other pages in that section mention template specific things like functions.

I agree it would be useful to show this kind of section somewhere but I don't know where. @southerntofu any ideas?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should probably go here with the get_url docs no? (by the way having built-ins in theme overview has always confused me i'd rather have a "FIlters/Functions" pages)

Also maybe explain what cachebust does (checksumming) by providing an example output HTML?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by the way having built-ins in theme overview has always confused me i'd rather have a "FIlters/Functions" pages

Yeah... this was made when there was one function, it should be split.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We now have 3 filters and 10 functions.

Maybe it would make sense to have two separate pages — one for filters and one for functions? I see a few advantages to having separate pages:

  1. Users may more quickly understand the difference between filters and functions.
  2. Shorter pages means less scrolling to get to the thing you are looking for.
  3. If we use one page now and add more content, we might want to split the pages later. In that case, we may break links to the documentation. Using two pages right away is more future-proof.

On the other hand, it would mean a bigger menu. I see that as less of a concern than the ones outlined above.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would put them all in the same page though, at least for now

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

Successfully merging this pull request may close these issues.

None yet

3 participants