- User Since
- Apr 16 2015, 7:53 PM (209 w, 1 d)
Wed, Apr 17
Tue, Apr 16
patch lgtm now, many thanks!
Mon, Apr 15
quite obviously libclang doesn't handle it
Sun, Apr 14
Sat, Apr 13
Oh, really? Hmm! I wouldn't be opposed to enable compilation with exceptions myself, what do the others say? We don't need to use them excessively, but for error handling in async promise chains, that would be quite useful I think?
Hey Thomas, please don't remove the cache. See f2a6941e086cdf506c8fb1798c52982bff43792d for why this was introduced. Your tests don't include other files, so probably that's why you didn't see any effect of the cache?
Mon, Apr 1
yes, QtPromise or AsyncFuture (https://github.com/benlau/asyncfuture) could be used - I wouldn't be opposed to introducing it as a thirdparty dependency (or git submodule)
if it applies cleanly, you can also push to 5.3, otherwise master is fine - it's not a really urgent bug fix after all (imo)
Tue, Mar 26
your simplified patch seems to work like a charm, thanks a lot!
Mon, Mar 25
Sun, Mar 24
one minor nit, otherwise looks like a good improvement
Fri, Mar 22
Mar 20 2019
Mar 18 2019
Mar 12 2019
thanks, lgtm - I'll amend the last nits and apply it for you - if you give me full name and email address such that I can set you as the main author of this patch
Mar 11 2019
ah, then let's get this in as-is
great! can we get a unit test for this? though I'm unsure if we ever revived the unit tests from our old cmake integration
Mar 10 2019
yes I would be fine with that personally!
ok cool, then please get rid of our internal copy!
nice, this is getting better! some suggestions on how to improve the code quality, and then some potential issues I can think of - please fix or document why they aren't an issue
cool, this is great! and it fixes the issue you originally found in testActiveDocumentsGetBestPriority?
Rene, instead of thinking about what-ifs, maybe try it out first? Most notably, the tooltips only show up when you press ALT and keep it pressed. Otherwise, the tooltips don't show up - unless you hover code with your mouse cursor. Moving the keyboard edit cursor won't ever trigger tooltips. Or are you somehow moving your mouse cursor while typing?! Don't do that :)
Mar 5 2019
Mar 2 2019
Feb 27 2019
Feb 21 2019
Feb 19 2019
we override execution in our own completion models, so this patch will only change the behavior for the builtin word and keyword completion models in ktexteditor I believe
Feb 18 2019
Feb 15 2019
Done, keyboard navigation should work now - at least it does according to my testing! Please test this as well and report back if you spot any issues.
I'm looking into the keyboard navigation now