[Kde-accessibility] Review Request: Focus Tracking in KWin's Zoom Plugin

[Frederik Gladhorn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106041/#review17796
-----------------------------------------------------------



kwin/effects/zoom/zoom.cpp
<http://git.reviewboard.kde.org/r/106041/#comment14037>

    This is in the dtor of registry, I don't think it makes much sense setting the member to 0 here.



kwin/effects/zoom/zoom.cpp
<http://git.reviewboard.kde.org/r/106041/#comment14038>

    You could also simply use disconnect(registry). Deleting it also disconnects it automatically btw.


- Frederik Gladhorn


On Aug. 19, 2012, 8:53 a.m., Amandeep Singh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106041/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2012, 8:53 a.m.)
> 
> 
> Review request for KDE Accessibility, kwin, Frederik Gladhorn, Sebastian Sauer, and Luboš Luňák.
> 
> 
> Description
> -------
> 
> This patch is for KWin, makes the focus-tracking feature of KWin work with applications. This makes KDE more accessible.
> A new accessibility client library "libkdeaccessibilityclient" (which can be found here: https://projects.kde.org/projects/playground/accessibility/libkdeaccessibilityclient) is also to be added in kde-support.
> 
> The focus-tracking feature will use the new library to get the accessibility information from at-spi.
> 
> Adjustments in CMakeLists have been made to add the library as an optional dependency, which if found enables the focus-tracking option in Zoom Plugin.
> And in absence of library, the option is removed.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 451a8b9 
>   config-workspace.h.cmake 4a2dcbf 
>   kwin/effects/CMakeLists.txt 29accb1 
>   kwin/effects/configs_builtins.cpp b291e38 
>   kwin/effects/zoom/CMakeLists.txt 36e11ac 
>   kwin/effects/zoom/zoom.h d809b21 
>   kwin/effects/zoom/zoom.cpp b275b1e 
>   kwin/effects/zoom/zoom_config.cpp 3edd10c 
> 
> Diff: http://git.reviewboard.kde.org/r/106041/diff/
> 
> 
> Testing
> -------
> 
> I tested KWin's new feature working with KWrite and Konsole.
> 
> 
> Thanks,
> 
> Amandeep Singh
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-accessibility/attachments/20120821/65b32d82/attachment.html>