Review Request 126149: [Icon widget] Bring back properties dialog

[Eike Hein]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126149/#review88741
-----------------------------------------------------------


The problem with this is that the old applet had the properties widget embedded as a page into the applet config dialog. This probably replaces the latter, killing the keyboard shortcut page and making it not possible to configure a hotkey anymore ...?

- Eike Hein


On Nov. 23, 2015, 8:54 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126149/
> -----------------------------------------------------------
> 
> (Updated Nov. 23, 2015, 8:54 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: ?
>     https://bugs.kde.org/show_bug.cgi?id=?
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This brings back the KPropertiesDialog to modify an icon's appearance. This has been requested at multiple occasions. This has been adapted from the Plasma 4 icon code.
> 
> 
> Diffs
> -----
> 
>   applets/icon/package/contents/ui/main.qml 9286b94 
>   applets/icon/plugin/icon_p.h dd7963c 
>   applets/icon/plugin/icon_p.cpp e086870 
> 
> Diff: https://git.reviewboard.kde.org/r/126149/diff/
> 
> 
> Testing
> -------
> 
> Dropped a file from my home onto the desktop -> dialog from the actual file, allowing to rename it. The applet reflected the changes.
> 
> Dropped an application from kickoff to the desktop -> dialog from a copy of the desktop file, allowing to change its icon and description. The applet reflected the changes.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151123/e7f661cd/attachment.html>