[Konsole-devel] Review Request 111937: Port to KF5/Qt5

Alexander Richardson arichardson.kde at googlemail.com
Tue Aug 20 21:37:19 UTC 2013


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

(Updated Aug. 20, 2013, 11:37 p.m.)


Review request for Konsole.


Changes
-------

Disable accessibility for now, this is stuff for another commit.
Looks like it changed quite a bit for Qt5 (was causing assertions).


Description
-------

Port to KF5/Qt5

TerminalDisplayAccessible is disabled for Qt5 currently since I don't
have any experience with accessible stuff and it is more complicated
than just changing a few includes

REVIEW: 111937


Diffs (updated)
-----

  CMakeLists.txt 5783804588bc4b1d1687e2dd1aaff76f3a65906e 
  src/BookmarkHandler.h 76c33269e5f20cbe20c97cfa621236cf05dc0f53 
  src/BookmarkHandler.cpp 9fc1600aa5a2901549f87295894acaf88b2f55ed 
  src/CMakeLists.txt 153e4bcf4d28b06e567f01f791bee6d11e7c49c8 
  src/ColorScheme.cpp 6945c39144d12d8faefbb5242edfa1899e278477 
  src/ColorSchemeEditor.cpp afe13c3a447197ff864639be3d6c136b146c8c66 
  src/ColorSchemeManager.cpp 802f5d7f9d3a1008a8785b27546a7ffa27fac24c 
  src/EditProfileDialog.cpp cf7be3d72ec83ae4a9b34cea80d3e444c0c36592 
  src/Emulation.cpp 02ed4be48a5916c75c7731864b33b4a825c31e6e 
  src/Filter.cpp 8f40c0c598bd1f61ec2ad9765c2399d0ba011744 
  src/KeyBindingEditor.cpp 005b9e32027dd14f9965ec6e2e7dd32e94b79272 
  src/KeyboardTranslator.h 99c71743d6c295c14381a83787baa061e7703878 
  src/KeyboardTranslator.cpp 8fe2acb40d75b62595b9823a990ba99f0e6e0509 
  src/KeyboardTranslatorManager.cpp 4d46210517e901b14e379dcfe435fd8fdedaeb09 
  src/MainWindow.h b297883eb9d62215ac71564b77c2145c1c8eeacc 
  src/MainWindow.cpp 285d6347be6680632b912b51961ced4d797a10bc 
  src/Part.h 2806bcdb066814ca32e1905772e99a0b2de426e5 
  src/Part.cpp 33e2d3849629e415817dc9a5f241e4ac3a323a28 
  src/PrintOptions.cpp 5e802987d5872f9041438a1ee69ec8162ddb962a 
  src/Profile.cpp 14946ce69000eef00b03864e4610c6775597a51d 
  src/SessionController.h 036e0534960fe7a1bd32be04136ba162c0469413 
  src/SessionController.cpp 88fc50c6a164fed94eefc72a71e4460f15ede57e 
  src/ShellCommand.cpp 2b059a16e6232146cc9d75b1b6af25919fdcc4e7 
  src/TerminalDisplay.cpp acfddc176bc68ae9b0f3a9ea52c958e37d14dff8 
  src/TerminalDisplayAccessible.h c6964412559e880dd27cf66a9fa8759196a09980 
  src/TerminalDisplayAccessible.cpp 8d37dc5e13448fc5d8619078c50334b0813597d4 
  src/ViewManager.cpp bebf1dd3198470d3c835d6a85d16c5d9cde857d6 
  src/Vt102Emulation.cpp 0b6d2ed2f75630df9fc7ac5d6d2b5d9a6702f254 
  src/main.cpp 2afc62c721b0e77264d0784caf7fbbd70b13951f 
  src/tests/CMakeLists.txt 8f65ebe8cf41d61e6b7fc5d555f3f1df0906fcb8 
  src/tests/DBusTest.cpp 729df8bf3dcb9bfed1aa62b40ca5d2ea08a40581 
  src/tests/PartTest.cpp 72cd27de07c0e96f8986db3a40dcf9239d9b3529 
  src/tests/ShellCommandTest.cpp d03964fda57a1b37d4a02c82c3c04fd00040aaff 

Diff: http://git.reviewboard.kde.org/r/111937/diff/


Testing
-------

It compiles fine (both Qt4 and Qt5).
After installation the terminal view of KF5 based okteta started working.


Thanks,

Alexander Richardson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20130820/beb33d82/attachment-0001.html>


More information about the konsole-devel mailing list