-
Notifications
You must be signed in to change notification settings - Fork 275
Conversation
TizenAPI-Bot
commented
May 22, 2026
Public API Changed
Please follow the ACR process for the changed API below.
Added: 109, Removed: 0, Changed: 0
Internal API Changed
Added: 0, Removed: 109, Changed: 0
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.
Code Review
This pull request promotes several gesture-related APIs to public status by replacing internal attributes with the <since_tizen> 10.1 </since_tizen> tag and renaming various 'TouchesRequired' methods to 'TouchCount' for better clarity. Feedback focuses on maintaining naming consistency; while several setters were renamed, their corresponding getters (e.g., GetMinimumTouchesRequired) were not, leading to an inconsistent public API. Additionally, a high-severity issue was noted in PanGestureDetector.cs where the ScreenPosition property appears to be a stub returning a default value rather than fetching data from the native layer.
0368ed1 to
407ccc7
Compare
TizenAPI-Bot
commented
May 26, 2026
Public API Changed
Please follow the ACR process for the changed API below.
Added: 109, Removed: 0, Changed: 0
Internal API Changed
Added: 0, Removed: 109, Changed: 0
407ccc7 to
40b11dd
Compare
TizenAPI-Bot
commented
May 26, 2026
Public API Changed
Please follow the ACR process for the changed API below.
Added: 109, Removed: 0, Changed: 0
Internal API Changed
Added: 0, Removed: 109, Changed: 0
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.
🤖 [AI Review]
🔴 Critical: Public-API setter/getter pair naming is asymmetric after this rename — setters became *TouchCount/*TapCount but the matching getters were kept as *TouchesRequired/*TapsRequired. The inconsistency is visible in-line here (GetMaximumTouchesRequired() next to SetMaximumTouchCount(2)). Since these APIs are being newly exposed in this PR (previously [EditorBrowsable(Never)]), renaming the getters in lock-step (GetMinimumTouchCount/GetMaximumTouchCount in PanGestureDetector/LongPressGestureDetector; GetMinimumTapCount/GetMaximumTapCount in TapGestureDetector) avoids a follow-up breaking rename after release.
Description of Change
[NUI] Add Gesture related APIs by TCSACR-651
API Changes
https://jira.sec.samsung.net/browse/TCSACR-651