D17826: Honor struts when placing Wayland transients
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag accepted this revision.
zzag added a comment.
This revision is now accepted and ready to land.
Off-topic: regarding the subject line, what's the preferred style
Some text Wayland
or
[wayland] Some text
?
INLINE COMMENTS
> placement.cpp:501
> {
> - const QRect screen = screens()->geometry(c->transientFor()->screen());
> + auto parent = c->transientFor();
> + const QRect screen = Workspace::self()->clientArea(parent->isFullScreen() ? FullScreenArea : PlacementArea, parent);
const auto *
REPOSITORY
R108 KWin
BRANCH
transient-honor-struts
REVISION DETAIL
https://phabricator.kde.org/D17826
To: graesslin, #kwin, zzag
Cc: zzag, kwin, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart