Review Request 128522: [KCharSelect] Add bookmarks support
Christoph Feck
cfeck at kde.org
Mon Jul 25 14:47:24 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128522/
-----------------------------------------------------------
Review request for KDE Utils and David Faure.
Bugs: 242333
http://bugs.kde.org/show_bug.cgi?id=242333
Repository: kcharselect
Description
-------
Added a standard Bookmarks menu. The bookmarks are named U+xxxxx for the currently selected character.
Two issues:
1. Is there a way to actually get an "Add Bookmark" dialog? The bookmarks are silently added.
2. I blindly copied the "filename" stuff from other code I have seen. To me it looks like the bookmarks are then not editable, if the system already has a bookmarks file?
Diffs
-----
CMakeLists.txt 5332c09
kcharselectdia.h 903f569
kcharselectdia.cc 18e5ad0
kcharselectui.rc d8556ee
Diff: https://git.reviewboard.kde.org/r/128522/diff/
Testing
-------
Tested with kwidgetsaddons from master.
To test with older frameworks: "currentCodePoint()" -> "currentChar().unicode()"
"setCurrentCodePoint(uc)" -> "setCurrentChar(QChar(uc))"
Thanks,
Christoph Feck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160725/1c530d55/attachment.html>
More information about the Kde-utils-devel
mailing list