Skip to content

Clarify paragraph value and add header comments to Touch_IRQ.ino and Touch_Polling.ino #7

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 2 commits into from
Jun 12, 2024

Conversation

aliphys
Copy link
Contributor

@aliphys aliphys commented Jun 10, 2024

In order to help Users better find and use the features provided by this library, this PR makes two improvements:

  • 📋 Reword the paragraph field by reducing word count from 48 words to 37. Knowing that the library is a C++ library, while true, is not a priority for users looking for this library in the IDE. Including two terms of interrupt/IRQ and on-request/polling should make it easier to find.
  • ✍️ Clarify objective of the Touch_IRQ.ino and Touch_Polling.ino example sketches by adding a header comment. This library provides an easy way to access the number of points and co-ordinates of the Goodix GT911 5-point capacitive touch controller without needing to configure the I2C registers by the Users. It is of value of Users to make this clear.

@leonardocavagnis leonardocavagnis merged commit 4101b8a into arduino-libraries:main Jun 12, 2024
1 of 3 checks passed
@leonardocavagnis
Copy link
Contributor

Thanks Ali!

@aliphys aliphys deleted the commentProperties branch June 12, 2024 07:52
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