Issue 21492

Summary:

Excel sheet conversion error of hyperlinks to cells, formatting en colors

Product:

Calc

Reporter:

rongroen <list00>

Component:

code

Assignee:

Status:

CLOSED FIXED

QA Contact:

issues@sc <issues>

Severity:

Trivial

Priority:

CC:

issues

Version:

OOo 1.1 RC4

Keywords:

ms_interoperability

Target Milestone:

---

Hardware:

OS:

Windows 2000

URL:

http://aha4adsl.nl/ooconversie1.xls

Issue Type:

DEFECT

Latest Confirmation in:

---

Developer Difficulty:

---

Attachments:

Description	Flags
proposed to fix invalid reference	none
Fix all URL fields not just the URL text mark	none

Description rongroen 2003-10-21 10:56:57 UTC

In the URL you will find a Excel sheet. Opening this sheet in OO 1.1. RC4 NL 
gives the following problems:

Cel A5
The color of the font is wrong: it is blue in stead of black.
The background of the font is wrong: It is grey in stead of yellow.

Cel B5
There is content in the cell. It should be empty. The colors of this cell are 
wrong as well.

Comment 1 john.marmion 2003-10-21 12:29:09 UTC

Yes, problems as reported by submitter. accepted.

Comment 2 john.marmion 2003-10-21 12:30:10 UTC

mark as accepted.

Comment 3 john.marmion 2003-10-21 16:34:40 UTC

There are a number of problems in the import of this document. The URL
are all to specific locations in the document. 

1. We get an invalid reference to the text mark location as we fail to
take into account that the text mark is zero terminated.
2. We default to displaying the URL in the cell if no text is present
- Excel simply displays blank text.
3. There is no formatting supported of the url in the import beyond
the default format.Thus the default blue font is used. 
4. Screen Tips are not imported. 

We do get the actual cell background colour correct.

Comment 4 john.marmion 2003-10-31 16:42:15 UTC

setting the target. Just to clarify one point, Calc displays the URL
text in blue with a grey back background. The overall cell background
colour is not affected by this.

Comment 5 john.marmion 2003-11-06 10:36:45 UTC

I will attach a patch to fix the invalid reference of the url text
mark. But I propose that we leave the other issues raised by this
issue as they are. 

1. On the blank url text cell, Excel treats the complete cell as an
URL cell while in Calc, the URL cell is only activated from the URL
text portion. Thus if no text is present, we default the Target as the
Text portion. Otherwise no activation would be possible.

2. The URL text/target portion is displayed in blue on a grey
background only. 

3. Screen tips not currently supported.

Comment 6 john.marmion 2003-11-06 10:38:47 UTC

Created attachment 10999 [details]
proposed to fix invalid reference

Comment 7 john.marmion 2003-11-06 19:16:04 UTC

Created attachment 11024 [details]
Fix all URL fields not just the URL text mark

Comment 8 john.marmion 2003-11-11 17:26:30 UTC

Checked in patch today into cws/calc18.

Comment 9 john.marmion 2004-02-12 13:33:19 UTC

back to QA.

Comment 10 oc 2004-02-26 12:01:03 UTC

reset resolution to fixed

Comment 11 oc 2004-02-26 12:02:40 UTC

Fix for broken URL verified in internal build cws_calc18

Comment 12 oc 2004-03-16 11:59:54 UTC

Fix  available in developer snapshot OOo680m38