----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120584/#review68403 -----------------------------------------------------------
Ship it! Sounds sensible. however, can someone try plasma without this patch and with the Qt patch: https://codereview.qt-project.org/#/c/97050/ this one may be a workaround for the qt xcb problem adressed in the above, but i'm not 100% sure since i can't reproduce the issue of panel starting up on wrong monitor (even if is a workaround, this one may be needed anyways since it would take aeons anyways for the qt patch making to releases) - Marco Martin On Ott. 14, 2014, 4:15 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120584/ > ----------------------------------------------------------- > > (Updated Ott. 14, 2014, 4:15 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > I was getting the Panels wrongly positioned because Qt would reset the > position at some point while initializing the class. This patch disables any > position while the panel is not shown and makes sure it's placed when it's > set to be shown. > > > Diffs > ----- > > shell/panelview.cpp 9260c18 > > Diff: https://git.reviewboard.kde.org/r/120584/diff/ > > > Testing > ------- > > Both my installations initialize properly now. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
