-
-
Notifications
You must be signed in to change notification settings - Fork 654
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
Various rewordings, punctuation additions, and light fixes to the Developer Guide. #16967
Conversation
WalkthroughThe recent modifications to the NVDA developer guide primarily focus on improving clarity and precision. Enhancements include restructuring sentences for better readability, updating terminology for consistency, and refining technical examples. These changes aim to assist developers in understanding the NVDA Add-on API, new features, and gesture remapping, ultimately making the document more user-friendly and accessible. Changes
Sequence Diagram(s)sequenceDiagram
participant Developer
participant NVDA_Documentation
participant NVDA_Addon_API
participant Translator
Developer->>NVDA_Documentation: Review guide
NVDA_Documentation->>Developer: Provide updated info
Developer->>NVDA_Addon_API: Implement new features
NVDA_Addon_API-->>Developer: Return feedback
Developer->>Translator: Share guidelines for translations
Translator-->>Developer: Provide translated content
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
0a4bd89
to
ef24b6e
Compare
See test results for failed build of commit de8d9ec846 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reads well, good work going through all of that!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for going through and making these changes, @XLTechie .
Summary of the issue:
While reading the dev guide, I noticed several things that could be improved.
Mainly these are punctuation changes (missing commas, backticks, etc.), or wording changes to improve the English.
I believe I also added a couple internal links.
Description of user facing changes
None.
Description of development approach
See description of issue.
Testing strategy:
Known issues with pull request:
Code Review Checklist:
Summary by CodeRabbit