D21746: [autotests] Fix remaining autotests with failing border calculation
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag added inline comments. INLINE COMMENTS > dont_crash_glxgears.cpp:78 > // send a mouse event to the position of the close button > - QPointF pos = decoration->rect().topRight() + QPointF(-decoration->borderRight() * 2, decoration->borderRight() * 2); > + // TODO: position is dependent on the decoration in use. We should use a static target instead, a fake deco for autotests. > + QPointF pos = decoration->rect().topRight() + QPointF(-decoration->borderTop() / 2, decoration->borderTop() / 2); Can't we just update the config? REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D21746 To: romangg, #kwin Cc: zzag, kwin, LeGast00n, ericadams, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart