> On March 26, 2015, 10:29 p.m., David Edmundson wrote: > > shell/panelview.cpp, line 433 > > <https://git.reviewboard.kde.org/r/123136/diff/1/?file=356709#file356709line433> > > > > and here.
topLeft is fine.. it is not affected > On March 26, 2015, 10:29 p.m., David Edmundson wrote: > > shell/panelview.cpp, line 443 > > <https://git.reviewboard.kde.org/r/123136/diff/1/?file=356709#file356709line443> > > > > and here? bottomLeft is fine, in general anythingLeft is fine. - Bhushan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123136/#review78069 ----------------------------------------------------------- On March 26, 2015, 10:11 p.m., Bhushan Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123136/ > ----------------------------------------------------------- > > (Updated March 26, 2015, 10:11 p.m.) > > > Review request for Plasma, Martin Gräßlin and Marco Martin. > > > Bugs: 339323 and 340549 > https://bugs.kde.org/show_bug.cgi?id=339323 > https://bugs.kde.org/show_bug.cgi?id=340549 > > > Repository: plasma-workspace > > > Description > ------- > > Appearantly right() / bottomRight() / topRight() returns values with 1 > pixel x off for historical reasons and hence 1px gap. This causes the > autohide panel not hiding automatically. > > > Diffs > ----- > > shell/panelview.cpp 3435736 > > Diff: https://git.reviewboard.kde.org/r/123136/diff/ > > > Testing > ------- > > added right aligned panel and tested various combinations > > > Thanks, > > Bhushan Shah > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel