D18762: Add support for portals in KRun

GeeXT updated this revision to Diff 51017. GeeXT added a comment. The result should be negated REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=50972&id=51017 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https://phabricator.kde.or

D18762: Add support for portals in KRun

jgrulich added a comment. Looks good from the portal point of view, but I would rather wait for someone who knows KRun internally to approve this review. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18762 To: GeeXT, jgrulich, apol, dfaure, #frameworks Cc: kde-framewo

D18778: [KRun] Fall back to reading text/html in mimeapps.list if others aren't there

ngraham created this revision. ngraham added reviewers: dfaure, Frameworks. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ngraham requested review of this revision. REVISION SUMMARY If System Settings > Applications > Default Applications > Web Browser is

D18776: Make sure only directories are added to the inotify watcher

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Maybe add a comment above this line explaining the reasoning that you put in the patch's summary section. REPOSITORY R293 Baloo BRANCH master REVISION DETAIL https://phabricator.k

D18776: Make sure only directories are added to the inotify watcher

bruns created this revision. bruns added reviewers: Baloo, Frameworks, poboiko, ngraham. Herald added projects: Frameworks, Baloo. Herald added a subscriber: kde-frameworks-devel. bruns requested review of this revision. REVISION SUMMARY Although watchFolder is only called for directories, there

D18772: Mark KCodecs::uuencode does nothing

aacid created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. aacid requested review of this revision. REPOSITORY R270 KCodecs BRANCH master REVISION DETAIL https://phabricator.kde.org/D18772 AFFECTED FILES CMakeLists.txt src/kcodecs

D18604: Implement support for writing rating information for taglib writer

astippich added inline comments. INLINE COMMENTS > bruns wrote in taglibwriter.cpp:229 > but in case file.tag() returns an ApeTag, the dynamic_cast will return a > nullptr. This is a dynamic_cast, not a static_cast. It always returns a nullptr REPOSITORY R286 KFileMetaData REVISION DETAIL

D18604: Implement support for writing rating information for taglib writer

bruns added inline comments. INLINE COMMENTS > astippich wrote in taglibwriter.cpp:229 > No. Mp3 supports ID3v1, ID3v2 or Ape tags, so simply casting to ID3v2 is not > possible. but in case file.tag() returns an ApeTag, the dynamic_cast will return a nullptr. This is a dynamic_cast, not a stat

D18770: [KStatusNotifierItem] use fallback sizes when none is available

pino created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. pino requested review of this revision. REVISION SUMMARY When converting a QIcon to a vector of images to D-Bus, the list of available sizes is used to extract the pixmaps of the ic

D17245: Add string formatting function to property info

astippich added a comment. Can someone please give this a yes or a no? REPOSITORY R286 KFileMetaData REVISION DETAIL https://phabricator.kde.org/D17245 To: astippich, broulik, bruns, mgallien, #frameworks Cc: ngraham, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spo

D18738: Add ebook thumbnailer

elvisangelaccio added a comment. LGTM INLINE COMMENTS > CMakeLists.txt:265 > + > +install(TARGETS ebookthumbnail DESTINATION ${PLUGIN_INSTALL_DIR}) > + `PLUGIN_INSTALL_DIR` is deprecated, new code should use `KDE_INSTALL_PLUGINDIR` REPOSITORY R320 KIO Extras REVISION DETAIL https://pha

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

bcooksley added a comment. Your name has now been updated. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: bcooksley, fvogt, aacid, bruns, dkorth, ngraham, kde-frameworks-devel, michaelh

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

This revision was automatically updated to reflect the committed changes. Closed by commit R296:6258f33fce3a: [GridViewKCM] improve contrast and legibility for delegates' inline hover… (authored by ngraham). REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham updated this revision to Diff 50990. ngraham added a comment. Reverse the order of clicking and triggering, since it looks a bit nicer this way REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18649?vs=50982&id=50990 BRANCH improve-grid-view-

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

broulik added a comment. +1 I think this is a fine tradeoff between aesthetics and usability REPOSITORY R296 KDeclarative BRANCH improve-grid-view-kcm-inline-hover-button-contrast (branched from master) REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, bro

D18680: Fix ENABLE_CLAZY option with clazy >= 1.5

This revision was automatically updated to reflect the committed changes. Closed by commit R240:ba3341c83b18: Fix ENABLE_CLAZY option with clazy >= 1.5 (authored by elvisangelaccio). REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18680?vs=50790&id=

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ndavis accepted this revision. REPOSITORY R296 KDeclarative BRANCH improve-grid-view-kcm-inline-hover-button-contrast (branched from master) REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik, GB_2, ndavis Cc: Codezela, bruns, rizzitello, ndavis, andrea

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

GB_2 accepted this revision. GB_2 added a comment. This revision is now accepted and ready to land. Very nice! REPOSITORY R296 KDeclarative BRANCH improve-grid-view-kcm-inline-hover-button-contrast (branched from master) REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham,

D18768: Extend Open Document thumbnailer to Office Open XML

acrouthamel added a comment. Thanks for expanding the thumbnailer lately, awesome work with this and the eBook one. :) REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D18768 To: broulik, #plasma, #vdg, #frameworks Cc: acrouthamel, kde-frameworks-devel, kfm-devel,

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham edited the summary of this revision. ngraham edited the test plan for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik Cc: Codezela, bruns, rizzitello, ndavis, andreask, filipf, abetts, GB_2, kde-frame

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham updated this revision to Diff 50982. ngraham added a comment. Use theme-following version for various reasons REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18649?vs=50976&id=50982 BRANCH improve-grid-view-kcm-inline-hover-button-contrast (b

D18768: Extend Open Document thumbnailer to Office Open XML

broulik created this revision. broulik added reviewers: Plasma, VDG, Frameworks. Herald added projects: Dolphin, Frameworks. Herald added subscribers: kfm-devel, kde-frameworks-devel. broulik requested review of this revision. REVISION SUMMARY This tries to extract the `docProps/thumbnail.*` fro

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. Actually there is a real issue here. Disabled button icons are much less legible with the dark version. The enabled version of the delete button is less legible too (red-on-black vs red-on-gray) I think we need to go with the version that 100% follows the color schem

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

trmdi added a comment. In D16643#406076 , @bcooksley wrote: > Please update your name on Identity, then i'll sync it from there. Done. Thank you! REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To:

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

bcooksley added a comment. Please update your name on Identity, then i'll sync it from there. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: bcooksley, fvogt, aacid, bruns, dkorth, ngraham, kde-fra

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. Now that I think about it, Elisa also has a variant of this UI that would be much better if it adopted this visual style, and i think that one would look good with dark buttons too. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649

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

trmdi added a comment. Nice, my first patch finally has been accepted. Thanks everyone! Btw, can you @bcooksley help me to change my name in my profile? Tranter Madi REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma,

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

ngraham added a comment. Never mind, missed that this was for a Frameworks repo. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: fvogt, aacid, bruns, dkorth, ngraham, kde-frameworks-devel, michaelh

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

fvogt closed this revision. fvogt added a comment. Landed (phab didn't notice): https://cgit.kde.org/kdeclarative.git/commit/?id=856672f370fb32c7a3c3d1a13d873fb8b767d0e8 REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, he

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

bcooksley added a comment. Phabricator can take up to 30 minutes to become aware of commits depending on how active a repository normally is. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: bcooksl

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

ngraham added a comment. Hmm, should this go to the 5.15 branch too? REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: fvogt, aacid, bruns, dkorth, ngraham, kde-frameworks-devel, michaelh

D18604: Implement support for writing rating information for taglib writer

astippich added inline comments. INLINE COMMENTS > bruns wrote in taglibwriter.cpp:229 > Is this the same as > > auto id3Tags = dynamic_cast(file.tag()); > if (id3Tags) { ... } > > ? No. Mp3 supports ID3v1, ID3v2 or Ape tags, so simply casting to ID3v2 is not possible. REPOSITORY R286

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

fvogt accepted this revision. fvogt added a comment. @trmdi: Do you have push access? If not, which name should be used for the commit? REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns, fvogt Cc: fvogt, aacid

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

trmdi added a comment. In D16643#406051 , @fvogt wrote: > @trmdi: Do you have push access? If not, which name should be used for the commit? No, I don't. tr...@yandex.com / Tranter Madi REPOSITORY R296 KDeclarative REVISION DETAIL

D18601: Rewrite taglib writer to use property interface

astippich added inline comments. INLINE COMMENTS > bruns wrote in taglibwriter.cpp:70 > When you make this > `void writeGenericProperties(Taglib::File *file, const PropertyMap > &newProperties)`, you can do `file->properties(); {/* merge */}; > file->setProperties(...);` here, saving most of th

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

GB_2 added a comment. In D18649#406032 , @ngraham wrote: > Or we could just use frikkin' light buttons. :) I think dark buttons just draw too much attention and doesn't really fit. When using the colorscheme the contrast of the button ico

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

trmdi updated this revision to Diff 50979. REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16643?vs=47815&id=50979 REVISION DETAIL https://phabricator.kde.org/D16643 AFFECTED FILES src/qmlcontrols/draganddrop/DeclarativeDropArea.cpp To: trmdi, mart,

D17302: Add test for adding properties to result

astippich added inline comments. INLINE COMMENTS > bruns wrote in resulttest.cpp:63 > This looks wrong to me ... > How many items do you get when you append "keyword3" first and ["keyword1", > "keyword2"] next? It's the same. The properties will get merged regardless of their form if the QVari

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. Or we could just use frikkin' light buttons. :) REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik Cc: Codezela, bruns, rizzitello, ndavis, andreask, filipf, abetts, GB_2, kde-frameworks-devel, mic

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. Well, this view is used in a lot of different places, so it's not like only one thing does it this way now. But we could do it in Gwenview too, which has the same basic UI but uses light buttons. Also maybe we could think about adjusting Dolphin's selection mark

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

GB_2 added a comment. Well, I really don't like that this is the only thing that has dark buttons. It is inconsistent and just looks out of place. We should probably do some more discussion elsewhere if we want do this in other places too. REPOSITORY R296 KDeclarative REVISION DETAIL ht

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. The buttons are dynamic and are chosen by the individual KCM; what you propose would be out of scope even if it were possible. I think we're going to stay with this basic design. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

Codezela added a comment. another option we put the delete button only and align it to the corner of the thumbnail make it circle button and ditch the other one no need for it here what do u think REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 T

D18762: Add support for portals in KRun

ngraham added reviewers: apol, dfaure, Frameworks. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18762 To: GeeXT, jgrulich, apol, dfaure, #frameworks Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. In D18649#406019 , @Codezela wrote: > how about make it global under > beside add image button aligned left > tow buttons disablef by default No, that was the old design we moved away from. The new thing is i

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

Codezela added a comment. how about make it global under beside add image button aligned left tow buttons disablef by default REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik Cc: Codezela, bruns, rizzitello, ndavis, a

D18487: Reparse background contrast settings when colors changed

broulik added inline comments. INLINE COMMENTS > mart wrote in theme_p.cpp:754 > but it wouldn't be guaranteed to be the same of the value stored in the > config file no? If the theme does not supply background contrast values, they are auto-determined from the background color (dark or light)

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham added a comment. Yeah, the dark version has grown on me too. I'm going to go with that for now. The diff is smaller too. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik Cc: bruns, rizzitello, ndavis, andreask, f

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham updated this revision to Diff 50976. ngraham added a comment. Go with the dark version REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18649?vs=50879&id=50976 BRANCH improve-grid-view-kcm-inline-hover-button-contrast (branched from master) R

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

ngraham edited the summary of this revision. ngraham edited the test plan for this revision. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D18649 To: ngraham, #vdg, #plasma, broulik Cc: bruns, rizzitello, ndavis, andreask, filipf, abetts, GB_2, kde-frameworks-deve

Re: [sysadmin/ci-tooling] build-specs/Plasma: Disable execution of tests for plasma-integration.

On Mon, Feb 4, 2019 at 6:35 AM David Faure wrote: > > On vendredi 1 février 2019 10:32:19 CET Ben Cooksley wrote: > > On Fri, Feb 1, 2019 at 9:15 PM David Faure wrote: > > > I'll try to get around to debugging this, but I'm surprised by one thing: > > > doesn't the same issue happen in KIO's own

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

bruns added a comment. In D16643#405814 , @fvogt wrote: > @trmdi: Can you do the small change @bruns suggested? ^ Then it can be landed and everyone's happy :-) 👍 REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

bruns added a comment. The inline version looks a little bit more polished, while the other one looks literally "just put on top". The "button" version though somewhat resembles the "selection" buttons/emblems in dolphin. There is a very tiny issue with the button version, although the b

D18763: [client] Fix typos

zzag abandoned this revision. zzag added a comment. Well ok. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D18763 To: zzag, #kwin Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns

D18763: [client] Fix typos

apol added a comment. These are not typos. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D18763 To: zzag, #kwin Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

bruns added inline comments. INLINE COMMENTS > bruns wrote in kinotify.cpp:429 > This is broken now, as you fail to emit created for any child. Ignore, looked at the wrong line ... REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D18698 To: poboiko, #baloo, #frameworks, n

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

bruns added inline comments. INLINE COMMENTS > kinotify.cpp:429 > +if (!it.next().isEmpty()) { > +d->addWatch(it.filePath()); > +} This is broken now, as you fail to emit created for any child. REPOSITORY R293 Baloo REVISION DET

D18762: Add support for portals in KRun

GeeXT updated this revision to Diff 50972. GeeXT added a comment. Restored the accidentally deleted empty line... REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18762?vs=50968&id=50972 BRANCH flatpak_open_url (branched from master) REVISION DETAIL https:/

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

bruns added a comment. Addendum: I have just checked, https://phabricator.kde.org/source/baloo/browse/master/src/file/filewatch.cpp$144 filewatch relies on the code here to emit created events for all children, so we need the traversal code not only for "moved", but also for "created",

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.12/25/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 05 Feb 2019 17:02:37 + Build duration: 7 min 1 sec and counting JUnit Tests Name: projectroot

D18763: [client] Fix typos

zzag created this revision. zzag added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. zzag requested review of this revision. REPOSITORY R127 KWayland BRANCH fix-typos REVISION DETAIL https://phabricator.kde.org/D18763 AFFECTED FILES

D18761: Add icon name to service providers in .desktop file

This revision was automatically updated to reflect the committed changes. Closed by commit R241:8657a239a28d: Add icon name to service providers in .desktop file (authored by shubham). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18761?vs=50953&id=50969 REVISIO

D18762: Add support for portals in KRun

GeeXT created this revision. GeeXT added a reviewer: jgrulich. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. GeeXT requested review of this revision. REVISION SUMMARY Currently KRun in Flatpak/Snap doesn't support portals so it can't launch an application

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

poboiko updated this revision to Diff 50967. poboiko added a comment. Cosmetics REPOSITORY R293 Baloo CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18698?vs=50932&id=50967 BRANCH add-watch-moved (branched from master) REVISION DETAIL https://phabricator.kde.org/D18698 AFFE

D18761: Add icon name to service providers in .desktop file

ngraham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO BRANCH icon REVISION DETAIL https://phabricator.kde.org/D18761 To: shubham, broulik, ngraham, #vdg, #okular, #frameworks Cc: #okular, kde-frameworks-devel, Pixel_Lime, jraleigh, squeakypan

D18734: Update octave.xml to Octave 4.2.0

sirgienko added inline comments. INLINE COMMENTS > octave.xml:2745 > + String="end_try_catch" endRegion="Try" /> > > + endRegion="block" /> Could one word (`end`) be in several region? Because, in Octave, you could end `if` statement by `endif` or just `end`. In this va

D18734: Update octave.xml to Octave 4.2.0

sirgienko marked an inline comment as done. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D18734 To: sirgienko, #framework_syntax_highlighting Cc: dhaumann, kwrite-devel, kde-frameworks-devel, bmortimer, hase, michaelh, genethomas, ngraham, bruns, demsking,

D18734: Update octave.xml to Octave 4.2.0

sirgienko updated this revision to Diff 50963. sirgienko added a comment. Add requested changes REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18734?vs=50877&id=50963 BRANCH octave REVISION DETAIL https://phabricator.kde.org/D18734 AFFECT

D18487: Reparse background contrast settings when colors changed

mart added inline comments. INLINE COMMENTS > broulik wrote in theme_p.cpp:754 > An alternative approach would be to store those values and then do the > default fallback and `qGray(color(Plasma::Theme::BackgroundColor).rgb()) < > 127` stuff on the fly on demand. > It's runtime cpu cycles vs pa

D18734: Update octave.xml to Octave 4.2.0

sirgienko added a comment. @dhaumann, look on the `eps` (this word contained in command list) on this image. The word have wrong highlighting. F6589241: изображение.png I am testing my `octave.xml` file on v5.49.0 (current master need ecm 5.55 and I

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.12/6/ Project: kf5-qt5 SUSEQt5.12 Date of build: Tue, 05 Feb 2019 16:00:17 + Build duration: 12 min and counting BUILD ARTIFACTS abi-compatib

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

bruns added a comment. In D18698#405504 , @poboiko wrote: > > I am not sure if I understood your description correctly, but I am quite sure this race condition does not exist - the files/folders inside the moved folder are not created/moved one

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

bruns added a comment. In D18698#405636 , @poboiko wrote: > Added code to work with first entry that pops from FilteredDirIterator (that is the directory itself) > Test still works; but we should emit `created()` signal for it as well.

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20SUSEQt5.10/14/ Project: kf5-qt5 SUSEQt5.10 Date of build: Tue, 05 Feb 2019 16:00:17 + Build duration: 3 min 39 sec and counting BUILD ARTIFACTS abi-c

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/plasma-framework/job/kf5-qt5%20FreeBSDQt5.12/18/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 05 Feb 2019 16:00:16 + Build duration: 2 min 5 sec and counting JUnit Tests Name

D18545: [breeze desktop theme/dialogs] Add rounded corners to dialogs

This revision was automatically updated to reflect the committed changes. Closed by commit R242:dda4edcc6eb0: [breeze desktop theme/dialogs] Add rounded corners to dialogs (authored by rooty). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1

D18738: Add ebook thumbnailer

ngraham accepted this revision as: VDG. ngraham added a comment. VDG,approved, still needs a proper code review. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D18738 To: broulik, #plasma, #vdg Cc: abetts, ngraham, kde-frameworks-devel, kfm-devel, alexde, feverfew

D18545: [breeze desktop theme/dialogs] Add rounded corners to dialogs

rooty added a comment. Okay seeing as I'm seeing this corner problem with my icons as well, F6589150: image.png vs F6589155: image.png I think it's safe to land this. Also it's way past Saturday so :D REPOSITOR

D15910: Implement wl_surface::damage_buffer

davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH damageBuffer REVISION DETAIL https://phabricator.kde.org/D15910 To: romangg, #kwin, davidedmundson Cc: davidedmundson, zzag, kde-frameworks-devel, michaelh, ngraham, bru

D18738: Add ebook thumbnailer

ngraham added a comment. Quite lovely. With the latest changes, this now generates thumbnails for all my my ePubs that have a cover page defined. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D18738 To: broulik, #plasma, #vdg Cc: abetts, ngraham, kde-frameworks-

D15910: Implement wl_surface::damage_buffer

zzag added a comment. Is it good now? REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D15910 To: romangg, #kwin, davidedmundson Cc: davidedmundson, zzag, kde-frameworks-devel, michaelh, ngraham, bruns

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

fvogt added a comment. @trmdi: Can you do the small change @bruns suggested? ^ Then it can be landed and everyone's happy :-) REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, bruns Cc: fvogt, aacid, bruns, dkorth, n

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

bruns added a comment. In D16643#405468 , @fvogt wrote: > > The second change fvogt mentioned is IMHO handled quite strangely here - !m_enabled || m_temporaryInhibition is the inverse of !m_enabled || m_temporaryInhibition, so doing a event->ign

D18649: [GridViewKCM] improve contrast and legibility for delegates' inline hover buttons

GB_2 added a comment. In D18649#405368 , @ngraham wrote: > One thing I will not do in this patch is synthesize a fake button that has different behaviors from the standard button. If there's a problem with our buttons we need to fix it everywher

D18761: Add icon name to service providers in .desktop file

shubham added a reviewer: Frameworks. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18761 To: shubham, broulik, ngraham, #vdg, #okular, #frameworks Cc: #okular, kde-frameworks-devel, Pixel_Lime, jraleigh, squeakypancakes, alexde, IohannesPetros, GB_2, rooty, trickyricky26

D18761: Add icon name to service providers in .desktop file

shubham added a comment. In the rest of service providers, very few have any icon specified, and most don't have. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18761 To: shubham, broulik, ngraham, #vdg, #okular Cc: #okular, kde-frameworks-devel, Pixel_Lime, jraleigh,

D18730: Use breeze icons for search providers

shubham abandoned this revision. shubham added a comment. Abandoned in favour of D18761 REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D18730 To: shubham, ngraham, #vdg, #frameworks, cfeck Cc: broulik, #vdg, okular-devel, Pixel_Lim

D18761: Add icon name to service providers in .desktop file

shubham added reviewers: VDG, Okular. shubham added projects: Okular, VDG. shubham added a subscriber: Okular. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18761 To: shubham, broulik, ngraham, #vdg, #okular Cc: #okular, kde-frameworks-devel, Pixel_Lime, jraleigh, squeakyp

D18761: Add icon name to service providers in .desktop file

shubham created this revision. shubham added reviewers: broulik, ngraham. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. shubham requested review of this revision. REVISION SUMMARY Earlier all entries had same icon ie. "text-html" See also D18755

D18163: Set the color scheme to Printing for Print Preview

dhaumann edited the summary of this revision. 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

This revision was automatically updated to reflect the committed changes. Closed by commit R39:28e59740d070: Set the color scheme to Printing for Print Preview (authored by ahmadsamir, committed by dhaumann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1

D18116: Add TODO KF6 note

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:574b08d08111: Add TODO KF6 note (authored by dhaumann). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https:

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

BUILD UNSTABLE Build URL https://build.kde.org/job/Frameworks/job/kio/job/kf5-qt5%20FreeBSDQt5.12/24/ Project: kf5-qt5 FreeBSDQt5.12 Date of build: Tue, 05 Feb 2019 12:01:59 + Build duration: 7 min 32 sec and counting JUnit Tests Name: projectroo

D18755: Read IKWS search provider icon from desktop file

This revision was automatically updated to reflect the committed changes. Closed by commit R241:e1c386f04303: Read IKWS search provider icon from desktop file (authored by broulik). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18755?vs=50934&id=50938 REVISION D

D18755: Read IKWS search provider icon from desktop file

shubham accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D18755 To: broulik, #frameworks, shubham Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18755: Read IKWS search provider icon from desktop file

broulik created this revision. broulik added reviewers: Frameworks, shubham. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. broulik requested review of this revision. REVISION SUMMARY Allows to give them more meaningful icons rather than the one derived from

D18730: Use breeze icons for search providers

broulik added a comment. Check out D18730 Then you can put `Icon=foo` in the respective search provider desktop file REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D18730 To: shubham, ngraham, #vdg, #frameworks, cfeck Cc: brou

D18730: Use breeze icons for search providers

shubham added a comment. @broulik Adding icons to the .desktop files does not work, I tried. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D18730 To: shubham, ngraham, #vdg, #frameworks, cfeck Cc: broulik, #vdg, okular-devel, Pixel_Lime, jraleigh, squeakypancakes, al

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

poboiko marked an inline comment as done. REPOSITORY R293 Baloo REVISION DETAIL https://phabricator.kde.org/D18698 To: poboiko, #baloo, #frameworks, ngraham, bruns Cc: bruns, ngraham, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, abrahams

D18698: [baloo/KInotify] Notify if folder was moved from unwatched place

poboiko updated this revision to Diff 50932. poboiko added a comment. Added code to work with first entry that pops from FilteredDirIterator (that is the directory itself) Test still works; but we should emit `created()` signal for it as well. REPOSITORY R293 Baloo CHANGES SINCE LAST UPD

  1   2   >