141809 - View doesn't scroll to caret on large indent

Status: RESOLVED WORKSFORME

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

Description

[Dan McGuirk]

Open a blank message, and keep hitting the indent button until you've indented
all the way off the side of the screen.  I would expect the view to scroll to
always keep the caret visible, but it doesn't; the caret disappears.

Comment 1

[Charles Rosendahl]

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

Comment 2

[Judson Valeski]

using trunk builds, the scrollbar moves as expected, however, the blinking
cursor disappears once you get to the window's edge. it re-appears once you
start typing.

expected results are that the scrollbar scrolls (as is happening) *and* the
cursor stays visible and adjacent to the rightmost window border.

Comment 3

[rubydoo123]

Using a current trunk build the horizontal scrollbars are activated, however,
the caret disappears until typing begins.

Comment 4

[kinmoz]

Just an observation from using composer, it looks like in CSS mode we implement
the indent as a left margin ... it looks like the caret disappears because the
br it attatches to for rendering is pushed out of the bounds of the div
containing it, by the margin ... I'm guessing that the div doesn't think it
needs to resize because brs don't really add width to a line.

Comment 5

[Michael Dunn]

Can we have an updated milestone?

Comment 6

[Kevin McCluskey (gone)]

Loading balancing bugs -> smontagu

Comment 7

[Simon Montagu :smontagu]

This is a combination of what kin said in comment 4 and the code in
nsCaret::GetCaretRectAndInvert() which tweaks the caret position so it doesn't
overhang the frame.

It's really a very special case of the behaviour in bug 105397

Comment 8

[Makoto Kato [:m_kato]]

old bug.  down to P3

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:hsinyi, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 10

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I don't reproduce this bug anymore on Daily and Firefox Nightly.