10000 Started updating extension documentation by gornostal · Pull Request #1467 · Ulauncher/Ulauncher · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Started updating extension documentation #1467

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

Merged
merged 1 commit into from
May 19, 2025
Merged

Conversation

gornostal
Copy link
Member

Started updating extension documentation, learning about new changes in v6 along the way.

@friday I like that you maintained compatibility in the extension API! I think it's the right way for v6.
This new API looks much simpler and more intuitive, too.

What I want to do next is to make it more type safe and ready for documentation autogeneration. Also by extracting logic from DTO entities.

Copy link
Member
@friday friday left a comment

Choose a reason for hiding this comment

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

These are all good changes. I think better types might be better as well, but I went the other way and use native python as much as possible, so extension devs don't have to import for example Query.

As mentioned earlier I think we should switch to rspress and markdown though, and if we want to keep flowcharts I think https://github.com/mermaid-js/mermaid would be even better for the ease of editing (the new one you uploaded looks nicers, but GitHub markdown supports mermaid natively, and rspress and all similar have plugins for it.

And about the code doc generation, I think we would be better off writing the docs ourselves. The current readthedocs site leaked the internal Ulauncher classes to the extension developer docs. All it really needs is event and actions? We have simplified both a lot by now.

themes/themes

:doc:`themes/themes`
Create your own color themes
Copy link
Member
@friday friday May 18, 2025

Choose a reason for hiding this comment

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

Related to this, I really think we should recommend people to use https://github.com/Ulauncher/Ulauncher/discussions/categories/themes instead of your gist (I just created this discussion).

Although I think it's better that we encourage people to post the new simplified css file themes there instead.

I'm not sure what validation we want for it. We can test it here first: https://github.com/Ulauncher/Test-Discussions

Copy link
Member Author

Choose a reason for hiding this comment

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

I like the discussions approach for themes. I want to create an automation for migrating them from the Gist. Post v6 release though.

@gornostal
Copy link
Member Author

I remember about rspress. The docs need updating for v6 release, so I thought I’d update them for sphinx and then after the release migrate to rpress.

Thanks for checking the PR!

@gornostal gornostal merged commit a4d9385 into v6 May 19, 2025
4 checks passed
@gornostal gornostal deleted the ext-docs-improvements branch May 19, 2025 10:25
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
0