Re: Dictionary Source Code

https://invent.kde.org/plasma/kdeplasma-addons/ applets/dict

Dictionary Source Code

Hi Plasma Dev Team, I use the dictionary widget often and I thought of a project that is similar to the dictionary widget. I'm a bit stuck right now because I'm new to Qt. To make the process a bit easier, I was wondering if you guys can send me the source code. It would be greatly a

Re: licensing terms of plasma dictionary runner

I'm fine with GPL-2.0-or-later, if that's fine with you. Otherwise let me know and I'll pick a different one. Thanks for getting in touch, and sorry for the delay. Jason

Re: licensing terms of plasma dictionary runner

On Fri, Aug 7, 2020 at 6:28 PM Jason A. Donenfeld wrote: > > I'm fine with GPL-2.0-or-later, if that's fine with you. Otherwise let > me know and I'll pick a different one. GPL-2.0-or-later is perfect, I'll update the files. > Thanks for getting in touch, and sorry for the delay. No worries. T

Re: licensing terms of plasma dictionary runner

I guess this got buried in the inbox. Try #2 :) | | v On Thu, Mar 26, 2020 at 11:11 AM Harald Sitter wrote: > > Hey Jason! > > Hope you are doing well. > I've just noticed that the dictionary runner you've made for Plasma > doesn't really have licensing ter

D28192: WIP: Refactor dictionary runner

sitter added inline comments. INLINE COMMENTS > alex wrote in dictionaryrunner_config.cpp:29 > That makes sense but one question: The doc says: `...However, if you for some > reason reimplement it and also are using KConfigXT, you must call this > function`, does this mean we can assume that th

D28192: WIP: Refactor dictionary runner

alex added inline comments. INLINE COMMENTS > sitter wrote in dictionaryrunner_config.cpp:29 > That's an implementation detail though, is it not? From the outside we > shouldn't make assumption about what the implementation does unless the > documentation says what we can assume. > Today the ba

D28192: WIP: Refactor dictionary runner

sitter added inline comments. INLINE COMMENTS > alex wrote in dictionaryrunner.cpp:90 > The current behavior is that KRunner closes when a match is selected. > But if you set the data, for instance: > `match.setData(QStringLiteral("Hello There!"));` > The text `Hello There!` will show up in KRunn

D28192: WIP: Refactor dictionary runner

alex added inline comments. INLINE COMMENTS > sitter wrote in dictionaryrunner.cpp:90 > You could simply use the lastPartOfSpeech I would guess. From what I've seen > and what Kai tells me you really only need to set setData if you want to > implement actions (`::actionsForMatch`) and need to c

D28192: WIP: Refactor dictionary runner

rs.values(source)) { > +for (ThreadData *data: m_lockers.values(source)) { > QMutexLocker locker(&data->mutex); I think our style is space before and after : > dictionaryrunner.cpp:37 > +} > +setSyntaxes({Plasma::RunnerSyntax(Plasma::RunnerSyntax(

licensing terms of plasma dictionary runner

Hey Jason! Hope you are doing well. I've just noticed that the dictionary runner you've made for Plasma doesn't really have licensing terms in its source files. https://cgit.kde.org/kdeplasma-addons.git/tree/runners/dictionary/dictionaryrunner.cpp It'd be really grand if you

D28192: WIP: Refactor dictionary runner

sitter added a comment. About fixing the UX: There is some kind of preview tech in milou which you can take a look at and maybe build upon. The line space is just never going to be enough for a text only representation of a dict entry, plus it looks fairly forced anyway. So, my thinking

D28192: WIP: Refactor dictionary runner

92 AFFECTED FILES runners/dictionary/config_keys.h runners/dictionary/dictionarymatchengine.cpp runners/dictionary/dictionaryrunner.cpp runners/dictionary/dictionaryrunner.h runners/dictionary/dictionaryrunner_config.cpp runners/dictionary/dictionaryrunner_config.h To: alex, broulik, ngr

D22814: Remove colon (:) prefix when looking up dictionary word

wbauer added a comment. Thank you! REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D22814 To: broulik, #plasma, mart, kossebau, davidedmundson Cc: wbauer, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-

D22814: Remove colon (:) prefix when looking up dictionary word

broulik added a comment. @wbauer done: https://cgit.kde.org/kdeplasma-addons.git/commit/?h=Plasma/5.12&id=eac0dbd5f1c0f1ba54bd9c9682da096d9c78db2a REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D22814 To: broulik, #plasma, mart, kossebau, davidedmundson Cc: wb

D22814: Remove colon (:) prefix when looking up dictionary word

wbauer added a comment. Can this be pushed to the 5.12 branch as well, please? It's broken in 5.12.8 as well (https://bugs.kde.org/390776 was actually reported against 5.12.1), the patch applies cleanly and does fix the problem. REPOSITORY R114 Plasma Addons REVISION DETAIL https://pha

D14063: KRunner Dictionary can't find anything

broulik added a comment. Oh, I totally missed that, sorry. This is basically committed now as D22814 REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D14063 To: McPain, broulik Cc: ngraham, mart, plasma-devel, LeGast00n, jral

D22814: Remove colon (:) prefix when looking up dictionary word

This revision was automatically updated to reflect the committed changes. Closed by commit R114:cda996dfb011: Remove colon (:) prefix when looking up dictionary word (authored by broulik). REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22814?vs=62763&am

D22814: Remove colon (:) prefix when looking up dictionary word

broulik created this revision. broulik added reviewers: Plasma, mart, kossebau. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision. REVISION SUMMARY The dictionary engine lets us specify a server and dictionary but we don't

D16038: Allow dictionary runner to search multiple times

This revision was automatically updated to reflect the committed changes. Closed by commit R114:04dff1e01632: Allow dictionary runner to search multiple times (authored by McPain, committed by aacid). REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16038

D16038: Allow dictionary runner to search multiple times

https://phabricator.kde.org/D16038 AFFECTED FILES runners/dictionary/dictionarymatchengine.cpp runners/dictionary/dictionarymatchengine.h To: McPain, jriddell Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

McPain added a comment. In D14064#322643 , @davidedmundson wrote: > Urgh. Just read the code. > > We don't reset when we type a new letter. To do so would have a jumpy ui. Instead we reset when we get results back. > > That leaves a prob

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

davidedmundson added a comment. Urgh. Just read the code. We don't reset when we type a new letter. To do so would have a jumpy ui. Instead we reset when we get results back. That leaves a problem when you search for "firefo" ( with 1 result) to "firefodfhhxtffrdh" with zero results

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. No point pinging if we haven't replied to Kai's comments/question. REPOSITORY R112 Milou REVISION DETAIL https://phabricator.kde.org/D14064 To: McPain, brouli

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

acrouthamel added a comment. Any movement on this? It would be great to use the dictionary in krunner. Still broken in 5.13.4. REPOSITORY R112 Milou REVISION DETAIL https://phabricator.kde.org/D14064 To: McPain, broulik Cc: acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

McPain added a comment. ping? REPOSITORY R112 Milou REVISION DETAIL https://phabricator.kde.org/D14064 To: McPain, broulik Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

McPain added a comment. In D14064#291125 , @broulik wrote: > I would rather see a thorough investigation on what this does and why it is or is not needed. There's a couple of slow runners that suffer issues because of premature resetting but I d

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

broulik added a comment. I would rather see a thorough investigation on what this does and why it is or is not needed. There's a couple of slow runners that suffer issues because of premature resetting but I don't understand why it has this code in the first place. REPOSITORY R112 Milou

D14063: KRunner Dictionary can't find anything

ngraham added a comment. Thanks for the patch! It will be nice to get the Dictionary runner working again. Please change `https://bugs.kde.org/show_bug.cgi?id=390776` to `BUG: 390776`. See https://community.kde.org/Infrastructure/Phabricator#Add_special_keywords REPOSITORY R114

D14064: Dictionary doesn't have enough time to complete query before resetting by milou

ngraham added a comment. Thanks for the patch! It will be nice to get the Dictionary runner working again. Please change `https://bugs.kde.org/show_bug.cgi?id=390776` to `CCBUG: 390776`. See https://community.kde.org/Infrastructure/Phabricator#Add_special_keywords REPOSITORY R112

D14063: KRunner Dictionary can't find anything

mart added a comment. needs a more complete comment in the summary on what this change does REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D14063 To: McPain, broulik Cc: mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abet

D14064: Dictionary haven't enough time to complete query before resetting by milou

McPain created this revision. McPain added a reviewer: broulik. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. McPain requested review of this revision. REVISION SUMMARY https://bugs.kde.org/show_bug.cgi?id=390776 REPOSITORY R112 Milou

D14063: KRunner Dictionary can't find anything

Plasma Addons REVISION DETAIL https://phabricator.kde.org/D14063 AFFECTED FILES BUILD/kdeplasma-addons-5.12.6/runners/dictionary/dictionarymatchengine.cpp To: McPain, broulik Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D9003: Dict applet: implement config dialog for choosing a dictionary

This revision was automatically updated to reflect the committed changes. Closed by commit R114:bf39f91db8de: Dict applet: implement config dialog for choosing a dictionary (authored by dfaure). Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma

D8320: Dictionary Engine: fix synchronization issues.

This revision was automatically updated to reflect the committed changes. Closed by commit R114:02d8fbfb1928: Dictionary Engine: fix synchronization issues. (authored by dfaure). REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8320?vs=20832&id=2

D8320: Dictionary Engine: fix synchronization issues.

mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R114 Plasma Addons BRANCH Plasma/5.11 REVISION DETAIL https://phabricator.kde.org/D8320 To: dfaure, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts

D8320: Dictionary Engine: fix synchronization issues.

tool for waiting for something to happen, allowing us to unlock the mutex correctly in all cases. TEST PLAN Alt+F2 "define test". Doesn't seem to work very reliably yet, though. REPOSITORY R114 Plasma Addons BRANCH Plasma/5.11 REVISION DETAIL https://phabricator.kde.o

D5524: Use System Dictionary for Suggestions Model

This revision was automatically updated to reflect the committed changes. Closed by commit R846:9018da923006: Use System Dictionary for Suggestions Model (authored by Aiix). REPOSITORY R846 Mycroft Plasma integration CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5524?vs=13636&am

D5524: Use System Dictionary for Suggestions Model

Aiix added inline comments. INLINE COMMENTS > markg wrote in main.qml:66-67 > That's smart! > > But is the file on that location on every distribution (assuming it even has > the file). I did a manual look up of that folder and file in Fedora, Neon, Ubuntu, Debian and they seem to have this f

D5524: Use System Dictionary for Suggestions Model

markg added inline comments. INLINE COMMENTS > main.qml:66-67 > var baseLocation = > '/usr/share/plasma/plasmoids/org.kde.plasma.mycroftplasmoid/contents/ui/suggestion/'; > -var path = baseLocation + 'words1.txt'; > +var diclocation = '/usr/share/dict/' > +var pa

D5524: Use System Dictionary for Suggestions Model

Aiix created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Ref https://phabricator.kde.org/T5918 https://phabricator.kde.org/T5892 Removed Word List and moved Suggestions Model to use default dict pro

The widget Dictionary seems to have become useless.

I have Mandriva 2010.2 on one computer and have been using the widget Dictionary for years now and I enjoy it very much. I am not able to find anything to improve at all. I have now installed Mageia 3 on an other computer and the Dictionary widget is now useless. The former icon is gone, it

Fwd: The widget Dictionary seems to have become useless.

have Mandriva 2010.2 on one computer and have been using the widget Dictionary for years now and I enjoy it very much. I am not able to find anything to improve at all. I have now installed Mageia 3 on an other computer and the Dictionary widget is now useless. The former icon is gone, it takes

Re: Review Request: Dictionary Runner

. One somewhat important. runners/dictionary/dictionarymatchengine.h <http://git.reviewboard.kde.org/r/106068/#comment14389> *important* This is not a valid license header. Copy and paste the correct one from somewhere else. runners/dictionary/dictionarymatchengine.cpp

Re: Review Request: Dictionary Runner

rd.kde.org/r/106068/ > --- > > (Updated Aug. 18, 2012, 3:57 a.m.) > > > Review request for Plasma, Jason A. Donenfeld and Aaron J. Seigo. > > > Description > --- > > Jason's dictionary runner plu

Re: Review Request: Dictionary Runner

uest for Plasma, Jason A. Donenfeld and Aaron J. Seigo. > > > Description > --- > > Jason's dictionary runner plugin for krunner. > > > Diffs > - > > runners/CMakeLists.txt f487563c58b5bacc5e49b9e1a0f6e306956bbf7f > runners/dictionary/

Re: Review Request: Dictionary Runner

gt; Description > > Jason's dictionary runner plugin for krunner. > > Testing > > Works well under various different loads. > > Diffs > >- runners/CMakeLists.txt (f487563c58b5bacc5e49b9e1a0f6e306956bbf7f) >- runners/dictionary/CMakeLists.txt (PRE-CREATION

Re: Dictionary KRunner: Complete

Buler? Buler? ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Dictionary KRunner: Complete

will not be > wasted effort) Cool, all set: https://git.reviewboard.kde.org/r/106068/ https://projects.kde.org/projects/kde/kdeplasma-addons/repository/show/runners/dictionary?rev=zx2c4%2Fdictionary-runner ___ Plasma-devel mailing list Plasma-deve

Review Request: Dictionary Runner

. Description --- Jason's dictionary runner plugin for krunner. Diffs - runners/CMakeLists.txt f487563c58b5bacc5e49b9e1a0f6e306956bbf7f runners/dictionary/CMakeLists.txt PRE-CREATION runners/dictionary/Messages.sh PRE-CREATION runners/dictionary/dictionarymatchengine.h PRE-CRE

Re: Dictionary KRunner: Complete

On Saturday, August 18, 2012 01:21:46 Jason A. Donenfeld wrote: > It currently lives in kde-review [2], and I'd like to merge it into > the plasma-addons git repository for 4.10, with your permission. Would > someone look over it and let me know? please put it in a branch in the kdeplasma-addons r

Dictionary KRunner: Complete

Hey Everyone, Over two years ago I wrote the initial revision of the dictionary krunner, but it was never completed, due to threading limitations in AbstractRunner's coordination with DataEngine. I've reworked things, now, to work around this, and the result is exactly as intende

Re: KConfigXT: store dictionary

On Sunday, March 18, 2012 17:29:52 Simone Gaiarin wrote: > Can I use the same kcfg file for the fields of different ui files? yes, it's possible. you need to do the associations yourself from C++ however. (this is what kontact does btw..) -- Aaron Seigo signature.asc Description: This is a dig

KConfigXT: store dictionary

Hi, I need to store with KconfigXT a value associated to each activity, but since the number of activities is not know a priori I can't create an entry for each activity in kcfg file. So I want to store a dictionary (activity,value), is there a way to do this with KConfigXT or should I fallba

dictionary dataengine how to use

i'm trying to figure out how to use the dict dataengine. I tried source request "eng-deu:house", which should return back "haus" apparently...it returns nothing it doesn't work from the qml applet like notmart's code said it would ;-) and i can't figure out how to get it to work through the datae

Re: dictionary widget QML

On Mon, Feb 6, 2012 at 6:44 AM, Shaun Reich wrote: > bit of a snag...how can i create a model/listview from > plasmasource.data["list-dictionaries"] ? > > there's 1 source which has many keys and values, as you'll see by > requesting that source from plasmaengineexplorer. > > thought of a model of

Re: dictionary widget QML

bit of a snag...how can i create a model/listview from plasmasource.data["list-dictionaries"] ? there's 1 source which has many keys and values, as you'll see by requesting that source from plasmaengineexplorer. thought of a model of a model, but that's not possible ;-p got to be an easy way...

Re: dictionary widget QML

On Sun, Feb 5, 2012 at 4:47 PM, Shaun Reich wrote: >> anyways having 100% of the default applets in pure qml is a very long >> term plan, since on the desktop plasma2 would continue to use a legacy >> compatible version with qgraphicsview for a (probably) long time > > sure, but i'm trying to chan

Re: dictionary widget QML

> anyways having 100% of the default applets in pure qml is a very long > term plan, since on the desktop plasma2 would continue to use a legacy > compatible version with qgraphicsview for a (probably) long time sure, but i'm trying to change that 1% at a time ;-) -- Shaun Reich, KDE Software D

Re: dictionary widget QML

On Sun, Feb 5, 2012 at 3:49 PM, Shaun Reich wrote: > On Sun, Feb 5, 2012 at 8:58 AM, Marco Martin wrote: >> On Sun, Feb 5, 2012 at 2:46 PM, Aaron J. Seigo wrote: if you want to make this choseable by the kcm it would have to become a (partly) c++ plasmoid >>> >>> assuming that it is un

Re: dictionary widget QML

On Sun, Feb 5, 2012 at 8:58 AM, Marco Martin wrote: > On Sun, Feb 5, 2012 at 2:46 PM, Aaron J. Seigo wrote: >>> if you want to make this choseable by the kcm it would have to become >>> a (partly) c++ plasmoid >> >> assuming that it is unnecessary, even unwanted (different instances of the >> pla

Re: dictionary widget QML

On Sun, Feb 5, 2012 at 2:46 PM, Aaron J. Seigo wrote: >> if you want to make this choseable by the kcm it would have to become >> a (partly) c++ plasmoid > > assuming that it is unnecessary, even unwanted (different instances of the > plasmoid -> different dicts?), this could in theory be done str

Re: dictionary widget QML

On Sun, Feb 5, 2012 at 2:46 PM, Aaron J. Seigo wrote: > > can we do better in libplasma2? i hope so :) > apart from the fact that it requires either config forms using plasma widgets or complete and functional desktop qtcomponents, i think the solution is quite easy ;) (and makes easy also writin

Re: dictionary widget QML

pulated. > > > > so should the qml one have a populated listview that enables you to > > select which dicts you can have? > > as far i know dict.org provides more than one dictionary. the > dataengine supports query to different doctionaries than the default > one,

Re: dictionary widget QML

to > select which dicts you can have? > as far i know dict.org provides more than one dictionary. the dataengine supports query to different doctionaries than the default one, a gui has never been implemented for it. if you want to make this choseable by the kcm it would have to become

Re: dictionary widget QML

On Saturday, February 4, 2012 20:55:24 Shaun Reich wrote: > so should the qml one have a populated listview that enables you to > select which dicts you can have? if there is any value to that, sure. i suppose the primary purpose would be to allow one to define different dict servers? -- Aaron

dictionary widget QML

is the dict widget supposed to have a kcm? i could have sworn it had one, and on my 4.8 system it has one but the populated list view is...unpopulated. so should the qml one have a populated listview that enables you to select which dicts you can have? (p.s. i'm getting ready to merge it in, it'

Re: Dictionary

On Wednesday 28 December 2011, David Hart wrote: > Hi folks. > > Dictionary was one of the most useful applications in plasma world - at > least for me. It no longer works. I suspect that dict.org changed their > interface. works fine here. maybe the site was down for a while? b

Dictionary

Hi folks. Dictionary was one of the most useful applications in plasma world - at least for me. It no longer works. I suspect that dict.org changed their interface. -- David Hart South Beach http://www.slowlyboiledfrog.com ___ Plasma-devel mailing

Re: Review Request: port dictionary qml to Plasma Components

> On Nov. 17, 2011, 11:34 a.m., Marco Martin wrote: > > dictionary/package/contents/ui/main.qml, line 80 > > <http://git.reviewboard.kde.org/r/103143/diff/1/?file=41047#file41047line80> > > > > clearbuttonShown is available again in the textfield api, so it c

Re: Review Request: port dictionary qml to Plasma Components

> On Nov. 17, 2011, 11:34 a.m., Marco Martin wrote: > > dictionary/package/contents/ui/main.qml, line 80 > > <http://git.reviewboard.kde.org/r/103143/diff/1/?file=41047#file41047line80> > > > > clearbuttonShown is available again in the textfield api, so it c

Re: Review Request: port dictionary qml to Plasma Components

> On Nov. 17, 2011, 11:34 a.m., Marco Martin wrote: > > dictionary/package/contents/ui/main.qml, line 80 > > <http://git.reviewboard.kde.org/r/103143/diff/1/?file=41047#file41047line80> > > > > clearbuttonShown is available again in the textfield api, so it

Re: Review Request: port dictionary qml to Plasma Components

o the very small thing below dictionary/package/contents/ui/main.qml <http://git.reviewboard.kde.org/r/103143/#comment7061> clearbuttonShown is available again in the textfield api, so it can stay - Marco Martin On Nov. 15, 2011, 3:54 p.m., Giorgos Tsiapal

Review Request: port dictionary qml to Plasma Components

plasmoid to the plasma components and also add an archors property in order to be rendered properly. Diffs - dictionary/package/contents/ui/main.qml e9cb9fc Diff: http://git.reviewboard.kde.org/r/103143/diff/diff Testing --- the PlasmaWidget.WebView doesn't expand in all the e

Fwd: Krunner Dictionary/KDE 4.6

Aaron, Thought you might want to see this. Folks are hungry for that krunner (and thus for the reentrancy flag we talked about). Jason Forwarded conversation Subject: Krunner Dictionary/KDE 4.6 From: ** Date: Wed, Jan 26, 2011 at 20:28 To: ja...@zx2c4.com -BEGIN

Re: kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

On Sun, Aug 29, 2010 at 17:54, Jason A. Donenfeld wrote: > > I'm going to re-factor the dictionary runner to use this approach so that > the new setReentrant(true) switch will have something to test against, for > whoever ends up implementing that. >

Re: kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

ly as you describe (and then delete itself when finished). I'm going to re-factor the dictionary runner to use this approach so that the new setReentrant(true) switch will have something to test against, for whoever ends up implementing that. ___

Re: kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

* if the user types more than $THREAD_POOL_COUNT letters before the dict > > engine returns, the thread pool will be filled with dictionary runner > > threads and therefore be exhausted > > > > looking at it further, it's evident that this runner is really working > &

Re: kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

 /some point/ ? Or should we not rely on this. > > * if the user types more than $THREAD_POOL_COUNT letters before the dict > engine returns, the thread pool will be filled with dictionary runner threads > and therefore be exhausted > > looking at it further, it's evident

kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

hi ... looking at the dictionary runner, i've noticed a few things that should be improved: * when the teardown() signal is emitted, the runner should disconnect from the DataEngine but i've also come to a few concerning conclusions: * a thread can wait indefinitely on m_wait.wai

Re: Review Request: Don't show dictionary tooltip when not collapsed

rd.kde.org/r/3923/ > --- > > (Updated 2010-05-08 05:43:21) > > > Review request for Plasma. > > > Summary > --- > > Like the weather applet, the dictionary applet should not show a tooltip when > placed on the desktop. It's di

Review Request: Don't show dictionary tooltip when not collapsed

dictionary applet should not show a tooltip when placed on the desktop. It's distracting, particularly with the animation, and we already know it's a dictionary. :-) This addresses bug 232763. https://bugs.kde.org/show_bug.cgi?id=232763 Diffs - /trunk/KDE/kdeplasma-addons/ap

Re: Review Request: Fix 'list-dictionaries' output of dictionary data engine

> Review request for Plasma. > > > Summary > --- > > This patch fixes the parsing of the "SHOW DB" output in the dictionary data > engine. > > Otherwise the "list-dictionaries" output included: > - Status codes 250 and 110 > - Empty lines > -

Review Request: Fix 'list-dictionaries' output of dictionary data engine

"SHOW DB" output in the dictionary data engine. Otherwise the "list-dictionaries" output included: - Status codes 250 and 110 - Empty lines - Lines starting with a dot Also the key value of "list-dictionaries" was wrong. It always contained the first wo

Re: add dictionaries to dictionary plasmoid

There are some available on dict.org, but there is no configuration method in the applet/engine to make use of it now. 2009/1/9 Felix Michel > Dear Panel-Devel team > > > the dictionary plasmoid is just great. i have it in the panel and use it > often, because the pop up is sm

add dictionaries to dictionary plasmoid

Dear Panel-Devel team the dictionary plasmoid is just great. i have it in the panel and use it often, because the pop up is small, useful and not intrusive. Just some question: are there other dictionaries for other languages? French, german or spanish or bilingual dictionaries? That would