can't outdent highlighted/selected text

VERIFIED FIXED in mozilla0.9.6

Status

()

P2
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: sujay, Assigned: mozeditor)

Tracking

({regression})

Trunk
mozilla0.9.6
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: EDITORBASE; 1 day; fixinhand; need r=, sr=)

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
using 8/21 build.

1) launch netscape
2) launch composer
3) insert text
4) select All
5) hit indent a few times

this works fine.

6) now try outdent

doesn't work.

works only if that text is not highlighted/selected.

Comment 1

18 years ago
brade
Assignee: beppe → brade

Comment 2

17 years ago
sounds like a core regression
-->jfrancis
Assignee: brade → jfrancis
Keywords: regression
(Assignee)

Comment 3

17 years ago
doh. outdent control (and menu item) is not even enabled.
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.5

Comment 4

17 years ago
Also the shortcut keys (CTRL+-)does not outdent the text, it resizes the text.
Indent's shortcut keys do the same thing(CTRL+=). Even if they are not
highlighted, the shortcut keys do not work for this function.
Windows NT
(Assignee)

Updated

17 years ago
Whiteboard: EDITORBASE; 1 day

Comment 5

17 years ago
This works the same for images too.  But, if you reselect the image the outdent
will work.
(Assignee)

Comment 6

17 years ago
Created attachment 52389 [details] [diff] [review]
patch for editor/libeitor/html/nsHTMLEditRules.cpp
(Assignee)

Comment 7

17 years ago
above patch should fix the problem.  act now and you will also receive a free
bonus fix for paste!
Whiteboard: EDITORBASE; 1 day → EDITORBASE; 1 day; fixinhand; need r=, sr=
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.5 → mozilla0.9.6

Comment 8

17 years ago
r=syd, and yes, I actually understand (after an hour lecture by Dr. J.)
(Assignee)

Comment 9

17 years ago
note to self:
kin suggests making util routine that does both GetPromotedRanges and
GetNodesForOperations.  I agree.
(Assignee)

Comment 10

17 years ago
fix checked in on trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 11

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