russellg removed R39 KTextEditor as the repository for this revision.
russellg updated this revision to Diff 11390.
russellg added a comment.
Updated the test to be slightly better. Use `QVERIFY(...)` instead of
`QCOMPARE(..., true)` in one case, and make sure that `saveEnabled()` is false
russellg created this revision.
russellg set the repository for this revision to R39 KTextEditor.
Restricted Application added subscribers: Frameworks, kwrite-devel.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
I'm currently writing an application where the user is
ivan added a comment.
I will use them, but don't do that just yet - I want to propose some changes
to them.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D4626
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: ivan, alex-l,
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:b79085c027c0: Renamed icons for encrypted and decripted
folders (authored by ivan).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D4626?vs=11385=11386#toc
REPOSITORY
R266 Breeze Icons
andreaska accepted this revision.
andreaska added a comment.
This revision is now accepted and ready to land.
good point. If you want to use this icons I'll make them for the other places
size.
REPOSITORY
R266 Breeze Icons
BRANCH
ivan/rename-icons
REVISION DETAIL
ivan created this revision.
ivan added reviewers: alex-l, andreaska.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
Since the icons are in places, not in actions, changed the names
to imply state, not action.
ivan created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
Merge branch 'master' into ivan/plasmavault-icons
REPOSITORY
R266 Breeze Icons
BRANCH
ivan/plasmavault-icons
REVISION DETAIL
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:bc60e3c2f85f: Added arc config file (authored by ivan).
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4624?vs=11382=11383
REVISION DETAIL
ivan created this revision.
ivan added a reviewer: Frameworks.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
Added arc configuration file
REPOSITORY
R266 Breeze Icons
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4624
AFFECTED FILES
.arcconfig
smartins added inline comments.
INLINE COMMENTS
> dfaure wrote in kdirwatch.cpp:1379
> Detaches? a std::vector?
doh!
REPOSITORY
R244 KCoreAddons
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D4584
EMAIL PREFERENCES
dfaure added inline comments.
INLINE COMMENTS
> hein wrote in dropjob.cpp:324
> Should this be && instead of ||? "Add actions if either of those is not
> empty" seems weird, what's the reasoning?
It's a copy of the if() on line 311. But there it makes sense. Here not so much
;)
What if there
dfaure added inline comments.
INLINE COMMENTS
> smartins wrote in kdirwatch.cpp:1379
> can m_clients be shared ? If yes, htis detaches
Detaches? a std::vector?
> mwolff wrote in kdirwatch_p.h:84
> right, but that's what you want to state here, no? anyhow, I'm also OK with
> leaving it as-is
smartins added inline comments.
INLINE COMMENTS
> kdirwatch.cpp:1379
>
> -Q_FOREACH (Client *c, e->m_clients) {
> -if (c->instance == nullptr || c->count == 0) {
> +for (Client : e->m_clients) {
> +if (c.instance == nullptr || c.count == 0) {
can m_clients be shared ?
mwolff accepted this revision.
mwolff added a reviewer: mwolff.
mwolff added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> dfaure wrote in kdirwatch_p.h:84
> Given that the same thing happens by default, what would be the reason to
> make it explicit?
drosca added a comment.
I now get tons of binding loop errors from FrameSvgItem, it also breaks
delegates in networkmanager and bluetooth applets. On the screenshot you can
see that the last 3 delegates (HUAWEI, UPC and Internet) are slightly moved to
the left and hovering over them will
hein added a comment.
Much nicer than the other approach, just one question.
INLINE COMMENTS
> dropjob.cpp:324
> +for (QMenu *menu : d->m_menus) {
> +if (!d->m_appActions.isEmpty() || !d->m_pluginActions.isEmpty()) {
> +menu->addSeparator();
Should this be && instead
mart abandoned this revision.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D4575
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: mart, #plasma
Cc: hein, plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg,
abetts,
mart updated this revision to Diff 11371.
mart added a comment.
- remove useless signal
REPOSITORY
R241 KIO
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4620?vs=11369=11371
BRANCH
phab/setActionsLater
REVISION DETAIL
https://phabricator.kde.org/D4620
AFFECTED FILES
mart created this revision.
mart added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY
even if the menu has already been created, cause setapplicationActions
to add actions in
mart updated this revision to Diff 11368.
mart added a comment.
- use dropjobs instead of directly accessing its qmenu
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4576?vs=11307=11368
BRANCH
arcpatch-D4576
REVISION DETAIL
atomsymbol created this revision.
atomsymbol added a reviewer: kfunk.
atomsymbol set the repository for this revision to R39 KTextEditor.
Restricted Application added subscribers: Frameworks, kwrite-devel.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
Fixes bug 376060
gszymaszek updated this revision to Diff 11366.
gszymaszek added a comment.
key and value are const.
REPOSITORY
R39 KTextEditor
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4537?vs=11365=11366
REVISION DETAIL
https://phabricator.kde.org/D4537
AFFECTED FILES
CMakeLists.txt
gszymaszek updated this revision to Diff 11365.
gszymaszek marked 2 inline comments as done.
gszymaszek added a comment.
Pulled check-variable-functions out of EditorConfig class, moved some
variable definitions into main loop of parser.
REPOSITORY
R39 KTextEditor
CHANGES SINCE LAST
dhaumann added a comment.
Patch looks already pretty good, I think we're soon there.
INLINE COMMENTS
> gszymaszek wrote in editorconfig.cpp:23
> Is it OK to initialize `m_handle` in the constructor? If so, is `m_handle(0)`
> necessary?
Yes, this is good now.
> editorconfig.cpp:34-56
>
adridg added a comment.
I'm going to claim feedback timeout, plus all it does is add some -I flags
when compiling the tests.
REPOSITORY
R159 KActivities Statistics
REVISION DETAIL
https://phabricator.kde.org/D4248
EMAIL PREFERENCES
This revision was automatically updated to reflect the committed changes.
Closed by commit R159:d2cbef5d94f3: Add Boost_INCLUDE_DIR to kactivities-stats
autotests. (authored by adridg).
REPOSITORY
R159 KActivities Statistics
CHANGES SINCE LAST UPDATE
This revision was automatically updated to reflect the committed changes.
adridg marked 2 inline comments as done.
Closed by commit R240:a02c4d0152a7: Detect inotify. (authored by adridg).
REPOSITORY
R240 Extra CMake Modules
CHANGES SINCE LAST UPDATE
27 matches
Mail list logo