[KDE Usability] Re: Review Request: Can't switch back to laptop display when external display gets disconnected

[Martin Bednar]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6513/#review9886
-----------------------------------------------------------



/branches/KDE/4.6/kdebase/workspace/kcontrol/randr/module/randrmonitor.cpp
<http://svn.reviewboard.kde.org/r/6513/#comment11048>

    rather than a UPower signal, use the Solid resumingFromSuspend() dbus signal.


- Martin


On Feb. 18, 2011, 1:48 a.m., alter_schwede wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6513/
> -----------------------------------------------------------
> 
> (Updated Feb. 18, 2011, 1:48 a.m.)
> 
> 
> Review request for kdelibs and usability.
> 
> 
> Summary
> -------
> 
> I already posted this patch here:
> https://bugs.kde.org/show_bug.cgi?id=265955
> 
> This is a patch to the kded_randrmonitor service that solves the below
> mentioned problem.
> If only one output is connected and the user tries to switch the display it
> makes sure the connected output gets activated and deactivates all disconnected
> output afterwards.
> 
> Reproducible: Always
> 
> Steps to Reproduce:
> 1. Connect an external display to your laptop and use your latop's hotkey (e.g.
> Fn+F6) to active it and deactivate your laptop's display.
> 2. Disconnect the external display
> 3. Try to switch back to internal display with your keyboard
> 
> Actual Results:  
> Nothing happens
> ----
> I improved my patch.
> Now it also automatically deactivates all disconnected outputs on resume from
> suspend and switches to a connected monitor if needed. This should fix e.g. Bug
> #257642.
> 
> 
> This addresses bugs 257642 and 265955.
>     https://bugs.kde.org/show_bug.cgi?id=257642
>     https://bugs.kde.org/show_bug.cgi?id=265955
> 
> 
> Diffs
> -----
> 
>   /branches/KDE/4.6/kdebase/workspace/kcontrol/randr/module/randrmonitor.h 1219928 
>   /branches/KDE/4.6/kdebase/workspace/kcontrol/randr/module/randrmonitor.cpp 1219928 
> 
> Diff: http://svn.reviewboard.kde.org/r/6513/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> alter_schwede
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110218/c92b88f8/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-usability mailing list
kde-usability at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability