<div dir="ltr"><div><div><div><div><div><div>Hey!<br></div>I am hereby attaching my bug after making all the needed changes :<br></div>*I have removed all the unnecessary changes i had made to the code,i.e. i did undo all the useless changes i had made to the code,thereby cleaning up the code.<br>
</div>*removed the command that gave the output : <br>debug: max width 78<br>debug: max width 55<br>[...]<br></div>*now i think the display also seems to be perfect and the purpose of this feature seems to be solved.<br><br>
</div><div>Any kind of help and suggestions for future hacking would be appreciated.<br><br></div>Regards,<br></div>Janit<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 9, 2013 at 8:10 AM, Janit Anjaria <span dir="ltr"><<a href="mailto:janit92@gmail.com" target="_blank">janit92@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im"><div><div>Hey Thorsten,Stefan ,<br><br></div>I just saw the patch,yes it
definitely need clean up based on the white spaces ,and yeah the
numbers in the patch are added so that the UI fixes right and hence the
toolbar appears to be right.I am not sure about the reason why the icons
dont appear in your case,as it does appear when i run the code on my
machine.<br>
<br></div><div>And regarding the following: <br><div>debug: max width 388<br>
debug: max width 78<br>
debug: max width 55<br>
[...]<br></div></div><div>This is just an output for the debugging
i had done for the UI purpose and yeah it would be fine,i would get
that part of the code out.<br><br></div>Regards,<br>Janit</div><div><div><img src="https://mail.google.com/mail/images/cleardot.gif"></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Tue, Apr 9, 2013 at 8:09 AM, Janit Anjaria <span dir="ltr"><<a href="mailto:janit92@gmail.com" target="_blank">janit92@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div>Hey Thorsten,Stefan ,<br><br></div>I just saw the patch,yes it definitely need clean up based on the white spaces ,and yeah the numbers in the patch are added so that the UI fixes right and hence the toolbar appears to be right.I am not sure about the reason why the icons dont appear in your case,as it does appear when i run the code on my machine.<br>
<br></div><div>And regarding the following: <br><div>debug: max width 388<br>
debug: max width 78<br>
debug: max width 55<br>
[...]<br></div></div><div>This is just an output for the debugging i had done for the UI purpose and yeah it would be fine,i would get that part of the code out.<br><br></div>Regards,<br>Janit<br></div><div>
<div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Apr 9, 2013 at 3:54 AM, Thorsten Behrens <span dir="ltr"><<a href="mailto:thb@documentfoundation.org" target="_blank">thb@documentfoundation.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div>Janit Anjaria wrote:<br>
> I am hereby submitting my patch for review on the bug/feature which i had<br>
> worked long back which solves the problem where the comboboxes now do<br>
> appear when the toolbar is vertical.<br>
><br>
</div></div>Hi Janit,<br>
<br>
the patch contains a lot of changes ala<br>
<br>
if ( eAlign == WINDOWALIGN_TOP )<br>
{<br>
- rLeft = borderwidth+dragwidth;<br>
+ rLeft = borderwidth+dragwidth-150;<br>
rTop = borderwidth;<br>
<br>
- where do those numbers come from; as Astron points out, it breaks<br>
more than it fixes?<br>
<br>
Also, please remove unrelated whitespace / formatting changes, ala<br>
<br>
{<br>
pThis->mpData->mbAssumeDocked = sal_True; // force non-floating mode during calculation<br>
- ImplCalcBorder( WINDOWALIGN_LEFT, nLeft, nTop, nRight, nBottom, pThis );<br>
+ ImplCalcBorder( WINDOWALIGN_LEFT, nLeft, nTop, nRight, nBottom, pThis );<br>
<br>
, or submit it separately (the example above though really makes<br>
matters worse, not better).<br>
<br>
Please spend a bit of time on reviewing / cleaning up your patch<br>
submissions *before* mailing them out. This increases your chances of<br>
getting useful and timely feedback.<br>
<br>
Cheers,<br>
<br>
-- Thorsten<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>