Selecting text in table cell and moving across cells shifts caret to the other cell

VERIFIED FIXED

Status

()

Core
Editor
P3
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: shrirang khanzode, Assigned: Blake Ross)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
I see this on the windows trunk build 0214

Steps:
1 Launch composer and add a table of 2x2
2 Type something in first and its adjacent cell (first row)
3 Now keep the caret at the last character of the second cell and highlight the   
  cell contents with mouse and cross the cell boundary.

Actual result : Observe that the caret is now actually blinking in the first 
cell 

not sure if this is the expected output but 4.x behaves differently

Comment 1

17 years ago
reassign to cmanske
Assignee: beppe → cmanske

Comment 2

17 years ago
I don't do carets ;-)
Both Simon and Tony are busy. Any ideas who could do this?
Assignee: cmanske → beppe

Comment 3

17 years ago
load balancing some bugs -- assigning to sfraser and setting to moz0.9.1
Assignee: beppe → sfraser
Priority: -- → P3
Target Milestone: --- → mozilla0.9.1

Comment 4

17 years ago
I think the bug here is that the caret shouldn't be visible when there is a 
non-collapsed selection ... the fact that the focus node of the selection snaps 
to the end of the text in the first cell, after you've crossed the cell boundary 
is irrelevant.

Comment 5

17 years ago
I agree and filed bug 64643 on the issue. Blake Ross owns that bug and claimed
he turned on the caret as a fix for bug 41077. It seems textfields (at least
in Windows) do show a blinking caret at the end of the selection highlighting,
but you usually don't see that behavior in word processors such as MS Word.
So we really need to decide if we want a caret or not!

Updated

17 years ago
Summary: Selecting text in table cell and moving across cells shifts cursor to the other cell → Selecting text in table cell and moving across cells shifts caret to the other cell

Comment 6

17 years ago
This is blake's bug.
Assignee: sfraser → blakeross

Comment 7

17 years ago
going to unset target milestone.  blake, set to something reasoable 
when you get a chance.
Target Milestone: mozilla0.9.1 → ---
(Assignee)

Comment 8

17 years ago
This is fixed now that the caret is off (and jfrancis' checkin may have fixed it 
anyways).
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
Verified on 8-28 build
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.