Review Request 105304: Allow usage of detailedsorry and detailederror in kdialog
Albert Astals Cid
aacid at kde.org
Thu Feb 7 21:09:28 GMT 2013
> On July 12, 2012, 4:17 p.m., David Faure wrote:
> > Looks good. For master only, though, given feature+message freeze in 4.9.
>
> Kai Uwe Broulik wrote:
> Totally forgot to commit this :( Will do for master (4.10) once my ssh key for my new notebook got imported.
I see this has a ship it but marked as uncommited. Was it commited and you forgot to close the request or is still uncommited?
- Albert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105304/#review15735
-----------------------------------------------------------
On June 20, 2012, 12:06 a.m., Kai Uwe Broulik wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105304/
> -----------------------------------------------------------
>
> (Updated June 20, 2012, 12:06 a.m.)
>
>
> Review request for KDE Base Apps.
>
>
> Description
> -------
>
> As somebody who loves bash scripting but also wants to have a nice userfriendly graphical "interface" for those scripts, I excessively use kdialog.
> In a new project I am working on, I need the detailedsorry thing in KMessageBox, so I can provide a simple error message "could not launch xyz" while allowing detailed error information such as "directory xyz does not exist" in an expandable Details box.
> This patch adds this functionality to kdialog.
> Usage:
> kdialog --detailedsorry "error message" "what should be in the details section"
> also works with detailederror
>
> [I am volunteering for an update of the respective Wiki page on techbase http://techbase.kde.org/index.php?title=Development/Tutorials/Shell_Scripting_with_KDE_Dialogs_%28de%29 as well. Those screenshots date back to KDE 3 and early KDE 4 times ;) ]
>
>
> Diffs
> -----
>
> kdialog/kdialog.cpp ca2df2f
>
> Diff: http://git.reviewboard.kde.org/r/105304/diff/
>
>
> Testing
> -------
>
> Tested and works. Works with linebreaks and other parameters as well.
>
>
> Screenshots
> -----------
>
> a sample message
> http://git.reviewboard.kde.org/r/105304/s/606/
>
>
> Thanks,
>
> Kai Uwe Broulik
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130207/1d22f9e0/attachment.htm>
More information about the kde-core-devel
mailing list