KDE CI: Frameworks kdoctools kf5-qt5 SUSEQt5.10 - Build # 48 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.10/48/ Project: Frameworks kdoctools kf5-qt5 SUSEQt5.10 Date of build: Wed, 15 Aug 2018 06:44:08 + Build duration: 1 min 3 sec and counting JUnit Tests

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.9 - Build # 121 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.9/121/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.9 Date of build: Wed, 15 Aug 2018 03:45:42 + Build duration: 20 min and counting CONSOLE OUT

KDE CI: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.10 - Build # 124 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20WindowsMSVCQt5.10/124/ Project: Frameworks plasma-framework kf5-qt5 WindowsMSVCQt5.10 Date of build: Wed, 15 Aug 2018 03:45:42 + Build duration: 19 min and counting

KDE CI: Frameworks kio kf5-qt5 SUSEQt5.9 - Build # 218 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.9/218/ Project: Frameworks kio kf5-qt5 SUSEQt5.9 Date of build: Wed, 15 Aug 2018 03:45:12 + Build duration: 18 min and counting CONSOLE OUTPUT [...truncated 17

KDE CI: Frameworks kio kf5-qt5 WindowsMSVCQt5.10 - Build # 217 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20WindowsMSVCQt5.10/217/ Project: Frameworks kio kf5-qt5 WindowsMSVCQt5.10 Date of build: Wed, 15 Aug 2018 03:45:12 + Build duration: 14 min and counting CONSOLE OUTPUT

KDE CI: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 - Build # 198 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20SUSEQt5.10/198/ Project: Frameworks plasma-framework kf5-qt5 SUSEQt5.10 Date of build: Wed, 15 Aug 2018 03:45:42 + Build duration: 8 min 1 sec and counting CONS

KDE CI: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.10 - Build # 48 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20plasma-framework%20kf5-qt5%20FreeBSDQt5.10/48/ Project: Frameworks plasma-framework kf5-qt5 FreeBSDQt5.10 Date of build: Wed, 15 Aug 2018 03:45:46 + Build duration: 6 min 42 sec and counting

KDE CI: Frameworks kio kf5-qt5 FreeBSDQt5.10 - Build # 128 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20FreeBSDQt5.10/128/ Project: Frameworks kio kf5-qt5 FreeBSDQt5.10 Date of build: Wed, 15 Aug 2018 03:45:12 + Build duration: 2 min 38 sec and counting CONSOLE OUTPUT [.

KDE CI: Frameworks kio kf5-qt5 SUSEQt5.10 - Build # 373 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.10/373/ Project: Frameworks kio kf5-qt5 SUSEQt5.10 Date of build: Wed, 15 Aug 2018 03:45:11 + Build duration: 1 min 27 sec and counting CONSOLE OUTPUT [...trun

D9247: Extend Scripting API to allow executing commands

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R39:72c51e15659f: Extend Scripting API to allow executing commands (authored by dhaumann, committed by cullmann). REPOSITOR

KDE CI: Frameworks ktexteditor kf5-qt5 FreeBSDQt5.10 - Build # 52 - Failure!

BUILD FAILURE Build URL https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20FreeBSDQt5.10/52/ Project: Frameworks ktexteditor kf5-qt5 FreeBSDQt5.10 Date of build: Tue, 14 Aug 2018 21:52:22 + Build duration: 41 sec and counting CONSOLE OUTPUT

D12587: Indentation script for R

This revision was automatically updated to reflect the committed changes. Closed by commit R39:467cf37fd04f: Indentation script for R (authored by cullmann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D12587?vs=33259&id=39754 REVISION DETAIL https://p

D14817: Convert manual install test to an autotest

habacker added a comment. I applied a fix to use absolute pathes. Unfortunally windows builds have an additional issue: > https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20WindowsMSVCQt5.10/36/console 21:26:34 -- Installing: C:/CI/workspace/Frameworks kdoctools kf5-qt5

D12587: Indentation script for R

cullmann accepted this revision. This revision is now accepted and ready to land. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12587 To: devillemereuil, #ktexteditor, #rkward, cullmann Cc: kwrite-devel, kde-frameworks-devel, cullmann, tfry, dhaumann, michaelh, kev

D12587: Indentation script for R

cullmann added a comment. Herald edited subscribers, added: kde-frameworks-devel, kwrite-devel; removed: Frameworks. Given it seems nobody has time to do tests, I will merge this. We can still remove it again but just letting rot this nice contribution in the phabricator won't help any R us

D7245: Improve reStructuredText highlighting

cullmann closed this revision. cullmann added a comment. Ok, then we just close this and wait for the next update. Thanks for the work on that! REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_highlighting, dhaumann, cullmann Cc: cullmann, dhaumann, #

D7245: Improve reStructuredText highlighting

turbov added a comment. @cullmann, Up to you... When I'll fix it for the cases I've found (trying to write the `hightlighting.rst` test file) I come back w/ a new PR :) REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_highlighting, dhaumann, cullm

D12768: Allow wrapping selection off top/bottom of autocomplete results

cullmann commandeered this revision. cullmann added a reviewer: sraizada. cullmann added a comment. I don't think such a behavior change is wanted. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12768 To: cullmann, #ktexteditor, sraizada Cc: dhaumann, cullmann, k

D12768: Allow wrapping selection off top/bottom of autocomplete results

cullmann abandoned this revision. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12768 To: cullmann, #ktexteditor, sraizada Cc: dhaumann, cullmann, kwrite-devel, kde-frameworks-devel, brauch, #ktexteditor, michaelh, kevinapavew, ngraham, bruns, demsking, head7, kfun

KDE CI: Frameworks kdoctools kf5-qt5 SUSEQt5.10 - Build # 47 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.10/47/ Project: Frameworks kdoctools kf5-qt5 SUSEQt5.10 Date of build: Tue, 14 Aug 2018 21:24:06 + Build duration: 1 min 1 sec and counting JUnit Tests

D5802: ViewPrivate, KateSearchBar, KateVi::MatchHighlighter: use selection foreground for search highlights

brauch added a comment. Herald added a project: Kate. Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks. Can't we simply update our shipped schemas, and expect users with custom schemas to fix them? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.

D14345: Give the PlasmaComponents3 TextField the ability to have a Clear button

ngraham added a comment. Ah, I'm sorry. I'm getting my patches confused. The PC2 TextField's bug is that it doesn't show placeholder text properly with light themes when software rendering isn't being used. I'm open to other ways of fixing that. REPOSITORY R242 Plasma Framework (Library)

D14345: Give the PlasmaComponents3 TextField the ability to have a Clear button

davidedmundson added a comment. > Placeholder text looks good with fractional scale factors (PC3 and QQC2 ones have this, PC2 and QQC1 ones do not) No, this is where we have crossed wires. PC3, QQC2 and PC2 look good. We control the renderType. Only QQC1 desktop theme has the fon

D14817: Convert manual install test to an autotest

habacker added a comment. In D14817#308903 , @ltoscano wrote: > This worked with Qt 5.9 on the CI and locally with 5.10 too, but the CI seems to be unhappy for openSUSE/Qt 5.10 and Windows/Qt 5.10 (it works on openSUSE/Qt 5.9 and FreeBSD/Qt 5.10

D14345: Give the PlasmaComponents3 TextField the ability to have a Clear button

ngraham added a comment. To back up a bit, the issue is that we currently have no TextField with has the following characteristics: 1. Has built-in functionality for clear and password reveal buttons, so each client doesn't need to re-invent the wheel (PC2 one has this, PC3, QQC1, and Q

D14345: Give the PlasmaComponents3 TextField the ability to have a Clear button

davidedmundson added a comment. > Upstream the features we wrote into our PlasmaComponents TextField and then just use that. Downsides: lengthy process, will take forever before we can actually use it here. Yes, but there's also no real rush from a PC POV. > Finally fix the Qt bug t

D11838: Turn on line numbers by default

This revision was not accepted when it landed; it landed in state "Needs Revision". This revision was automatically updated to reflect the committed changes. Closed by commit R40:d7b19c5abffc: turn on line numbers and line modifications per default for Kate (authored by cullmann). CHANGED PRIOR

D14345: Give the PlasmaComponents3 TextField the ability to have a Clear button

ngraham added a comment. So from in-person conversations this week, it seems like we have a few paths forward here: - Upstream the features we wrote into our PlasmaComponents TextField and then just use that. **Downsides**: lengthy process, will take forever before we can actually use i

D6473: Crash when replacing new lines with spaces

dhaumann commandeered this revision. dhaumann edited reviewers, added: jsalatas; removed: dhaumann. dhaumann added a comment. Reassign REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D6473 To: dhaumann, #ktexteditor, jsalatas Cc: kde-frameworks-devel, mwolff, antho

D14847: Fix crash when replacing \n around empty lines

cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. Ok :P REPOSITORY R39 KTextEditor BRANCH fix-replace-newline-empty-lines (branched from master) REVISION DETAIL https://phabricator.kde.org/D14847 To: dhaumann, jsalatas, ltoscan

D14847: Fix crash when replacing \n around empty lines

cullmann requested changes to this revision. cullmann added a comment. This revision now requires changes to proceed. Better test with (a.isEmpty() && b.isEmpty()) for the special case to avoid comparing long lines again. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde

D14847: Fix crash when replacing \n around empty lines

dhaumann created this revision. dhaumann added reviewers: jsalatas, ltoscano, kfunk, cullmann, anthonyfieroni, mwolff. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. dhaumann requested review of this revision. REVISION SUMMARY The problem

D9247: Extend Scripting API to allow executing commands

cullmann accepted this revision. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D9247 To: dhaumann, cullmann, mwolff, kfunk Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann

D14847: Fix crash when replacing \n around empty lines

cullmann accepted this revision. This revision is now accepted and ready to land. REPOSITORY R39 KTextEditor BRANCH fix-replace-newline-empty-lines (branched from master) REVISION DETAIL https://phabricator.kde.org/D14847 To: dhaumann, jsalatas, ltoscano, kfunk, cullmann, anthonyfieroni,

D6473: Crash when replacing new lines with spaces

dhaumann abandoned this revision. dhaumann added a comment. Fixed in D14847 in a different way. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D6473 To: dhaumann, #ktexteditor, jsalatas Cc: kde-frameworks-devel, mwolff, anthony

D14828: remove highlighting download dialog

rjvbb added a comment. Downloading already didn't work for me since quite a while (the operation never completed) but what does this mean for highlighting files I write myself (or someone's mom does)? > Other users who do not follow the latest development releases [...] are likely not t

D14847: Fix crash when replacing \n around empty lines

dhaumann updated this revision to Diff 39737. dhaumann added a comment. - Optimize by checking isEmpty() REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14847?vs=39736&id=39737 BRANCH fix-replace-newline-empty-lines (branched from master) REVISION DET

D14817: Convert manual install test to an autotest

ltoscano added a comment. This worked with Qt 5.9 on the CI and locally with 5.10 too, but the CI seems to be unhappy for openSUSE/Qt 5.10 and Windows/Qt 5.10 (it works on openSUSE/Qt 5.9 and FreeBSD/Qt 5.10): - https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20SUSEQt5.10/46/

D13940: Add syntax highlighting support for Stan

This revision was automatically updated to reflect the committed changes. Closed by commit R216:0a39f67256b3: Add syntax highlighting support for Stan (authored by cullmann). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D13940?vs=37388&id=39723#toc REPOSITORY R216 Syntax Highlighting

D14847: Fix crash when replacing \n around empty lines

This revision was automatically updated to reflect the committed changes. Closed by commit R39:c90064799dfb: Fix crash when replacing \n around empty lines (authored by dhaumann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14847?vs=39737&id=39738 REVIS

Re: Upcoming reorganisation of the CI system

On Tue, 14 Aug 2018, 17:09 Christoph Feck, wrote: > On 14.08.2018 15:03, Ben Cooksley wrote: > > Currently CI jobs are all created within a flat namespace, meaning > > that it is quite difficult to view the overall status of an individual > > project. Additionally, it creates the issue that the m

D14826: inline note interface wip #2

brauch updated this revision to Diff 39721. brauch added a comment. add noteActivated notifier function When a note is mouse-overed or clicked, a function on the note provider is called, giving the point it was hovered or clicked in note coordinates, the type of event, and the note the

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 - Build # 100 - Fixed!

BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/100/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 Date of build: Tue, 14 Aug 2018 15:17:15 + Build duration: 1 min 39 sec and counting

D14274: Modelines: add byte-order-mark & small fixes

cullmann requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D14274 To: nibags, turbov, dhaumann, cullmann, vkrause, #kate Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapa

D14274: Modelines: add byte-order-mark & small fixes

cullmann reopened this revision. cullmann added a comment. This revision is now accepted and ready to land. I needed to revert this again, as we get tests failures: https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/97/testReport/junit/(root)/TestSuite/testhi

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 - Build # 99 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/99/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 Date of build: Tue, 14 Aug 2018 15:10:44 + Build duration: 6 min 30 sec and counting

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 - Build # 155 - Fixed!

BUILD SUCCESS Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.10/155/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 Date of build: Tue, 14 Aug 2018 15:13:22 + Build duration: 2 min 15 sec and counting

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 - Build # 154 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.10/154/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 Date of build: Tue, 14 Aug 2018 15:08:04 + Build duration: 5 min 17 sec and counting

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 - Build # 98 - Still Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/98/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 Date of build: Tue, 14 Aug 2018 15:08:04 + Build duration: 1 min 46 sec and counting

Re: Upcoming reorganisation of the CI system

On 14.08.2018 15:03, Ben Cooksley wrote: Currently CI jobs are all created within a flat namespace, meaning that it is quite difficult to view the overall status of an individual project. Additionally, it creates the issue that the main default view can take a significant amount of time to load.

D11543: Optimize many syntax highlighting files and fix the '/' char of SQL

This revision was automatically updated to reflect the committed changes. Closed by commit R216:43396e0a9773: Optimize many syntax highlighting files and fix the '/' char of SQL (authored by cullmann). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D11543?vs=31565&id=39719#toc REPOSITORY

D10937: Retouching of Screen Layout Selection OSD Icons

pstefan added a comment. Sure, I can do that, but I'd like to get feedback on the final form first :) REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10937 To: pstefan, #breeze, ngraham Cc: broulik, bruns, #vdg, kde-frameworks-devel, emateli, abetts, mart, ngrah

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 - Build # 97 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/97/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.9 Date of build: Tue, 14 Aug 2018 14:55:59 + Build duration: 6 min 10 sec and counting

D8964: Fix two bugs in KMessageWidget

dhaumann accepted this revision. dhaumann added a comment. I just tested again with KWrite - I think it behaves good there. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D8964 To: aacid, #frameworks, dhaumann Cc: kde-frameworks-devel, ngraham, dhaumann, antho

KDE CI: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 - Build # 153 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.10/153/ Project: Frameworks syntax-highlighting kf5-qt5 SUSEQt5.10 Date of build: Tue, 14 Aug 2018 14:55:59 + Build duration: 1 min 56 sec and counting

D14274: Modelines: add byte-order-mark & small fixes

This revision was automatically updated to reflect the committed changes. Closed by commit R216:bffb66cb3ed8: Modelines: add byte-order-mark & small fixes (authored by nibags, committed by cullmann). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1

D6473: Crash when replacing new lines with spaces

anthonyfieroni added a comment. I'm aware of that it will fix the crash but functionally still will be missing, but i'll give a try. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D6473 To: jsalatas, #ktexteditor, dhaumann Cc: kde-frameworks-devel, mwolff, anthon

D6473: Crash when replacing new lines with spaces

dhaumann added a comment. Hm, can you send a patch that fixes this? :-) REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D6473 To: jsalatas, #ktexteditor, dhaumann Cc: kde-frameworks-devel, mwolff, anthonyfieroni, dhaumann, kfunk, ltoscano, kwrite-devel, michaelh,

D12295: Supporting nested brackets for Kate autobrackets

cullmann added a reviewer: brauch. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12295 To: sraizada, #ktexteditor, #kate, cullmann, dhaumann, brauch Cc: brauch, ngraham, kwrite-devel, kde-frameworks-devel, #ktexteditor, michaelh, kevinapavew, bruns, demsking, head7

D12295: Supporting nested brackets for Kate autobrackets

cullmann added a subscriber: brauch. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12295 To: sraizada, #ktexteditor, #kate, cullmann, dhaumann Cc: brauch, ngraham, kwrite-devel, kde-frameworks-devel, #ktexteditor, michaelh, kevinapavew, bruns, demsking, head7, cull

D4234: Change algorithm for autobrace.

cullmann added a reviewer: brauch. Herald added a project: Kate. Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D4234 To: cactus, #ktexteditor, mwolff, brauch Cc: kde-frameworks-devel, nalvar

D7245: Improve reStructuredText highlighting

cullmann added a comment. Hmm, should we not commit this state at least as is? Or do you want to do a new request including a new state & tests? I would be fine with just pushing this as an intermediate step, too. REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #fra

D14236: Add some improvements to kate-syntax-highlighter for use in scripting

xciml updated this revision to Diff 39700. xciml added a comment. Hi! Good idea, what about this? REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14236?vs=38102&id=39700 REVISION DETAIL https://phabricator.kde.org/D14236 AFFECTED FILES src/

KDE CI: Frameworks kdoctools kf5-qt5 WindowsMSVCQt5.10 - Build # 35 - Unstable!

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks%20kdoctools%20kf5-qt5%20WindowsMSVCQt5.10/35/ Project: Frameworks kdoctools kf5-qt5 WindowsMSVCQt5.10 Date of build: Tue, 14 Aug 2018 14:07:08 + Build duration: 5 min 38 sec and counting JUn

D14826: inline note interface wip #2

brauch added a comment. Thanks for the feedback! I will try doing a few more things with this interace and then maybe discuss again with the other kate people here at Akademy about which one they like better. About the tracking, I don't think anything is needed on the side of the interf

D14817: Convert manual install test to an autotest

This revision was automatically updated to reflect the committed changes. Closed by commit R238:f0cc8b2ccb4a: Convert manual install test to an autotest (authored by habacker). Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. CHANGED PRIOR TO COMMIT https://p

D6473: Crash when replacing new lines with spaces

anthonyfieroni added a comment. About me when line(...) is accessed it should check for nullptr as well if (auto l = line(a)) { return l->accessor(); } https://phabricator.kde.org/source/ktexteditor/browse/master/src/render/katelayoutcache.cpp;ad51cece443bc6bb643fa4ca94

D7245: Improve reStructuredText highlighting

turbov added a comment. You can just close this MR, I'll be back sometime later w/ highlighting example and tests... REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_highlighting, dhaumann, cullmann Cc: cullmann, dhaumann, #frameworks, michaelh, ngraha

D7245: Improve reStructuredText highlighting

turbov marked an inline comment as done. turbov added a comment. I'm using this syntax 'till nowadays from my repo... I know about some not covered cases in it, but have no time these days to improve it even more %) Most of the time I quite satisfied w/ it... and sorry for the tests :( REVI

D10937: Retouching of Screen Layout Selection OSD Icons

broulik added a comment. While you're at it can't you move them to plasma-framework? Having them in Breeze makes it not work with other themes and also they are only shown in the context of Plasma anyway REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D10937 To

D6473: Crash when replacing new lines with spaces

dhaumann requested changes to this revision. dhaumann added a comment. This revision now requires changes to proceed. Herald added a project: Kate. Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks. I was just about to merge this patch. But the crash still happens for

D14826: inline note interface wip #2

michalsrb added a comment. Thank you for working on this. This interface would work for the kdev-sourceinfo use case just as well as the old one. I think the problem with the old interface that you described is valid. This version gives more flexibility to the `InlineNotesProvider` with

D10937: Retouching of Screen Layout Selection OSD Icons

pstefan added a comment. Sorry folks for the endless period of no activity. After some talking with ngraham I removed the laptop metaphor for a much more general standalone display. Also the projection screen was swapped out. My last feedback was that the window icons were too detailed.

D11470: SQL: various improvements and fix if/case/loop/end detection with SQL (Oracle)

dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. I guess ok - if issues arise, we have to fix them :-) REPOSITORY R216 Syntax Highlighting BRANCH sql2 REVISION DETAIL https://phabricator.kde.org/D11470 To: jpoelen, #framework_

D7245: Improve reStructuredText highlighting

dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Sorry for the delay. Shall we take care of this? REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_highlighting, dhaumann, cullmann Cc: cullmann,

D7245: Improve reStructuredText highlighting

cullmann accepted this revision. REVISION DETAIL https://phabricator.kde.org/D7245 To: turbov, #kate, #framework_syntax_highlighting, dhaumann, cullmann Cc: cullmann, dhaumann, #frameworks, michaelh, ngraham, bruns

D10587: Restore KTextEditor Document Dbus bindings

cullmann commandeered this revision. cullmann edited reviewers, added: lukedashjr; removed: cullmann. cullmann added a comment. As said, I agree a fix for the unsaved documents vs. no swap files problem is wanted, but a single use case is not enough to bring back the dbus overhead per documen

D10587: Restore KTextEditor Document Dbus bindings

cullmann abandoned this revision. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D10587 To: cullmann, dhaumann, gregormi, lukedashjr Cc: kwrite-devel, kde-frameworks-devel, anthonyfieroni, #ktexteditor, #kate, asturmlechner, gregormi, dhaumann, cullmann, michaelh, ke

Re: Upcoming reorganisation of the CI system

On mardi 14 août 2018 15:03:42 CEST Ben Cooksley wrote: > Hi all, > > Currently CI jobs are all created within a flat namespace, meaning > that it is quite difficult to view the overall status of an individual > project. Additionally, it creates the issue that the main default view > can take a si

D14826: inline note interface wip #2

brauch updated this revision to Diff 39692. brauch added a comment. add missing files REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14826?vs=39682&id=39692 BRANCH inlinenotes REVISION DETAIL https://phabricator.kde.org/D14826 AFFECTED FILES src

Upcoming reorganisation of the CI system

Hi all, Currently CI jobs are all created within a flat namespace, meaning that it is quite difficult to view the overall status of an individual project. Additionally, it creates the issue that the main default view can take a significant amount of time to load. To resolve this we intend to shif

D14661: Force reevaluation of Predicates if interfaces are removed

This revision was automatically updated to reflect the committed changes. bruns marked an inline comment as done. Closed by commit R245:8e9fc89918c4: Force reevaluation of Predicates if interfaces are removed (authored by bruns). REPOSITORY R245 Solid CHANGES SINCE LAST UPDATE https://phabri

D14829: Remove warning about old kiosk feature that no longer applies.

dfaure closed this revision. REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D14829 To: dfaure, davidedmundson, arichardson, broulik Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14828: remove highlighting download dialog

This revision was automatically updated to reflect the committed changes. Closed by commit R39:705f8e0419c4: remove highlighting download dialog (authored by cullmann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14828?vs=39686&id=39688 REVISION DETAIL

D14828: remove highlighting download dialog

dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Looks good. But along with this change, please change the docbook in kate.git: We have doc/katepart/configuring.docbook Download Highlighting Files...

D14829: Remove warning about old kiosk feature that no longer applies.

broulik accepted this revision. This revision is now accepted and ready to land. REPOSITORY R237 KConfig BRANCH master REVISION DETAIL https://phabricator.kde.org/D14829 To: dfaure, davidedmundson, arichardson, broulik Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D14829: Remove warning about old kiosk feature that no longer applies.

dfaure created this revision. dfaure added reviewers: davidedmundson, arichardson. Herald added a project: Frameworks. Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks. dfaure requested review of this revision. REVISION SUMMARY The concept of KStandardDirs resources no

D14828: remove highlighting download dialog

cullmann created this revision. cullmann added a reviewer: dhaumann. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. cullmann requested review of this revision. REVISION SUMMARY rational: 1. there is one frameworks release per month tha

D14779: Fix 'kformattest fails with installed kcoreaddons language package'

aacid added a comment. In D14779#308593 , @habacker wrote: > You have remembered that in this case the loader created by ecm_create_qm_loader(kcoreaddons_QM_LOADER kcoreaddons5_qt) must be adjusted to find this - not installed - fake translation

D14824: Relicense modelines.xml to MIT license

dhaumann added a comment. Thanks, Martin just change the kateversion number, which was changed again later by us again. So I will proceed now. Thanks. REPOSITORY R216 Syntax Highlighting BRANCH relicense-modelines-mit (branched from master) REVISION DETAIL https://phabricator.kde.org/

D14824: Relicense modelines.xml to MIT license

This revision was automatically updated to reflect the committed changes. Closed by commit R216:857a277110d5: Relicense modelines.xml to MIT license (authored by dhaumann). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14824?vs=39680&id=39685 REV

D14824: Relicense modelines.xml to MIT license

turbov accepted this revision. turbov added a comment. I agree w/ relicense REPOSITORY R216 Syntax Highlighting BRANCH relicense-modelines-mit (branched from master) REVISION DETAIL https://phabricator.kde.org/D14824 To: dhaumann, vkrause, cullmann, martinwalch, turbov Cc: kwrite-deve

D14779: Fix 'kformattest fails with installed kcoreaddons language package'

habacker added a comment. You have remembered that in this case the loader created by ecm_create_qm_loader(kcoreaddons_QM_LOADER kcoreaddons5_qt) must be adjusted to find this - not installed - fake translation by default ? REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator

D14824: Relicense modelines.xml to MIT license

vkrause accepted this revision. REPOSITORY R216 Syntax Highlighting BRANCH relicense-modelines-mit (branched from master) REVISION DETAIL https://phabricator.kde.org/D14824 To: dhaumann, vkrause, cullmann, martinwalch, turbov Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew,

D14824: Relicense modelines.xml to MIT license

dhaumann added a comment. For those who are curious: This change is required since modelines.xml is a file that is included in many other xml files via IncludeRules. Given our policy of new highlighting files should be MIT licensed, this also applies to included definitions. That's why this

D14826: inline note interface wip #2

brauch added a comment. Sample patch for KDevelop's problem highlighter plus screenshot: F6192637: hl.png F6192639: inline-problems.diff REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org

D14824: Relicense modelines.xml to MIT license

cullmann accepted this revision. This revision is now accepted and ready to land. REPOSITORY R216 Syntax Highlighting BRANCH relicense-modelines-mit (branched from master) REVISION DETAIL https://phabricator.kde.org/D14824 To: dhaumann, vkrause, cullmann, martinwalch, turbov Cc: kwrite-de

D14826: inline note interface wip #2

brauch edited the summary of this revision. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D14826 To: brauch, michalsrb, dhaumann, cullmann Cc: anthonyfieroni, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhauma

D14779: Fix 'kformattest fails with installed kcoreaddons language package'

aacid added a comment. Personally i would include (and use) a fake translation as part of the tests and then it would just evaluate to the actual result, i.e. QCOMPARE(format.formatSpelloutDuration(361), QStringLiteral("1 hour(s)")); would end up being QCOMPARE(format.f

D14779: Fix 'kformattest fails with installed kcoreaddons language package'

habacker updated this revision to Diff 39684. habacker edited the summary of this revision. habacker added a comment. - after feedback from ecm maintainers the fix could be refactored to not be depending on ecm patch REPOSITORY R244 KCoreAddons CHANGES SINCE LAST UPDATE https://phabricat

  1   2   >