mart added a reviewer: Plasma.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D7085
To: davidedmundson, #plasma
Cc: #frameworks
mart updated this revision to Diff 17703.
mart added a comment.
ignore resizes to empty sizes
on multiscreen, it happens that notification windows areresized to 0,0
by somebody else (not dialog code, perhaps kwin, perhaps notification applet
this makes the notification main item los
mart added a comment.
looking into it, i still think the dialog code is correct.
what happens is that someone external resizes the notification window to 0x0
pixels wide right after the componentcomplete event. if as in this patch the
dialog is resized explicitly in componentcomplete, that
mart created this revision.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
on multiscreen systems, the notification dialog gets resized to 0x0
right after componentcomplete by someone else (either
mart updated this revision to Diff 17706.
mart added a comment.
remove unrelated changes
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D6964?vs=17703&id=17706
BRANCH
arcpatch-D6964
REVISION DETAIL
https://phabricator.kde.org/D6964
broulik added inline comments.
INLINE COMMENTS
> dialog.cpp:1045
>
> +if (re->size().isEmpty() || re->size() != re->oldSize()) {
> +return;
So it will never update its size? Shouldn't it be size == oldSize?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://ph
matank added a comment.
I can confrim that mart's patch ( https://phabricator.kde.org/D7127) solves
the issue.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D7071
To: matank, #plasma, davidedmundson
Cc: mart, plasma-devel, davidedmundson, ltoscan
kfunk added inline comments.
INLINE COMMENTS
> ktimecombobox.cpp:33
> +public:
> +KTimeValidator(QString timeFormat, QObject *parent = Q_NULLPTR);
> +KTimeValidator(QString timeFormat, QTime min, QTime max, QObject *parent
> = Q_NULLPTR);
Here and below: `nullptr` can be used directly i
gregormi created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REPOSITORY
R304 KNewStuff
BRANCH
mydev
REVISION DETAIL
https://phabricator.kde.org/D7130
AFFECTED FILES
autotests/kmoretools/kmoretoolstest.cpp
da
ltoscano requested changes to this revision.
ltoscano added a comment.
This revision now requires changes to proceed.
I think there is more than what you want: the translations of desktop are
handled automatically by scripty.
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.
gregormi retitled this revision from "Update and fix desktopfiles" to
"KMoreTools: Update and fix desktopfiles".
gregormi edited the summary of this revision.
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D7130
To: gregormi, ltoscano
Cc: ltoscano, #frameworks
mart marked an inline comment as done.
mart added inline comments.
INLINE COMMENTS
> broulik wrote in dialog.cpp:1045
> So it will never update its size? Shouldn't it be size == oldSize?
ouch, sure, sorry :)
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde
gregormi added a comment.
In https://phabricator.kde.org/D7130#132397, @ltoscano wrote:
> I think there is more than what you want: the translations of desktop are
handled automatically by scripty.
Thanks for the hint. I copied the desktop files from the system installation.
I kn
mart updated this revision to Diff 17712.
mart added a comment.
- fix check
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7127?vs=17704&id=17712
BRANCH
arcpatch-D7127
REVISION DETAIL
https://phabricator.kde.org/D7127
AFFECTED FILE
ltoscano added a comment.
Just update the English messages.
More generally, why do we need copies of the system desktop files (even the
ones developed by KDE, org.kde.filelight.desktop?) Isn't there a way to use the
existing desktop files if installed? We don't want to duplicate what exists
gregormi added a comment.
In https://phabricator.kde.org/D7130#132409, @ltoscano wrote:
> Just update the English messages.
> More generally, why do we need copies of the system desktop files (even
the ones developed by KDE, org.kde.filelight.desktop?) Isn't there a way to use
the exi
BUILD FAILURE
Build URL
https://build.kde.org/job/Frameworks%20kdesignerplugin%20kf5-qt5%20FreeBSDQt5.7/8/
Project:
Frameworks kdesignerplugin kf5-qt5 FreeBSDQt5.7
Date of build:
Fri, 04 Aug 2017 16:23:48 +
Build duration:
1 min 6 sec and counting
CONS
BUILD FAILURE
Build URL
https://build.kde.org/job/Frameworks%20kdesignerplugin%20kf5-qt5%20XenialQt5.7/8/
Project:
Frameworks kdesignerplugin kf5-qt5 XenialQt5.7
Date of build:
Fri, 04 Aug 2017 16:23:48 +
Build duration:
2 min 25 sec and counting
CONSO
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:e7dcacfe64bf: Add cross-compilation support for the
highlighting indexer (authored by vkrause).
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7102?vs=1
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:703b48665f3d: Allow to build KSyntaxHighlighter without
Qt5Gui (authored by vkrause).
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7103?vs=17646&id=17
This revision was automatically updated to reflect the committed changes.
Closed by commit R237:ba75a6c6a023: Allow to build KConfig without Qt5Gui
(authored by vkrause).
REPOSITORY
R237 KConfig
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D6994?vs=17588&id=17722
REVISION DETAIL
gregormi updated this revision to Diff 17723.
gregormi edited the summary of this revision.
gregormi added a comment.
- add FSearch and AngrySearch to search tools
- Add new grouping "screenrecorder" with Peek and Simples Screen Recorder
- Add new grouping "mouse-tools"
- Add new grouping
gregormi edited the summary of this revision.
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D7130
To: gregormi, ltoscano
Cc: ltoscano, #frameworks
gregormi added a reviewer: elvisangelaccio.
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D7130
To: gregormi, ltoscano, elvisangelaccio
Cc: ltoscano, #frameworks
gregormi added a comment.
BUG: 369646
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D7130
To: gregormi, ltoscano, elvisangelaccio
Cc: ltoscano, #frameworks
gregormi edited the summary of this revision.
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D7130
To: gregormi, ltoscano, elvisangelaccio
Cc: ltoscano, #frameworks
vkrause updated this revision to Diff 17727.
vkrause added a comment.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
Fix homepage URL
REPOSITORY
R868 Yocto: KDE Frameworks Packaging
CHANGES SINCE LAST UPDATE
https://phabricator.k
vkrause created this revision.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
It's not necessary, and it breaks when cross-compiling, as find_file
changes its behavior then.
REPOSITORY
R311 KWallet
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7136
dfaure created this revision.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
commit
https://phabricator.kde.org/R241:4e2a52c67167507b62553ca595bb561b925dc440
prepends a scheme (like socks://), which interfers with
the assumption that after the first ':' comes the port.
dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.
Urls without schemes shouldn't really happen, so this fix doesn't make me too
happy. I debugged this KCM a bit and it seems the issue is the commit that
prepends a scheme (like so
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.
The diff has no context in phabricator, which is a problem for this one (e.g.
for the "do we need this here?" questions). Can you upload the update using
`arc diff` for example?
INLINE COM
dfaure added inline comments.
INLINE COMMENTS
> file_unix.cpp:659
> +{
> +if (error != EACCES || !isPrivilegeOperationAllowed()) {
> +return false;
with EPERM we should try as root too, no?
(e.g. for unlink, rename, chown...)
REVISION DETAIL
https://phabricator.kde.org/D6829
To:
dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.
Please resubmit with context.
Also, did you check that unittests still pass? No kauth prompt should occur
:-)
INLINE COMMENTS
> file.cpp:242
> #if defined(ENOTSUP)
> -
dfaure accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D6834
To: chinmoyr, dfaure
Cc: #frameworks
davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.
> on multiscreen systems, the notification dialog gets resized to 0x0
> by someone else (either kwin, notification applet or Qt code)
So which is it?
REPOSIT
davidedmundson accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7142
To: dfaure, dmariano, davidedmundson
Cc: #frameworks
BUILD SUCCESS
Build URL
https://build.kde.org/job/Frameworks%20kdesignerplugin%20kf5-qt5%20WindowsQt5.7/12/
Project:
Frameworks kdesignerplugin kf5-qt5 WindowsQt5.7
Date of build:
Fri, 04 Aug 2017 21:30:53 +
Build duration:
23 min and counting
JUnit Te
BUILD UNSTABLE
Build URL
https://build.kde.org/job/Frameworks%20ktexteditor%20kf5-qt5%20FreeBSDQt5.7/44/
Project:
Frameworks ktexteditor kf5-qt5 FreeBSDQt5.7
Date of build:
Sat, 05 Aug 2017 02:45:47 +
Build duration:
19 min and counting
JUnit Tests
38 matches
Mail list logo