103957 - Composer allows you to put a link in a blank cell without text or image by selecting the cell t hen clicking on link, but unable to make entire cell a link

Status: VERIFIED DUPLICATE of bug 97155

(Core :: DOM: Editor, defect)

Description

[TucsonTester2]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win 9x 4.90; en-US; rv:0.9.4)
Gecko/20011009 Netscape6/6.2
BuildID:    20011009

If you select an empty cell and then click on link, you are able to create a
link without being required to enter text or insert an image.  The Link
Properties dialog box shows "mixed selection" in the section for Link Text. 
This is inconsistent with how adding a link works elsewhere on a page.  Anywhere
else, you are required to enter Link Text (unless you are adding a link to an
image).  This only happens if you select the entire cell.  If you simply click
on it so the cursor is in the cell and then click link, it will ask for Link Text.

Reproducible: Always
Steps to Reproduce:
1.Open a blank page in composer
2.Click on Table, create a default table
3.Right click on a cell, and then click on Table Select->Cell
4.Click on Link
5.Enter a webaddress for a link (i.e. http://www.netscape.com) and click ok
6.Select file->Browse Page
&.Click on yes to save page

Actual Results:  If you put the mouse pointer in the very upper left corner of
the cell and click, it will activate the link.  Clicking anywhere else in the
cell will not activate the link.  There is nothing visible to indicate the
presence of a link (although this isn't necessarily a bad thing).

Expected Results:  What I would really like to see is the entire cell as a link.
 I think it would be desirable to be able to set it so that clicking anywhere in
the cell will activate the link, instead of having to click on an image or click
on text in the cell.  If this is not possible, then I would expect Link
Properties to work the same as anywhere else on the page, asking you for Link
Text if there is no image or text already present.

If I were to take a cell and put something like "Click Me" in it, I would like
to have the ability to set it so that clicking anywhere in the cell will work,
instead of having to click on the text to activate the link.  The only way I see
to do this currently is to create an image with the text I want as part of the
image and insert the image so that it fills the cell, then make it the link.

I was a little unsure about whether to put this as a bug, an enhancement, or
just leave it that reality doesnt work the way I want it to.  But the way it
works currently, the inconsistency in the way Link Properies handles the lack of
text or image would seem to be a bug.

Comment 1

[Syd Logan]

Charles, could you determine if this is by design, or if not, what we need to do
about it, if anything?

Comment 2

[Charles Manske]

Joe: What are the rules when an inline style, such as a link, is applied to 
block elements? I thought they created attribute elements around the contents 
*within* each block element in the selection. That is what happens if you select
across block element boundaries such as various paragraph styles, such as <p>
and <h1> etc.
I'm not sure if table cells should be a special case, but obviously it seems that 
making a link out of selected cell(s) should apply the above logic to all contents
of each cell in the selection.

Comment 3

[michael wendell]

Confirming.  I see this problem on build 2001111203.  I think we are waiting to
see if this is working by design or not, but no comments have been made in about
a month.  

Comment 4

[Joe Francis]

Charlie, I'm happy to help here but the first issue seems to be what is the link
dialog doing to determine if it needs to ask for link text?  I'm not really
familiar with the code there.  Hoping you can look into that and tell me what
editor call needs to behave differently for you, or perhaps what new
functionality you need.

Comment 5

[Charles Manske]

JFrancis: The link dialog looks for text by simply getting the 

*** This bug has been marked as a duplicate of 97155 ***

Comment 6

[sujay]

verified.