Viole07 opened a new pull request, #119:
URL: https://github.com/apache/cordova-plugin-vibration/pull/119

   ### Platforms affected
   JavaScript (test.js)
   
   ### Motivation and Context
   This refactoring improves the readability and maintainability of the 
vibration test code by consolidating repetitive functions and reducing 
redundancy.
   
   ### Description
   - Refactored code to streamline vibration test functions and minimize 
duplication.
   - Reorganized HTML generation and action button creation to improve clarity.
   - Simplified the cancellation logic and optimized timeout handling.
   
   ### Testing
   Manually verified vibration functionality across tests, ensuring both 
initiation and cancellation work as expected for all supported patterns.
   
   ### Checklist
   - [x] I've run the tests to ensure all new and existing tests pass.
   - [ ] I added automated test coverage where applicable.
   - [ ] Commit is prefixed with `(js)` as this change affects all JavaScript 
platforms supporting vibration.
   - [ ] If this Pull Request resolves an issue, I linked to the issue in the 
text above.
   - [ ] I've updated the documentation if necessary.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org

Reply via email to