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

Wiki refurbishment #312

Draft
wants to merge 114 commits into
base: main
Choose a base branch
from
Draft

Wiki refurbishment #312

wants to merge 114 commits into from

Conversation

ronzulu
Copy link

@ronzulu ronzulu commented Oct 28, 2024

Hello

Talon is great, and the community is awesome in helping users with issues, but as a newcomer, I found the documentation not all that beginner friendly.

From comments on slack, I see that this isn't just me and I've taken it upon myself to overhaul the wiki. I hope I haven't stepped on anyone's toes, and I know that it may not be adopted, but I thought that it is worth a short.

The aims have been to:

1. Incorporate all material from the community README

Incorporate from https://github.com/talonhub/community/blob/main/README.md,
so that the README would for the most part be a pointer to the wiki.

As a newcomer it is difficult to know what to glean from the wiki and what to learn from the README, especially as they cover overlapping topics. Hopefully not offending anyone, but given how comprehensive Talon is, I don't think that having documentation in a long single page README is all that suitable.

IMHO only information relevant to those building or maintaining the community file set should remain there, such as the sections on contributing and automated testing.

2. Make the documentation more beginner friendly

Sometimes this it's just simple wording changes, for example from "essential integrations" to "popular integrations". ("essential" can freak out a beginner as if taken literally means installing and learning about seven integrations)

Separate out Talon programming related info from beginner level topics. For example, the first paragraph on the page about talon lists https://talon.wiki/Customization/talon_lists says ".talon-list files do exactly the same things as a Python Context". Something confusing for anyone other than a programmer.

In this case these details were moved to a separate section called "notes for programmers"

3. Include all user relevant commands that are missing

Some user relevant commands have been missing from the wiki and also from the README as well.

For example, "clone line" or "go up" is missing. There is a reference in the README, but (unless I've missed something) a user would need to read through edit.talon to find these useful commands.

4. Restructuring

I've done a moderate amount of restructuring, I hope this is okay!

5. Feedback

I think this is 80 or 90% complete, but before I do any more I wanted to know if it is liked and if the community elders will want to adopt it.

If so, then I will incorporate feedback and complete tidying it up.

Cheers
Ronny

@ronzulu
Copy link
Author

ronzulu commented Nov 13, 2024

I've beefed up the wiki, fixed up all the links, and @nriley for the moment added some placeholders for your comments.
https://deploy-preview-312--talon-wiki-refactor.netlify.app/

- `ctrl.pyi` - 'Low level' mouse and keyboard event injection. You should prefer using the exposed mouse/keyboard actions rather than this.
- `fs.pyi` - Watch files and folders for changes.

You are also able to use almost all of the CPython standard library. So you can use that do do network requests, maths operations, or execute subprocesses for example. Other Python packages like numpy may be included in the Talon distribution as an implementation detail, but are not guaranteed to be included forever.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit (typo): "So you can use that do do network..." -> "So you can use that to do network..."

@@ -0,0 +1,21 @@
# API functions
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for this section, which fills in a gap in the Talon API documentation ✨

Copy link
Author

Choose a reason for hiding this comment

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

Actually, I can't take much credit. It was already there but I gave it its own page to make it easier to find 😃
https://talon.wiki/Customization/misc-tips#api-functions

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.

4 participants