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