[Konsole-devel] Review Request: making the keymapping description in the keyboard setting page not editable

[Kurt Hindenburg]

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

Ship it!


Ship It!

- Kurt Hindenburg


On Nov. 10, 2012, 11:49 p.m., Yichao Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107289/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2012, 11:49 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> The description in the list of keymapping in the keyboard tab of the setting window is editable but is a no-op. Since it is possible to edit the name in the Edit pop-up window, it should be ok to just disable editing here.
> 
> 
> Diffs
> -----
> 
>   src/EditProfileDialog.cpp 8e44df6 
> 
> Diff: http://git.reviewboard.kde.org/r/107289/diff/
> 
> 
> Testing
> -------
> 
> Compiled and installed.
> The item description in the list is not editable but the popup edit window is still working and the list is updated properly when the description is changed in the Edit popup.
> 
> 
> Thanks,
> 
> Yichao Yu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20121111/d3bddf9f/attachment.html>