januz added a comment.
> I think this explains why you haven't had a review, this existing code is
already sketchy as-is :/
> But we should be trying to fix that not just all of us ignoring it. Sorry
about that.
No problem, I can understand the dread of piling stuff on code that needs
This revision was automatically updated to reflect the committed changes.
Closed by commit R124:a88cad3bc72c: Header as one ToolButton for Back action
(authored by subdiff).
REPOSITORY
R124 System Settings
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7749?vs=19338&id=19384
REVISIO
graesslin added inline comments.
INLINE COMMENTS
> davidedmundson wrote in blur.cpp:76
> I was expecting you to do what you for the present windows effect with
> calling your local lambda announceSupportProperties.
>
> IMHO it's neater than duplicating - but this is fine too.
I didn't go for t
davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> blur.cpp:76
> connect(effects, SIGNAL(screenGeometryChanged(QSize)), this,
> SLOT(slotScreenGeometryChanged()));
> +connect(effects, &EffectsHa
graesslin updated this revision to Diff 19381.
graesslin added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
Addressed David's comments
REPOSITORY
R108 KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7761?vs=19367&id=19381
BRANCH
effects-xcb
graesslin updated this revision to Diff 19375.
graesslin added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
Check whether we have SCHED_RESET_ON_FORK and ifdef accordingly
REPOSITORY
R108 KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7757?vs
mart accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R124 System Settings
REVISION DETAIL
https://phabricator.kde.org/D7749
To: subdiff, #plasma, mart
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,
abetts, sebas, apol, mart
mart added a comment.
now pings and tries to close at the same time, so either will be closed
cleanly or killed if not responsive
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D6591
To: mart, #plasma, graesslin, davidedmundson
Cc: mart, graesslin, kwin, plasma-devel,
mart updated this revision to Diff 19373.
mart added a comment.
Restricted Application edited projects, added KWin; removed Plasma.
- don't ping a window before killing setActive works also for wl_shell
REPOSITORY
R108 KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D6591?vs=19
mart added a comment.
i removed the ping to close window.. however it still doesn't sound right, as
the client could not answer a close request, there may need a close and a ping
at the same time, then whoever wins, either the window gets gracefully closed
or brutally killed
REPOSITORY
R1
mart updated this revision to Diff 19372.
mart added a comment.
Restricted Application edited projects, added Plasma; removed KWin.
- Temporary popup stuff
- Update to signal rename
- skip taskbar/pager/windowmanagement for xdgpopups
- send configure on popup resize
- Merge branch 'mart
mart commandeered this revision.
mart edited reviewers, added: davidedmundson; removed: mart.
Restricted Application edited projects, added KWin; removed Plasma.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D6591
To: mart, #plasma, graesslin, davidedmundson
Cc: mart, grae
bshah added a comment.
In https://phabricator.kde.org/D7740#144077, @graesslin wrote:
> @bshah could you please try this patch on hwcomposer platform? I tried to
get all required changes into it, but I currently don't have a compiler to
verify.
Can you commit them in one branch?
mart added inline comments.
INLINE COMMENTS
> graesslin wrote in shell_client.cpp:608-609
> That logic I don't understand: why do we ping to close? Why a roundtrip to
> the app, when all we want to do is close it? Just send it a close?
i copied from the x11 client, where on client::CloseWindow
davidedmundson accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R108 KWin
BRANCH
xrandr-event-filter
REVISION DETAIL
https://phabricator.kde.org/D7654
To: graesslin, #kwin, #plasma, davidedmundson
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, leslie
davidedmundson requested changes to this revision.
davidedmundson added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> contrast.cpp:63
> +[this] {
> +net_wm_contrast_region =
> effects->announceSupportProperty(s_contrastAtomName, this);
> +
This revision was automatically updated to reflect the committed changes.
Closed by commit R108:8522ef17eea0: Do not hard runtime depend on X11 in
RuleBook (authored by graesslin).
REPOSITORY
R108 KWin
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7653?vs=19083&id=19369
REVISION DE
graesslin created this revision.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
Several effects announce a support property atom on the root window. This
change forwards the KWin::Application's signal that the xcbCon
tcberner added a comment.
I could not find anything similar. So that's probably the way to go.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D7757
To: graesslin, #kwin, #plasma
Cc: tcberner, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai,
ali-mohamed, harde
graesslin created this revision.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
Only needed for kwin_x11 variant (required for the non-composited
Outline). As that's nowadays in the x11 platform, we can move the
co
This revision was automatically updated to reflect the committed changes.
Closed by commit R846:3a4cd88c7f27: Improved Suggestions Model (authored by
Aiix).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D5497?vs=13576&id=19360#toc
REPOSITORY
R846 Mycroft Plasma integration
CHANGES SIN
graesslin added a comment.
In https://phabricator.kde.org/D7757#144415, @tcberner wrote:
>^
> /home/Tobias/git/kwin/main_wayland.cpp:452:38: error: use of undeclared
identifier 'SCHED_RESET_ON_FORK'
> sched_setscheduler(0, SCHED_RR | SCHED_RESET_ON_FORK, &s
davidedmundson accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R108 KWin
BRANCH
rulebook-no-x11
REVISION DETAIL
https://phabricator.kde.org/D7653
To: graesslin, #kwin, #plasma, davidedmundson
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai
graesslin added a comment.
ping
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D7654
To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed,
hardening, jensreuterberg, abetts, sebas, apol, mart
graesslin added a comment.
ping
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D7653
To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed,
hardening, jensreuterberg, abetts, sebas, apol, mart
This revision was automatically updated to reflect the committed changes.
Closed by commit R320:ffc2a64d1f97: Don't enter test subdirectories if
BUILD_TESTING=OFF (authored by asturmlechner).
REPOSITORY
R320 KIO Extras
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7756?vs=19352&id=1
davidedmundson accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R320 KIO Extras
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7756
To: asturmlechner, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jens
tcberner added a comment.
^
/home/Tobias/git/kwin/main_wayland.cpp:452:38: error: use of undeclared
identifier 'SCHED_RESET_ON_FORK'
sched_setscheduler(0, SCHED_RR | SCHED_RESET_ON_FORK, &sp);
^
Yes, `SCHED_RESET_ON_F
graesslin added a subscriber: tcberner.
graesslin added a comment.
@tcberner could you please try to compile this change on FreeBSD? I just want
to be sure that it doesn't introduce a linux-ism. The most prominent dangerous
part could be SCHED_RESET_ON_FORK where I'm not sure whether it is li
graesslin created this revision.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
The base idea behind this change is to keep the system responsive no
matter what other processes do. All input and rendering needs to go
asturmlechner created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Qt5Test is an optional dependency but not followed in some subdirs.
REPOSITORY
R320 KIO Extras
BRANCH
master
REVISION DETAIL
htt
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:940adc7e6e69: [Plasma Components 3] Fix RTL in some
widgets. (authored by safaalfulaij).
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7715?vs=1
hein accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D7733
To: broulik, #plasma, hein
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,
abetts, sebas, apol, mart
33 matches
Mail list logo
