-
Notifications
You must be signed in to change notification settings - Fork 47
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
Feature Request: Edit this page #47
Comments
Hi @saurabh-mish, if you enable Git info with |
I ran
but don't see the time on any page. I have
in my config file. Am I missing a configuration and is this correct given that the view file URL for
and edit file URL for
? |
I meant page date, not time. It's displayed between the page authors and reading time rows, below the page title and description. It's shown by default, but you may have hidden it in your style settings. The %s in commit_link will be replaced with the commit hash, and the date will be wrapped in an anchor that references that link. It won't be a direct link to edit the page in GitHub, though. So users would need to take an extra step to edit the file. For Codeberg, it looks like the commit_url should be https://codeberg.org/mishra/patterns.se/commit/%s. |
It looks like there's a way to get the file path for a page, so it's possible to add an edit link. I don't know off the top of my head where it should go, or how it should be represented, though. I almost didn't include the commit link because I wasn't sure how to represent it, and I'm still not happy about using the date for it. I don't want to clutter the metadata or footer unnecessarily. I'll have to give it some thought. How were you thinking the link would be presented? |
I removed
in the config.
I've updated the config, but it doesn't seem to work.
There could be "Edit this page", which works like this - its towards the bottom on the left side. For the representation: the author(s) could be on the left side, reading time in the center, and "Edit this page" on the right. Paige is using a good amount of width which could be used to represent this information horizontally. |
You can see the date on https://willfaught.com/paige/customization/show/: "February 1, 2023"
What are your style overrides? Have you set paige.style in page front matter or site params? Have you overridden style.html? If you look at the HTML with the inspector, do you see the element for the date? |
Added in the latest version, v0.70.0. See the new @saurabh-mish Can you confirm it works for you? |
I don't have any style overrides configured. I've also not set
Quick question about the
but when I open the edit link for a
why is there an extra context I have added the below in
|
Can you add "Edit this page" to an existing page in exampleSite? |
For reference, it's here: https://github.com/willfaught/paige/blob/master/README.md?plain=1#L184 You don't have any dates set in your published pages' front matter, which is why no dates are showing.
I added a bit more documentation: 254710f. The For https://codeberg.org/mishra/patterns.se/src/branch/main/content/_index.md, you probably want https://codeberg.org/mishra/patterns.se/_edit/main/content/%s. I can't verify because I don't have a Codeberg account. |
Gotcha! I have set date in the archetype and also have
|
If you have an "Edit this page" in the exampleSite, please point me towards it so that I can ask about it in Codeberg groups. |
I added file link examples to https://willfaught.com/paige/customization/ and its sub-pages. |
Note that paige.link was renamed to paige.file_link. |
And paige.link.hide_page was renamed to paige.file_link.disable. |
Thanks a lot @willfaught!
This was very helpful. |
Paige is a great theme for personal and technical blogs. A reader may want to update a part of the webpage where the information is outdated, incorrect, etc.
Providing a link to the markdown file in the repository is helpful as the reader can create an issue, pull request, etc.
The text was updated successfully, but these errors were encountered: