Review Request 128522: [KCharSelect] Add bookmarks support

Christoph Feck cfeck at kde.org
Thu Sep 1 16:30:55 UTC 2016


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

(Updated Sept. 1, 2016, 12:30 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Utils and David Faure.


Changes
-------

Submitted with commit c25c21fa4415ff25db49b674ec3003bdcf4c565e by Christoph Feck to branch master.


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 6db5c66 
  kcharselectdia.h 27515a4 
  kcharselectdia.cc 28d65f9 
  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/20160901/fa48ec00/attachment.html>


More information about the Kde-utils-devel mailing list