14611 - [BLOCKER] Can't select non-text nodes such as HR, IMG, and table elements

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P1)

Description

[Charles Manske]

We used to be able to click on a HR or Image to select it. That doesn't work
any more. If you can't select an image, you can't bring up the property dialog
for that image, thus it blocks work on editing properties of an image.
Tables are a problem. While I understand that people complained about not
being able to select text in tables, we need to be able to select table cells
in both the browser and editor.

Comment 1

[buster]

the portion about hr and image is critical for beta.  If it's an easy, safe fix,
please try to get it in for M11.  Otherwise, M12.

Comment 2

[Charles Manske]

*** Bug 14821 has been marked as a duplicate of this bug. ***

Comment 3

[Kathleen :Brade]

I thought I put comments in here earlier today but apparently not.  The
workaround for this bug should be to place the caret immediately before or after
the image/hrule you want to select and then use the shift and arrow to select it.
(Though this is far from obvious.)

Unfortunately the selection seems to disappear.  Also, no dialog comes up at all
for hrule (an empty image dlog comes up for an image).  Has anyone reported this
problem?

This happens on Mac as well so changing platform/OS to ALL.

Comment 4

[Charles Manske]

I don't think your "workaround" sets the same selection as clicking on the
object used to. Thus when you try to run a property dialog, GetSelectedElement()
fails because it is very fussy about having just the object in question
selected. HRules will be inserted instead of property editing in this case,
other dialogs will not show the settings of the intended object.
MIKE: ARE YOU FIXING THIS SOON PLEASE?

Comment 5

[buster]

Mike, please look at this today.  If it can be fixed safely, it should go in for
M10, which pretty much means today sometime.

Comment 6

[mjudge]

this is fixed. i thought i updated the bug sorry

Comment 7

[sujay]

okay you can select it all right but now we have another
problem in that you can't un-select it so you can type
text next to the object...I filed another bug...