D18114: FakeInput: add support for pointer move with absolute coordinates

2019-01-16 Thread Jan Grulich
jgrulich added a comment. Hi, can I get please this re-approved? It's now just again about the additional mouse support. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D18114 To: jgrulich, davidedmundson, graesslin, zzag Cc: graesslin, zzag, kde-frameworks-devel,

D18010: Allow zero altitude/longitude/latitude exiv gps data

2019-01-16 Thread Alexander Stippich
astippich added inline comments. INLINE COMMENTS > bruns wrote in exiv2extractor.cpp:298 > when you rewrite this as > > if (it != data.end()) { > auto ratio = it->value().toRational(); > it = data.findKey(Exiv2::ExifKey("Exif.GPSInfo.GPSAltitudeRef")); > if ((ratio.second !=

D18308: Ignore all non-storage deviceAdded signals from Solid

2019-01-16 Thread Stefan Brüns
bruns created this revision. bruns added reviewers: Baloo, Frameworks, ngraham. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY createCacheEntry is also called from

D18161: [WIP/RFC] [kioslave/file] Add a codec for legacy filenames

2019-01-16 Thread Christoph Feck
cfeck added a comment. Thanks for testing, btw. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18161 To: cfeck, #frameworks, #dolphin, dfaure Cc: nerdopolist, ngraham, kde-frameworks-devel, michaelh, bruns

D18161: [WIP/RFC] [kioslave/file] Add a codec for legacy filenames

2019-01-16 Thread Christoph Feck
cfeck added a comment. It's possible that Kate doesn't use KIO for local files, but falls back to QFile. It _would_ be possible somehow (e.g. via the QPA plugins) to inject the hack into all Qt applications, but I suggest to improve Dolphin in a way that it shows an error message or even a

D17816: Support for xattrs on kio copy/move

2019-01-16 Thread Christoph Feck
cfeck added a comment. I am still not fond of having a local event loop in KIO. The whole point of KIO is that it should work asynchronously. If the tests fail, could they be adapted? Or is the reason why the tests fail unfixable? REPOSITORY R241 KIO REVISION DETAIL

D15999: Add icons with added background to system.svgz

2019-01-16 Thread Nathaniel Graham
ngraham added a comment. In D15999#394818 , @filipf wrote: > +1 in regards to at least getting the new switch user icon into master, it's great @pstefan, any chance we can cudgel you into doing this?  REPOSITORY R242 Plasma Framework

D13316: manually set format in jpeg thumbnail creator

2019-01-16 Thread Nathaniel Graham
ngraham added a comment. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. Is there any reason not to commit this now? REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D13316 To: miklosm, dfaure, elvisangelaccio,

D18161: [WIP/RFC] [kioslave/file] Add a codec for legacy filenames

2019-01-16 Thread Nerdopolis Turfwalker
nerdopolist added a comment. Awesome! For what I can input, base file operations work. I can copy, delete, rename files, and testing, and diffing the trees with my kiocopy test script, it copies them perfectly. However, for example, some of the files I cannot open with kate REPOSITORY

D18161: [WIP/RFC] [kioslave/file] Add a codec for legacy filenames

2019-01-16 Thread Christoph Feck
cfeck updated this revision to Diff 49684. cfeck added a comment. Fix worst case estimate. We need up to two UTF-16 code units for each byte. REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18161?vs=49176=49684 REVISION DETAIL

D11320: Fix QVariantMapList operator >> implementation

2019-01-16 Thread Albert Astals Cid
This revision was automatically updated to reflect the committed changes. Closed by commit R281:66362ec67b8f: Fix QVariantMapList operator implementation (authored by aleksanderm, committed by aacid). Herald added a subscriber: kde-frameworks-devel. REPOSITORY R281 ModemManagerQt CHANGES

D13416: image thumbnailer: instruct Qt to look at the file contents when detecting its format

2019-01-16 Thread Albert Astals Cid
aacid closed this revision. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D13416 To: miklosm, #frameworks, #dolphin, dfaure, elvisangelaccio Cc: kde-frameworks-devel, kfm-devel, aacid, alexde, feverfew, michaelh, spoorun, navarromorales, firef, ngraham,

D13416: image thumbnailer: instruct Qt to look at the file contents when detecting its format

2019-01-16 Thread Albert Astals Cid
aacid added a comment. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. I pushed this at https://commits.kde.org/kio-extras/0be80392f01e0ec01174990c430dcd8f1ed35d9e REPOSITORY R320 KIO Extras REVISION DETAIL

D15999: Add icons with added background to system.svgz

2019-01-16 Thread Filip Fila
filipf added a comment. They only work for me when using a dark theme: F6552425: image.png F6552428: image.png We could just do QML circles. Pro is they assure legibility with many non-Breeze icons,

D15999: Add icons with added background to system.svgz

2019-01-16 Thread Nathaniel Graham
ngraham requested changes to this revision. ngraham added a subscriber: rooty. ngraham added a comment. This revision now requires changes to proceed. I still haven't managed to get these icons to actually work, per recent discussions. The background is always light instead of dark. Also the

D17664: fix for macOS

2019-01-16 Thread Albert Astals Cid
This revision was automatically updated to reflect the committed changes. Closed by commit R6:74f3eeb69ead: fix for macOS (authored by yurikoles, committed by aacid). Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. CHANGED PRIOR TO COMMIT

KDE CI: Frameworks » kio » kf5-qt5 FreeBSDQt5.12 - Build # 17 - Still Unstable!

2019-01-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.12/17/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Wed, 16 Jan 2019 23:07:00 + Build duration: 5 min 31 sec and counting JUnit Tests Name:

D17726: Add helper function to disable redirections (useful for kde-open)

2019-01-16 Thread Albert Astals Cid
This revision was automatically updated to reflect the committed changes. Closed by commit R241:8c3f000b72ed: Add helper function to disable redirections (useful for kde-open) (authored by achauvel, committed by aacid). CHANGED PRIOR TO COMMIT

D16643: Correct the accept flag of the event object on DragMove

2019-01-16 Thread Albert Astals Cid
aacid added a comment. @bruns waiting for your comment on whether your issues were addressed or not REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns Cc: aacid, bruns, dkorth, ngraham, kde-frameworks-devel,

D18237: Fix ResultIterator

2019-01-16 Thread Nathaniel Graham
ngraham added reviewers: Baloo, bruns, poboiko. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D18237 To: aacid, #baloo, bruns, poboiko Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams

D18237: Fix ResultIterator

2019-01-16 Thread Albert Astals Cid
aacid set the repository for this revision to R293 Baloo. Herald added projects: Frameworks, Baloo. Herald added subscribers: Baloo, kde-frameworks-devel. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D18237 To: aacid Cc: kde-frameworks-devel, #baloo, ashaposhnikov,

Power Management and Inhibition by Applications

2019-01-16 Thread Matthieu Gallien
Hello, I am trying to work on a feature request to add the ability to suppress laptop sleep when playing music. Currently, Juk is doing it via dbus calls to "org.kde.Solid.PowerManagement". I also had a look to Solid frameworks and the power classes that add an asynchronous way to do that.

Power Management and Inhibition by Applications

2019-01-16 Thread Matthieu Gallien
Hello, I am trying to work on a feature request to add the ability to suppress laptop sleep when playing music. Currently, Juk is doing it via dbus calls to "org.kde.Solid.PowerManagement". I also had a look to Solid frameworks and the power classes that add an asynchronous way to do that.

D18164: Review KateGotoBar

2019-01-16 Thread loh tar
loh.tar updated this revision to Diff 49665. loh.tar added a comment. - Connect/Disconnect - Remove m_gotoRange->setFocus Like this? :-/ I noticed bad behavior and have remove these setFocus, test it if its OK. On edit was always the focus set to the spinbox. CHANGES SINCE LAST

D18302: Add Android notification channel support

2019-01-16 Thread Volker Krause
vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REVISION SUMMARY This is required for API level 28 or higher. This actually matches the notfication events KNotification defines quite

D18288: Make the clock svg's shadows more logically correct and visually appropriate

2019-01-16 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R242:d755013c7421: Make the clock svgs shadows more logically correct and visually appropriate (authored by ngraham). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

KDE CI: Frameworks » plasma-framework » kf5-qt5 SUSEQt5.11 - Build # 16 - Still Unstable!

2019-01-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.11/16/ Project: kf5-qt5 SUSEQt5.11 Date of build: Wed, 16 Jan 2019 16:44:14 + Build duration: 20 min and counting BUILD ARTIFACTS

KDE CI: Frameworks » plasma-framework » kf5-qt5 SUSEQt5.10 - Build # 6 - Still Unstable!

2019-01-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.10/6/ Project: kf5-qt5 SUSEQt5.10 Date of build: Wed, 16 Jan 2019 16:44:14 + Build duration: 16 min and counting BUILD ARTIFACTS

D18164: Review KateGotoBar

2019-01-16 Thread Christoph Cullmann
cullmann added a comment. I would disconnect the textChanged in ::closed() to avoid having updates there even if the bar is hidden again (and reconnect on show). Otherwise I think this is ready to go in. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18164

KDE CI: Frameworks » knotifications » kf5-qt5 AndroidQt5.11 - Build # 12 - Still Failing!

2019-01-16 Thread CI System
BUILD FAILURE Build URL https://build.kde.org/job/Frameworks/job/knotifications/job/kf5-qt5%20AndroidQt5.11/12/ Project: kf5-qt5 AndroidQt5.11 Date of build: Wed, 16 Jan 2019 16:51:14 + Build duration: 49 sec and counting CONSOLE OUTPUT

D18274: Make notifications work on Android with API level < 23

2019-01-16 Thread Volker Krause
This revision was automatically updated to reflect the committed changes. Closed by commit R289:60dc31f81b4f: Make notifications work on Android with API level 23 (authored by vkrause). REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE

KDE CI: Frameworks » plasma-framework » kf5-qt5 FreeBSDQt5.12 - Build # 10 - Still Unstable!

2019-01-16 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20FreeBSDQt5.12/10/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Wed, 16 Jan 2019 16:44:14 + Build duration: 3 min 55 sec and counting JUnit Tests

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Oswald Buddenhagen
ossi added a comment. will also need to wait for commit message update, like the other change. INLINE COMMENTS > ptracer.cpp:92 > + > +qCWarning(DRKONQI_LOG) << "debugged process did not acknowledge setting > ptracer to" << debuggerpid; > +close(sfd); not wrong per se, but mildly

D18164: Review KateGotoBar

2019-01-16 Thread loh tar
loh.tar updated this revision to Diff 49653. loh.tar edited the summary of this revision. loh.tar set the repository for this revision to R39 KTextEditor. loh.tar added a comment. - Morph updateData() into a slot - connect to Document::textChanged signal REPOSITORY R39 KTextEditor

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49651. croick marked 5 inline comments as done. croick added a comment. - emit warning if no socket is available REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11235?vs=49649=49651 BRANCH ptracer REVISION DETAIL

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick added inline comments. INLINE COMMENTS > ossi wrote in ptracer.cpp:65 > you really could just use != here ... ^^ wow... REPOSITORY R871 DrKonqi REVISION DETAIL https://phabricator.kde.org/D11235 To: croick, #plasma_workspaces, #frameworks, ossi Cc: ossi, lepagevalleeemmanuel,

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Christoph Roick
croick marked 2 inline comments as done. croick added a comment. > that sounds suspicious. i don't think the kernel's behavior did changed, and the process hierarchy presumably didn't, either. the right is handed down the ancestry, and that's irrespective of whether the tracer is a "natural"

D18164: Review KateGotoBar

2019-01-16 Thread Christoph Cullmann
cullmann added a comment. In D18164#394498 , @loh.tar wrote: > - Fix typo of member vars > - Add missing setView(m_view) > - Remove FIXME hint about timer, however would a comment for me nice > - Use QString() > > Things which could

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49650. croick marked 2 inline comments as done. croick added a comment. - modify comment and remove check for EAGAIN REPOSITORY R285 KCrash CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11236?vs=49617=49650 BRANCH ptracer REVISION DETAIL

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49649. croick marked an inline comment as done. croick added a comment. - change order and add comment to kdeinit option in crashtest - add warning if debuggee does not respond as expected REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE

D18125: KateStatusBar: Add dictionary button

2019-01-16 Thread loh tar
loh.tar added a subscriber: cullmann. loh.tar added a comment. @cullmann @dhaumann A short not how to progress would be nice. - Keep codes as they are - Shorten codes as suggested - Replace codes by flags/ensign (no idea how) - Add (unneeded) label - Show (unneeded long) full name

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Oswald Buddenhagen
ossi requested changes to this revision. ossi added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > ptracer.cpp:65 > +bytes += r; > +else if (r == -1 && !(errno == EINTR)) > +break; you really could just use != here

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Oswald Buddenhagen
ossi added a comment. In D11236#394363 , @croick wrote: > Actually the point I removed does not seem to be true any longer. I'm almost certain, that the `prctrl(PR_SET_PTRACER, ...)` was not required until some time ago when starting DrKonqi as

D18299: ComboBox: fix default delegate

2019-01-16 Thread Kai Uwe Broulik
broulik added inline comments. INLINE COMMENTS > ComboBox.qml:58 > +controlRoot.currentIndex = index; > +controlRoot.popup.visible = false; > +} Are you sure this is needed? Qt docs say for that using `ItemDelegate` for a `ComboBox` is recommended as: "This

D18299: ComboBox: fix default delegate

2019-01-16 Thread Aleix Pol Gonzalez
This revision was automatically updated to reflect the committed changes. Closed by commit R858:cb49b355f946: ComboBox: fix default delegate (authored by apol). REPOSITORY R858 Qt Quick Controls 2: Desktop Style CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18299?vs=49646=49648

D18299: ComboBox: fix default delegate

2019-01-16 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R858 Qt Quick Controls 2: Desktop Style BRANCH master REVISION DETAIL https://phabricator.kde.org/D18299 To: apol, #frameworks, davidedmundson Cc: plasma-devel, jraleigh, GB_2, ragreen,

D18299: ComboBox: fix default delegate

2019-01-16 Thread Aleix Pol Gonzalez
apol created this revision. apol added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Should use the selected item when it's interacted with using the mouse. TEST PLAN Selecting an item

D18274: Make notifications work on Android with API level < 23

2019-01-16 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL https://phabricator.kde.org/D18274 To: vkrause, apol Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18164: Review KateGotoBar

2019-01-16 Thread loh tar
loh.tar updated this revision to Diff 49636. loh.tar added a comment. - Fix typo of member vars - Add missing setView(m_view) - Remove FIXME hint about timer, however would a comment for me nice - Use QString() Things which could be improved or not - The up/down buttons are a

D18249: [datamodel] Rework items insert/remove

2019-01-16 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 49630. REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18249?vs=49469=49630 REVISION DETAIL https://phabricator.kde.org/D18249 AFFECTED FILES src/declarativeimports/core/datamodel.cpp To:

D18249: [datamodel] Rework items insert/remove

2019-01-16 Thread Anthony Fieroni
anthonyfieroni added inline comments. INLINE COMMENTS > anthonyfieroni wrote in datamodel.cpp:379 > Optimization is you have [5, 6, 8] then [5, 6, 8, 9] so it make only > beginInsertRows({}, 3, 3); > full replace (m_items[sourceName] = list.toVector();) > endInsertRows() > Same if you [5, 6] >

D18288: Make the clock svg's shadows more logically correct and visually appropriate

2019-01-16 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH clock-spindle-shadow (branched from master) REVISION DETAIL https://phabricator.kde.org/D18288 To: ngraham, #vdg, #plasma, davidedmundson Cc:

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick marked 11 inline comments as done. croick added a comment. `read()` and `write()` were assumed to work right away, I added a loop around. INLINE COMMENTS > ossi wrote in crashtest.cpp:133 > this addition isn't used or explained anywhere for all i can tell. It's used to unset the

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Christoph Roick
croick marked 10 inline comments as done. croick added a comment. > please explicitly mark all handled issues as done - you'll notice on the way that you didn't address some of them. Sorry, you were just responding too quickly. > you adjusted the summary kinda the wrong way around

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49618. croick added a comment. - use debuggee pid instead of DrKonqi pid for socket name - make sure to read/write all bytes from/to socket REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11235?vs=49607=49618 BRANCH

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49617. croick marked 2 inline comments as done. croick added a comment. - unlink old sockets before binding - remove superfluous EAGAIN check - use debuggee pid instead of DrKonqi pid for socket name - fix comments REPOSITORY R285 KCrash CHANGES

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Oswald Buddenhagen
ossi requested changes to this revision. ossi added a comment. This revision now requires changes to proceed. mark handled issues as done here as well. INLINE COMMENTS > ptracer.cpp:47 > + >

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Oswald Buddenhagen
ossi requested changes to this revision. ossi added a comment. This revision now requires changes to proceed. mostly minor issues left. please explicitly mark all handled issues as done - you'll notice on the way that you didn't address some of them. you adjusted the summary kinda

D17691: Add rows info to the plasma virtual desktop protocol

2019-01-16 Thread Marco Martin
mart updated this revision to Diff 49610. mart added a comment. - remove dead symbol REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17691?vs=49425=49610 BRANCH phab/virtualdesktoprows REVISION DETAIL https://phabricator.kde.org/D17691 AFFECTED

D11236: [KCrash] Establish socket to allow change of ptracer

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49609. croick retitled this revision from "[KCrash] Establish socket to allow change of ptrace scope" to "[KCrash] Establish socket to allow change of ptracer". croick edited the summary of this revision. croick added a comment. - use poll instead of

D11235: [DrKonqi] Request change of ptrace scope from KCrash

2019-01-16 Thread Christoph Roick
croick updated this revision to Diff 49607. croick edited the summary of this revision. croick edited the test plan for this revision. croick added a comment. - rename queryPtrace to setPtracer - add a kdeinit option in crashtest REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE

D18249: [datamodel] Rework items insert/remove

2019-01-16 Thread Anthony Fieroni
anthonyfieroni added inline comments. INLINE COMMENTS > davidedmundson wrote in datamodel.cpp:379 > I don't understand what you're changing here, can you provide a bit more > detail on the exact problem. Optimization is you have [5, 6, 8] then [5, 6, 8, 9] so it make only beginInsertRows({},

D18174: Ruby: fix RegExp after ": " and fix/improve detection of HEREDOC

2019-01-16 Thread Nibaldo González
nibags closed this revision. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D18174 To: nibags, #framework_syntax_highlighting, dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D18174: Ruby: fix RegExp after ": " and fix/improve detection of HEREDOC

2019-01-16 Thread Nibaldo González
nibags edited the summary of this revision. REPOSITORY R216 Syntax Highlighting BRANCH fix-ruby REVISION DETAIL https://phabricator.kde.org/D18174 To: nibags, #framework_syntax_highlighting, dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns,

D18174: Ruby: fix RegExp after ": " and fix/improve detection of HEREDOC

2019-01-16 Thread Nibaldo González
nibags updated this revision to Diff 49600. nibags edited the summary of this revision. nibags added a comment. - Small correction in comment REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18174?vs=49212=49600 BRANCH fix-ruby REVISION

D18164: Review KateGotoBar

2019-01-16 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. I think it is a nice enhancement over the current state. I give it an accept, but wait for the "Regarding code comments, will fix it when it got in general an "OK"" before I apply