D23708: [CopyJob] Fix crash when copying an already existing dir and pressing "Skip"

ahmadsamir marked an inline comment as done. ahmadsamir added a comment. Please commit it. Thanks. REPOSITORY R241 KIO BRANCH ahmad/copyjob (branched from master) REVISION DETAIL https://phabricator.kde.org/D23708 To: ahmadsamir, #frameworks, dfaure Cc: kde-frameworks-devel, LeGa

D23708: [CopyJob] Fix crash when copying an already existing dir and pressing "Skip"

ahmadsamir marked an inline comment as done. ahmadsamir added a comment. In D23708#525848 , @dfaure wrote: > Thanks for the fix! > > Do you feel like writing a unittest for it? > JobTest::chmodFileError shows how to use PredefinedAnswerJobU

D23708: [CopyJob] Fix crash when copying an already existing dir and pressing "Skip"

ahmadsamir updated this revision to Diff 65486. ahmadsamir retitled this revision from "[CopyJob] Fix crash when copying all files is skipped for an already existing dir" to "[CopyJob] Fix crash when copying an already existing dir and pressing "Skip"". ahmadsamir edited the summary of this revis

D23708: [CopyJob] Fix crash when copying all files is skipped for an already existing dir

ahmadsamir created this revision. ahmadsamir added reviewers: Frameworks, dfaure. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY In copyNextFile() if all files have been skipped QList::erase() will return end() iterator, accessing the element

D23457: Port regex search to QRegularExpression

ahmadsamir added a comment. In D23457#519773 , @cullmann wrote: > In D23457#519217 , @ahmadsamir wrote: > > > Maybe they'll also see it as ktexteditor/kate using a regex engine that matches what the

D23472: Mimic QInputControl::isAcceptableInput() when filtering typed characters

ahmadsamir updated this revision to Diff 64709. ahmadsamir added a comment. Improve comment in header file REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D23472?vs=64708&id=64709 BRANCH ahmad/soft-hyphen (branched from master) REVISION DETAIL https:

D23472: Mimic QInputControl::isAcceptableInput() when filtering typed characters

ahmadsamir updated this revision to Diff 64708. ahmadsamir retitled this revision from "Enable typing soft-hyphen characters" to "Mimic QInputControl::isAcceptableInput() when filtering typed characters". ahmadsamir edited the summary of this revision. ahmadsamir edited the test plan for this revi

D23472: Enable typing soft-hyphen characters

ahmadsamir added a comment. Digging around in git log I found: https://git.reviewboard.kde.org/r/127026/ But I think that's covered by the method in QInputControl, and ktexteditor has a unit test for that; I'll see how it goes. REPOSITORY R39 KTextEditor REVISION DETAIL https://phab

D23472: Enable typing soft-hyphen characters

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, dhaumann, cullmann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY The typeChars() functions filters out non-printable characters, except for: \t, \n, \r; add QCh

D23457: Port regex search to QRegularExpression

ahmadsamir added a comment. Maybe they'll also see it as ktexteditor/kate using a regex engine that matches what the abundance of online pcre docs say, and how other editors that use pcre behave? IIUC, '\s' was workedaround so as not to match a newline so that the search pattern wouldn'

D23457: Port regex search to QRegularExpression

ahmadsamir added a comment. In D23457#519205 , @cullmann wrote: > Hi, without any further look at the code changes, I don't think an behavior change like "\s can match a newline" is a good idea. > Or do I misunderstand that? It's exact

D23457: Port regex search to QRegularExpression

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, dhaumann, cullmann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY - Do away with the kateregexp class; move isMultiLine() to kateregexpsearch - \s can match a ne

D22477: With auto completion don't show completions that don't match from beginning of typed word

ahmadsamir marked 2 inline comments as done. ahmadsamir added a comment. Ping. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D22477 To: ahmadsamir, #ktexteditor, cullmann, dhaumann, #kdevelop, kossebau, mwolff, kfunk Cc: brauch, kwrite-devel, kde-frameworks-deve

D22500: Make keyword completion model return HideListIfAutomaticInvocation by default

ahmadsamir added a comment. FWIW, the combined effect of this patch and D22477 , hopefully you'll no longer see the Q_RETRUN_ARG and co. completions unless the menu is manually invoked with the mode set to C++. REPOSITORY R39 KTextEditor REVISION DETAI

D22500: Make keyword completion model return HideListIfAutomaticInvocation by default

ahmadsamir updated this revision to Diff 61932. ahmadsamir edited the test plan for this revision. ahmadsamir added a comment. Fix test plan :/ REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22500?vs=61880&id=61932 BRANCH ahmad/keyword-auto-completion

D22500: Make keyword completion model return HideListIfAutomaticInvocation by default

ahmadsamir added a comment. I messed up the test plan; try with the mode set to Python. It looks like a corner case where if there's only one exact match, the completions menu doesn't get auto hidden. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D22500 To: ahm

D22477: With auto completion don't show completions that don't match from beginning of typed word

ahmadsamir added a comment. In D22477#496214 , @brauch wrote: > I'm not sure if this solves the right problem. > > Where I notice this issue a lot is when typing "return". The keyword completion suggests "return", and when I want a newline, I

D22477: With auto completion don't show completions that don't match from beginning of typed word

ahmadsamir updated this revision to Diff 61882. ahmadsamir edited the summary of this revision. ahmadsamir added a comment. Take 2 REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22477?vs=61881&id=61882 BRANCH less-trigger-happy-completions (branched f

D22477: With auto completion don't show completions that don't match from beginning of typed word

ahmadsamir updated this revision to Diff 61881. ahmadsamir retitled this revision from "Add a config to only show completions matching word beginning" to "With auto completion don't show completions that don't match from beginning of typed word". ahmadsamir edited the summary of this revision. ah

D22500: Make keyword completion model return HideListIfAutomaticInvocation by default

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, KDevelop, cullmann, dhaumann, brauch. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY Change KateKeywordCompletionModel::matchingItem to return HideListIfAutomatic

D22477: Add a config to only show completions matching word beginning

ahmadsamir updated this revision to Diff 61879. ahmadsamir added a comment. Change the diff altogether to not show the menu with "ContainsMatch" REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22477?vs=61807&id=61879 BRANCH less-trigger-happy-completio

D22477: Add a config to only show completions matching word beginning

ahmadsamir added a comment. In D22477#496059 , @kossebau wrote: > Default is `false,` so does not change behaviour unless someone toggles the switch, right? > > I would rename the option to `WordCompletionMatchFromWordStartOnly` though, as f

D22477: Add a config to only show completions matching word beginning

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, cullmann, dhaumann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY This disables the current behaviour where if no match is found, completions that "contain" the

D22276: Add an action to insert a non-indented newline

ahmadsamir added a comment. If this is OK, please commit it too. Thanks. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D22276 To: ahmadsamir, #ktexteditor, cullmann, dhaumann Cc: bruns, mickaelbo, kde-frameworks-devel, kwrite-devel, LeGast00n, domson, micha

D22276: Add an action to insert a non-indented newline

ahmadsamir updated this revision to Diff 61373. ahmadsamir added a comment. Fix action name, don't use camel case, use _ Fix comment indentation :) REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22276?vs=61363&id=61373 BRANCH regular-newline (branch

D22276: Add an action to insert a non-indented newline

ahmadsamir updated this revision to Diff 61363. ahmadsamir added a comment. Use an Enum instead of bool REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22276?vs=61156&id=61363 BRANCH regular-newline (branched from master) REVISION DETAIL https://pha

D22315: Add setting to enable/disable text drag-and-drop (on by default)

ahmadsamir added a comment. IMHO, "Enable/Disable" is already implied by the checkbox :) REPOSITORY R39 KTextEditor BRANCH text_drag_and_drop_setting REVISION DETAIL https://phabricator.kde.org/D22315 To: thomassc, #ktexteditor, cullmann Cc: ahmadsamir, dhaumann, cullmann, kwrite-deve

D22276: Add an action to insert a non-indented newline

ahmadsamir added inline comments. INLINE COMMENTS > dhaumann wrote in katedocument.h:827 > I dislike the double negation: noIndentation = false. Later even > !noIndentation. This is bad API design. > > Please change to bool indent = true. Noted. Although I'll use an enum per cullmann's recomme

D22276: Add an action to insert a non-indented newline

ahmadsamir added a comment. Ctrl+Enter/Return may be a too common shortcut, i.e. users probably have it already mapped to do something, so maybe we shouldn't set a default shortcut for that action... REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D22276 To: ahma

D22276: Add an action to insert a non-indented newline

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, cullmann, dhaumann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY Currently pressing Enter to insert a new line, KTextEditor will, in most cases, indent the new

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir added a comment. Great :). Please commit it too (as I don't have access to kde git). REPOSITORY R39 KTextEditor BRANCH document-reloaded-spellcheck (branched from master) REVISION DETAIL https://phabricator.kde.org/D21940 To: ahmadsamir, #ktexteditor, cullmann, dhaumann Cc:

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir marked 5 inline comments as done. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D21940 To: ahmadsamir, #ktexteditor, cullmann, dhaumann Cc: sars, kde-frameworks-devel, kwrite-devel, LeGast00n, domson, michaelh, ngraham, bruns, demsking, cullmann, dhaumann

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir updated this revision to Diff 60467. ahmadsamir removed a subscriber: dhaumann. ahmadsamir added a comment. Use a lambda instead of adding a standalone method REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21940?vs=60154&id=60467 BRANCH doc

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir added a comment. Qt docs say that the signal must have at least as many arguments as the slot, and there is an implicit conversion between the types of the corresponding arguments in the signal and the slot. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.o

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir added inline comments. INLINE COMMENTS > dhaumann wrote in ontheflycheck.cpp:75 > But refreshSpellCheck() has a default parameter, so this function exists > without any parameter. Afaik this should work?! In file included from /usr/include/qt5/QtCore/qalgorithms.h:43,

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir added inline comments. INLINE COMMENTS > dhaumann wrote in ontheflycheck.cpp:75 > Directly connect to this, refreshSpellCheck. That wouldn't work, reloaded() takes a Document pointer parameter whereas refreshSpellcheck takes a Range parameter. REPOSITORY R39 KTextEditor REVISION

D21940: Make automatic spellcheck work after reloading a document

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, cullmann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY When a document gets reloaded for whatever reason, automatic spellcheck should be refreshed so as to re-p

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. In D18793#437220 , @cullmann wrote: > Hi, I still don't like that we do different things in replaceText depending on the selection of the potential activeView, that makes this function harder to use correctly, as it m

D18793: Handle text completion with block selection mode

ahmadsamir added a subscriber: loh.tar. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18793 To: ahmadsamir, #ktexteditor, cullmann, dhaumann, mwolff Cc: loh.tar, kde-frameworks-devel, kwrite-devel, #ktexteditor, gennad, domson, michaelh, ngraham, bruns, demsking, c

D18793: Handle text completion with block selection mode

ahmadsamir updated this revision to Diff 54150. ahmadsamir edited the summary of this revision. ahmadsamir edited the test plan for this revision. ahmadsamir removed a reviewer: KDevelop. ahmadsamir removed subscribers: loh.tar, mwolff. ahmadsamir added a comment. Verbatim REPOSITORY R39 KTe

D18793: Handle text completion with block selection mode

ahmadsamir updated this revision to Diff 54149. ahmadsamir added a comment. - Split the code responsible for inserting text in block selection from typeChars() to a new function - Don't add words where the block selection cursor is inside to the possible completion matches from the document

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. In D18793#424628 , @cullmann wrote: > I don't like the change to call typeChars. > > 1. that needs a valid view, activeView() might be null. > 2. replaceText is not interactive per-default, but typeChars is. >

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. Do we still need a unit test for this? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18793 To: ahmadsamir, #ktexteditor, cullmann, dhaumann, #kdevelop, mwolff Cc: loh.tar, mwolff, kde-frameworks-devel, kwrite-devel, #ktexteditor, doms

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. In D18793#420599 , @loh.tar wrote: > Fix this patch also Bug 382213 ? No, it doesn't fix it. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18793 To: ahmadsamir, #ktexteditor, cul

D19161: Use QTextFormat::TextUnderlineStyle instead of QTextFormat::FontUnderline

ahmadsamir added a comment. > ! In D19161#415702 , @dhaumann wrote: > Btw, searching in lxr.kde.org for FontUnderline reveals some more hits: https://lxr.kde.org/ident?_i=FontUnderline&_remember=1 Turns out these are false positives; I checke

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. I changed the diff to make replaceText handle inserting the word completion text. Is this a new behaviour? it exactly matches what users expect when they type something in block selection mode. Also why would this be opt-in? if you type something in block

D18793: Handle text completion with block selection mode

ahmadsamir updated this revision to Diff 52658. ahmadsamir added a comment. Change replaceText to handle inserting word completion with block selection mode REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18793?vs=51051&id=52658 BRANCH block-selection

D19161: Use QTextFormat::TextUnderlineStyle instead of QTextFormat::FontUnderline

ahmadsamir added a comment. In D19161#415701 , @dhaumann wrote: > This patch looks good to me, even though I cannot reproduce the issue following the steps in https://bugs.kde.org/show_bug.cgi?id=399278. Another way to test, try setting t

D19161: Use QTextFormat::TextUnderlineStyle instead of QTextFormat::FontUnderline

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, cullmann, dhaumann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY QTextFormat::FontUnderline has been deprecated upstream for a long time. It seems that for s

D18793: Handle text completion with block selection mode

ahmadsamir added a comment. Ping? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18793 To: ahmadsamir, #ktexteditor, cullmann, dhaumann, #kdevelop Cc: kde-frameworks-devel, kwrite-devel, #ktexteditor, gennad, michaelh, ngraham, bruns, demsking, cullmann, sars, d

D18907: Handle the case if createSpeller is passed an unavailable language

ahmadsamir added a comment. I don't have commit access, so please commit the diff. Thanks. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D18907 To: ahmadsamir, sandsmark, cullmann, dhaumann Cc: dhaumann, cullmann, loh.tar, kde-frameworks-devel, michaelh, ngraham

D18907: Handle the case if createSpeller is passed an unavailable language

ahmadsamir updated this revision to Diff 51578. ahmadsamir added a comment. Avoid shorthand in api docs REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18907?vs=51552&id=51578 REVISION DETAIL https://phabricator.kde.org/D18907 AFFECTED FILES src/core/lo

D18907: Handle the case if createSpeller is passed an unavailable language

ahmadsamir updated this revision to Diff 51552. ahmadsamir added a comment. Improve api docs. Use constFind() with languageClients, as that's more efficient than searching two separate data structures. REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18907?

D18907: Handle the case if createSpeller is passed an unavailable language

ahmadsamir added a comment. In D18907#410973 , @cullmann wrote: > I think the const at the end for the signal is not wanted, otherwise, this seems to make sense for me. The **this** pointer of Loader::createSpeller is const, so the signal

D18317: Display an error message if loading a dictionary fails

ahmadsamir added a comment. I've split the changes to the Loader in https://phabricator.kde.org/D18907. And will hopefully try to handle the Ui part in this current diff. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D18317 To: ahmadsamir, sandsmark, loh.tar Cc:

D18907: Handle the case if createSpeller is passed an unavailable language

ahmadsamir created this revision. ahmadsamir added reviewers: sandsmark, cullmann, dhaumann. ahmadsamir added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY In Loader::createSpeller, retun early if lang isn't one of the available languages(); otherwise a

D18793: Handle text completion with block selection mode

ahmadsamir created this revision. ahmadsamir added reviewers: KTextEditor, cullmann, dhaumann. Herald added projects: Kate, Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY Handle text completion with block selection mode BUG: 359763 FIXED-IN: 5.56 TEST PLAN W

D18163: Set the color scheme to Printing for Print Preview

ahmadsamir marked 2 inline comments as done. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18163 To: ahmadsamir, cullmann, #ktexteditor, dhaumann, mwolff Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D18163: Set the color scheme to Printing for Print Preview

ahmadsamir updated this revision to Diff 50588. ahmadsamir retitled this revision from "Set the color scheme to Printing when print preview'ing" to "Set the color scheme to Printing for Print Preview". ahmadsamir added a comment. Use a setter REPOSITORY R39 KTextEditor CHANGES SINCE LAST U

D18163: Set the color scheme to Printing when print preview'ing

ahmadsamir updated this revision to Diff 50503. ahmadsamir retitled this revision from "KateRenderer: when printing initially set the color scheme to Printing" to "Set the color scheme to Printing when print preview'ing". ahmadsamir edited the summary of this revision. ahmadsamir edited the test

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir added a comment. In D18163#398206 , @dhaumann wrote: > Is KateRenderer::setPrinterFriendly() only called in print preview mode? I think not. > > What is the bahavior currently when you print a page with another schema, and then pr

D18317: Display an error message if loading a dictionary fails

ahmadsamir added a reviewer: loh.tar. ahmadsamir added a subscriber: pino. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D18317 To: ahmadsamir, sandsmark, loh.tar Cc: pino, kde-frameworks-devel, michaelh, ngraham, bruns

D18317: Display an error message if loading a dictionary fails

ahmadsamir updated this revision to Diff 50486. ahmadsamir retitled this revision from "Don't fail if defaultLanguage dictionary can't be loaded" to "Display an error message if loading a dictionary fails". ahmadsamir edited the summary of this revision. ahmadsamir edited the test plan for this r

D18089: KLauncher: handle processes exiting without error

ahmadsamir added a comment. Ping. REPOSITORY R303 KInit REVISION DETAIL https://phabricator.kde.org/D18089 To: ahmadsamir, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir added a comment. A gentle ping. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18163 To: ahmadsamir, cullmann, #ktexteditor, dhaumann, mwolff Cc: mwolff, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dha

D18317: Don't fail if defaultLanguage dictionary can't be loaded

ahmadsamir added a comment. In D18317#396483 , @loh.tar wrote: > Well, I'm not a Sonnet Guru, more a normal user. Me too; your views are appreciated all the same. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D18

D18317: Don't fail if defaultLanguage dictionary can't be loaded

ahmadsamir added a comment. In D18317#395928 , @loh.tar wrote: > Just my thoughts: > > - I think there shouldn't be the (default) dictionary changed by some smart logic. Just hint the user that the setting is not applicable. The user

D18317: Don't fail if defaultLanguage dictionary can't be loaded

ahmadsamir added a comment. @loh.tar: I'll think that over, thanks for the pointers :) REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D18317 To: ahmadsamir, sandsmark, loh.tar Cc: pino, kde-frameworks-devel, michaelh, ngraham, bruns

D18317: Don't fail if defaultLanguage dictionary can't be loaded

ahmadsamir added a comment. In D18317#395530 , @pino wrote: > In D18317#395513 , @pino wrote: > > > This makes a "core" library grow a dependency on widgets -- not really a good idea, considering the

D18317: Don't fail if defaultLanguage dictionary can't be loaded

ahmadsamir created this revision. ahmadsamir added a reviewer: sandsmark. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY If the files of the the dictionary set in defaultLanguage= can't be loaded, instead of failing silently, try to load the

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir added a comment. @dhaumann: in the screenshot that's "file -> print"; the issue here is with "file -> print preview", where the extra options tabs (including "Layout") aren't added by ktexteditor. :) REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D1816

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir added reviewers: KTextEditor, dhaumann. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18163 To: ahmadsamir, cullmann, #ktexteditor, dhaumann Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir added a reviewer: cullmann. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18163 To: ahmadsamir, cullmann Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D18163: KateRenderer: when printing initially set the color scheme to Printing

ahmadsamir created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. ahmadsamir requested review of this revision. REVISION SUMMARY Otherwise the print preview dialog somehow gets the text color from the current default color

D18089: KLauncher: handle processes exiting without error

ahmadsamir added a comment. I don't have commit access, so please commit the diff. Thanks. REPOSITORY R303 KInit REVISION DETAIL https://phabricator.kde.org/D18089 To: ahmadsamir, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18089: KLauncher: handle processes exiting without error

ahmadsamir marked an inline comment as done. ahmadsamir added inline comments. INLINE COMMENTS > dfaure wrote in klauncher.cpp:340 > To make this less scary (since it's not an error) I would suggest > > << "exited without error, request done. status=" << Good point. Done. REPOSITORY R303 K

D18089: KLauncher: handle processes exiting without error

ahmadsamir marked 2 inline comments as done. REPOSITORY R303 KInit REVISION DETAIL https://phabricator.kde.org/D18089 To: ahmadsamir, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18089: KLauncher: handle processes exiting without error

ahmadsamir updated this revision to Diff 49063. ahmadsamir retitled this revision from "KLauncher: handle process dying with exitStatus 0" to "KLauncher: handle processes exiting without error". ahmadsamir edited the summary of this revision. ahmadsamir added a comment. Make commit and debug m

D18089: KLauncher: handle process dying with exitStatus 0

ahmadsamir updated this revision to Diff 48948. ahmadsamir edited the summary of this revision. ahmadsamir added a comment. Fix commit message REPOSITORY R303 KInit CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18089?vs=48947&id=48948 BRANCH klauncher-kateSessionManager (branc

D18089: KLauncher: handle process dying with exitStatus 0

ahmadsamir added a reviewer: dfaure. REPOSITORY R303 KInit REVISION DETAIL https://phabricator.kde.org/D18089 To: ahmadsamir, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18089: KLauncher: handle process dying with exitStatus 0

ahmadsamir created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ahmadsamir requested review of this revision. REVISION SUMMARY When a process dies with exitStatus 0, it could well be the app exited cleanly, and there's no need to display a

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir updated this revision to Diff 48336. ahmadsamir added a comment. Add doxygen doc for the new method including @since REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17629?vs=48218&id=48336 BRANCH addWordToSession (branched from master) REVISION

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir updated this revision to Diff 48218. ahmadsamir edited the summary of this revision. ahmadsamir added a comment. arc diff verbatim REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17629?vs=48217&id=48218 BRANCH addWordToSession REVISION DETAIL

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir updated this revision to Diff 48217. ahmadsamir added a comment. Fix commit message REPOSITORY R246 Sonnet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17629?vs=47683&id=48217 BRANCH addWordToSession REVISION DETAIL https://phabricator.kde.org/D17629 AFFECTED F

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir added a reviewer: sandsmark. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D17629 To: ahmadsamir, davidedmundson, cullmann, sandsmark Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17630: Don't re-mark words added/ignored to the dictionary as misspelled

ahmadsamir added reviewers: cullmann, davidedmundson. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D17630 To: ahmadsamir, cullmann, davidedmundson Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir added a dependent revision: D17630: Don't re-mark words added/ignored to the dictionary as misspelled. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D17629 To: ahmadsamir, davidedmundson, cullmann Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17630: Don't re-mark words added/ignored to the dictionary as misspelled

ahmadsamir created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. ahmadsamir requested review of this revision. REVISION SUMMARY When a word is added to the dictionary or ignored via the spellingMenu, the BackgroundChecker

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir added reviewers: davidedmundson, cullmann. REPOSITORY R246 Sonnet REVISION DETAIL https://phabricator.kde.org/D17629 To: ahmadsamir, davidedmundson, cullmann Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17629: Add method to BackgroundChecker to add word to session

ahmadsamir created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ahmadsamir requested review of this revision. REVISION SUMMARY Add convience method to add a word to BackgroundChecker the session of the currentDict speller. REPOSITORY R2

<    5   6   7   8   9   10