Apache OpenOffice (AOO) Bugzilla – Issue 122709
Change default size of application windows on Linux
Last modified: 2017-05-20 11:42:06 UTC
The default sizes of application windows on Linux are almost square (when not fullscreen for Calc, Draw, Impress). This does not work well with the new sidebar, which was designed for wide screen displays. Change the default sizes to what is used on Mac: use 80% of width and height.
Requesting release blocker flag because a better default size would greatly benefit the new sidebar feature. The change is small and the probability for introducing regressions is small. Only Linux (GTK module) is affected.
grant showstopper flag
"af" committed SVN revision 1501668 into trunk: 122709: Set default size of application windows to 80% of the screen size.
Modified GtkSalFrame::calcDefaultSiz() so that for screens with a width larger than 1024 the returned size is 80% of width and height of the screen size.
http://svn.apache.org/viewvc/openoffice/trunk/main/vcl/unx/gtk/window/gtkframe.cxx?view=markup&pathrev=1501668#l1248 if (true || aScreenSize.Width() <= 1024) this looks like a typo, it evaluates always to true, so the first branch is chosen, and the new code with the fix is never executed.
Yes, I used the "true ||" part for preparing screen shots. Fortunately this is harmless. It just means that the old sizes are still used.
"af" committed SVN revision 1502467 into trunk: 122709: Removed debug code.
@jsc: unless you plan another release candidate for this fix, shouldn't release blocker status be removed - as verification in the current RC will fail?
yes, I changed it back to ?
"af" committed SVN revision 1503702 into trunk: 122709: Also test for small heights.
grant showstopper flag again, was already planned for a potential respin and fix is available
"af" committed SVN revision 1503704 into branches/AOO400: 122709: Removed debug code and testing for small heights.
Merged revisions r1502467 and r1503702 from trunk into the AOO400 branch. The first removes the debug code (see comments 5 to 7), the second makes sure that the old hand crafted sizes are also used when the screen height is smaller than or equal to 768 pixels. Status remains resolved/fixed.
I check it on Ubuntu 32 bit, it has fullscreen for Writer, Impress, Calc and Draw
I check it on RHEL 64bit, it has fullscreen for Writer, Impress, Calc and Draw