Skip to content

Add gdbus callback for Aurum (accessibility status monitoring)#167

Merged
seungsoo47 merged 3 commits intoflutter-tizen:masterfrom
seungsoo47:dev.embedder.aurum.260330
Apr 7, 2026
Merged

Add gdbus callback for Aurum (accessibility status monitoring)#167
seungsoo47 merged 3 commits intoflutter-tizen:masterfrom
seungsoo47:dev.embedder.aurum.260330

Conversation

@seungsoo47
Copy link
Copy Markdown
Contributor

@seungsoo47 seungsoo47 commented Apr 2, 2026

  • Connect to org.a11y.Bus to check Accessibility state changes
  • Monitor IsEnabled and ScreenReaderEnabled property to properly enable/disable semantics

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a 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 updates the AccessibilitySettings class to monitor accessibility status via D-Bus in addition to existing TTS settings. It introduces new initialization methods, a unified UpdateSemanticsState function to manage semantics based on both TTS and accessibility states, and proper resource cleanup in the destructor. Feedback was provided regarding a potential null pointer dereference when accessing D-Bus properties and a suggestion to improve log consistency.

@seungsoo47 seungsoo47 force-pushed the dev.embedder.aurum.260330 branch from acc21c4 to 33a8b4d Compare April 7, 2026 01:39
@seungsoo47 seungsoo47 merged commit fcea92c into flutter-tizen:master Apr 7, 2026
10 checks passed
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