----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105975/#review18374 -----------------------------------------------------------
This review has been submitted with commit c313fa5323b0e8b7de50d1eecfbc3b66c6066e23 by Friedrich W. H. Kossebau to branch kexi-features-staniek. - Commit Hook On Aug. 10, 2012, 11:52 p.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105975/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2012, 11:52 p.m.) > > > Review request for Calligra and C. Boemann. > > > Description > ------- > > I had the idea to render the column separators using the background shapes. > What do you think? Would that be alright at least as a temporary solution? > > Patch also fixes page background ignoring the margins. At least LO/AOO does > not paint the background in the margin. Sadly the attribute > "draw:background-size" is not available for <style:page-layout-properties>, > only for <style:drawing-page-properties>. Filed a complain on the > office-comment mailinglist. > > And patch removes redundant setting of background to frame shapes in > KWTextFrameSet::setPageStyle(...). > > Okay to backport to 2.5? > > > Diffs > ----- > > words/part/frames/KWFrameLayout.cpp d9675b0 > words/part/frames/KWTextFrameSet.cpp 7afa7bf > > Diff: http://git.reviewboard.kde.org/r/105975/diff/ > > > Testing > ------- > > > Thanks, > > Friedrich W. H. Kossebau > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
