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 link to projsync #3531

Closed
wants to merge 1 commit into from
Closed

Conversation

oleg-alexandrov
Copy link
Contributor

The page https://proj.org/resource_files.html mentions projsync but doesn't link to it. This pull requests addresses that.

@kbevers
Copy link
Member

kbevers commented Jan 2, 2023

The proposed fix changes the formatting from the black bold text of the program-directive to blue, regular font of clickable links. That's probably not a big deal in and of itself but it's inconsistent to just about all other references to the PROJ CLI apps used in the docs. I'm not sure this is the best fix to the problem, as this just scratces a particular itch of yours instead of introducing a general improvement across all of the documentation that keeps the look and feel of the docs consistent.

I agree that it would be much better to have the program-directive link to the actual doc-page for that application but that is not possible and is a well-known limitation of Sphinx. A better solution might land in Sphinx at some point cf sphinx-doc/sphinx#880.

@hobu
Copy link
Contributor

hobu commented Jan 2, 2023

That's probably not a big deal in and of itself but it's inconsistent to just about all other references to the PROJ CLI apps used in the docs.

Why shouldn't all of the links to the PROJ CLI apps be clickable links?

@oleg-alexandrov
Copy link
Contributor Author

this just scratces a particular itch of yours

I don't appreciate being second-guessed in my intentions.

The proposed change offers a link to a relevant tool. I would think that is more important than look-and-feel. But that is for you folks to figure out.

@snowman2
Copy link
Contributor

snowman2 commented Jan 2, 2023

I really like links when navigating documentation and always appreciate it when they exist as it makes for a better experience, for me at least.

Thoughts about updating this PR with a general search and replace to change projsync references to links for consistency sake?

@rouault
Copy link
Member

rouault commented Jan 2, 2023

cf #3534 for an attempt at a generic solution for linkable program directive

@kbevers
Copy link
Member

kbevers commented Jan 2, 2023

Why shouldn't all of the links to the PROJ CLI apps be clickable links?

That'd be lovely, which is why I was asking for a general solution in favour of the local fix. Seems @rouault how found a way to make that way.

@rouault
Copy link
Member

rouault commented Jan 2, 2023

superseded per #3534

@rouault rouault closed this Jan 2, 2023
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.

5 participants