Hi YunJingtao, On Thursday, 2007-11-15 17:13:34 +0800, YunJingtao wrote:
> aLogicEdit is to show the area of hyperlink with mouse,when the text is > beyond the cell border, border of aLogicEdit has to be coordinated with cell. > > the coordinates of text and mouse are different, and it leads to the > result.Findings is shown in ImpEditEngine::GetChar. > > the original position of text is up to aPaperSize and nTextWidth, > position of mouse is up to aTempView and OutputArea. Well, yes, I think we know what the variables are used for, but that doesn't explain why setting a new paper width would be necessary, as it is done in the patch. Also, the portion around if ( nTextWidth > nThisColLogic ) introduced in the patch is very similar to the already existing one near line 4741 if ( nTextWidth < nThisColLogic ) that treats the aLogicEdit's nStartX, so I wonder whether it is coincidence that these cases differ from the default path, or maybe could be rewritten to generalize the behavior. This especially because Rectangle aOutputArea = PixelToLogic( aEditRect, aEditMode ); is essentially the same as Rectangle aLogicEdit = PixelToLogic( aEditRect, aEditMode ); at line 4705 (line numbers as present in m236). Eike -- OOo/SO Calc core developer. Number formatter stricken i18n transpositionizer. SunSign 0x87F8D412 : 2F58 5236 DB02 F335 8304 7D6C 65C9 F9B5 87F8 D412 OpenOffice.org Engineering at Sun: http://blogs.sun.com/GullFOSS Please don't send personal mail to the [EMAIL PROTECTED] account, which I use for mailing lists only and don't read from outside Sun. Use [EMAIL PROTECTED] Thanks.
pgpaX37UMSAhn.pgp
Description: PGP signature