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

Add :ref: links. Also reorder code to be closer to alphabetical order. #2515

Merged
merged 2 commits into from
Nov 12, 2024

Conversation

cbhaley
Copy link
Contributor

@cbhaley cbhaley commented Nov 12, 2024

This commit assumes your answer to my last email (about :ref:) is "yes". If it isn't then reject the commit.

This commit assumes your answer to my last email (about :ref:) is "yes". If it isn't then reject the commit.
Note: I will look at generating HTML links once I see what the anchors look like in the new generated template_ref document. Today they look like
https://manual.calibre-ebook.com/generated/en/template_ref.html#add-x-y
I'm hoping that in the new system they will look something like
https://manual.calibre-ebook.com/generated/en/template_ref.html#ff_add
adding the ff_ and removing the parameters.
@cbhaley
Copy link
Contributor Author

cbhaley commented Nov 12, 2024

Added changes to formatter_functions.py to start using the :ref: tag.

@kovidgoyal kovidgoyal merged commit 1ee299e into kovidgoyal:master Nov 12, 2024
0 of 4 checks passed
@kovidgoyal
Copy link
Owner

merged, but you might want to have your :ref: handler discard trailing
() automatically as it is a common mistake to write :ref:format_date()
instead of :ref:format_date. See 60d0f43

@kovidgoyal
Copy link
Owner

I implemented it in: ee3a949

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.

2 participants