Closed
Bug 386189
Opened 17 years ago
Closed 17 years ago
Cursor navigation by keyboard (arrow keys) does not work on designMode (including composer)
Categories
(Core :: DOM: Editor, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha8
People
(Reporter: jiha.bugzilla, Assigned: peterv)
References
()
Details
(Keywords: regression)
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jst
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a6pre) Gecko/20070628 Mnenhy/0.7.5.0 SeaMonkey/2.0a1pre
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a6pre) Gecko/20070628 Mnenhy/0.7.5.0 SeaMonkey/2.0a1pre
no navigation with $arrowkey or ctrl+$arrowkey
Reproducible: Always
Steps to Reproduce:
1. Open Composer
2. input some words/lines
3. try to use the arrow keys on edited text
Actual Results:
nothing happens. Cursor is not moved.
Expected Results:
Cursor should be moved word wise by using ctrl+$arrowkey or character wise by using $arrowkey.
bug appears with this build:
cvs checkout start: Do 2007-06-28 14:26:11 CEST (+0200) (should be 5:26 PDT)
it did not appear with that build:
cvs checkout start: Mi 2007-06-27 11:06:27 CEST (+0200)
Maybe this is fallout from bug 237964 for bug 386009
See http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-06-27+02%3A00%3A00&maxdate=2007-06-28+05%3A00%3A00&cvsroot=%2Fcvsroot
Assignee | ||
Updated•17 years ago
|
Blocks: contenteditable
Updated•17 years ago
|
Keywords: regression
Comment 1•17 years ago
|
||
I would bet it's due to bug 237964, due the bug 237964 comment 0, by Glazman.
Can confirm on a SeaMonkey build using the mail composer, as of some hours ago.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•17 years ago
|
||
(this is from bug 237964)
==> editor
Assignee: composer → nobody
Component: Composer → Editor
Product: Mozilla Application Suite → Core
QA Contact: editor
Version: unspecified → Trunk
Comment 3•17 years ago
|
||
Cursor navigation also no longer works in any vBulletin reply form, which uses designMode.
Updated•17 years ago
|
Flags: blocking1.9?
Copy & Paste also does not work.
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a6pre) Gecko/20070702 Minefield/3.0a6pre - Build ID: 2007070204
Comment 5•17 years ago
|
||
See URL for another testcase:
http://www.mister-pixel.com/ever/bugs/firefox/content_editable_testcase2.237964.html
Neither Control+C, Control+V, Home, End, PageUp or PageDown works.
Summary: Cursor navigation by keyboard (arrow keys) does not work in composer → Cursor navigation by keyboard (arrow keys) does not work on designMode (including composer)
Updated•17 years ago
|
Flags: blocking1.9? → blocking1.9+
Assignee | ||
Comment 8•17 years ago
|
||
Did this get fixed somehow?
With a trunk build from today, I go to http://www.mozilla.org/editor/midasdemo/ and enter a bunch of words. I can then navigate with the keyboard without problem, shortcuts for copy and paste work too.
Same with the vBulletin demo site (http://www.vbulletin.com/admindemo.php).
Same with message compose in a Thunderbird trunk build from today.
doesn't work for me with http://www.mozilla.org/editor/midasdemo/.
DOES work with the vbulletin demo.
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a7pre) Gecko/2007071304 Minefield/3.0a7pre - Build ID: 2007071304
Assignee | ||
Comment 10•17 years ago
|
||
Anyone seeing this and not on Linux?
Comment 11•17 years ago
|
||
Just did a fresh linux build after seeing your 12:11 posting. The vbulletin demo also wfm but the midasdemo does not. No change in mail/news compose.
Comment 12•17 years ago
|
||
Build identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9a7pre) Gecko/200707132020 SeaMonkey/2.0a1pre
Updated 2 hours ago... It does not work for me on either midas or Vbulletin demo.
Comment 13•17 years ago
|
||
WFM on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007071506 Minefield/3.0a7pre
Comment 14•17 years ago
|
||
I just compiled the trunk version of Thunderbird on Linux and the arrow keys do not work.
Here are the build options I used:
. $topsrcdir/mail/config/mozconfig
export MOZILLA_OFFICIAL=1
export BUILD_OFFICIAL=1
mk_add_options MOZILLA_OFFICIAL=1
mk_add_options BUILD_OFFICIAL=1
ac_add_options --enable-official-branding
mk_add_options MOZ_OBJDIR=@TOPSRCDIR@/thunderbird-bin
ac_add_options --enable-default-toolkit=cairo-gtk2
ac_add_options --with-user-appdir=.mozilla
#ac_add_options --with-system-png=/usr
ac_add_options --with-system-jpeg=/usr
ac_add_options --enable-postscript
ac_add_options --disable-installer
ac_add_options --disable-xprint
ac_add_options --enable-crypto
ac_add_options --enable-strip-libs
ac_add_options --enable-canvas
ac_add_options --enable-svg
ac_add_options --enable-mathml
ac_add_options --disable-tests
ac_add_options --disable-gtktest
ac_add_options --disable-debug
ac_add_options --enable-xft
ac_add_options --enable-optimize="-O4 -march=opteron -mtune=opteron -pipe -ftracer -fomit-frame-pointer -msse3"
ac_add_options --with-system-zlib=/usr
ac_add_options --without-system-nspr
ac_add_options --enable-xinerama
ac_add_options --enable-extensions=default
ac_add_options --disable-pedantic
ac_add_options --disable-long-long-warning
ac_add_options --enable-single-profile
ac_add_options --disable-profilesharing
ac_add_options --enable-gnomevfs
ac_add_options --disable-updater
Assignee | ||
Comment 16•17 years ago
|
||
We started focussing editable elements (contentEditable or the documentElement of editable documents), so we need to return the window's controllers for those elements in nsFocusController::GetControllers.
Assignee: nobody → peterv
Status: NEW → ASSIGNED
Attachment #272548 -
Flags: superreview?(jst)
Attachment #272548 -
Flags: review?(jst)
Comment 17•17 years ago
|
||
Perhaps I tried this prematurely. I patched my CVS pull from "checkout finish: Sat Jul 14 17:51:56 EDT 2007" after a distclean and received the following:
/mozilla/dom/src/base/nsFocusController.cpp: In member function 'virtual nsresult nsFocusController::GetControllers(nsIControllers**)':/mozilla/dom/src/base/nsFocusController.cpp:267: error: 'class nsDerivedSafe<nsIDOMElement>' has no member named 'IsEditable'
gmake[6]: *** [nsFocusController.o] Error 1
Comment 18•17 years ago
|
||
Comment on attachment 272548 [details] [diff] [review]
v1
>+ nsCOMPtr<nsIContent> content = do_QueryInterface(mCurrentElement);
I suspect that the rest of the |mCurrentElement| should be |content|. That compiles, but unforunately does not seem to help with this bug.
Assignee | ||
Comment 19•17 years ago
|
||
Forgot to diff again before attaching.
Attachment #272548 -
Attachment is obsolete: true
Attachment #272617 -
Flags: superreview?(jst)
Attachment #272617 -
Flags: review?(jst)
Attachment #272548 -
Flags: superreview?(jst)
Attachment #272548 -
Flags: review?(jst)
Assignee | ||
Comment 20•17 years ago
|
||
Comment on attachment 272617 [details] [diff] [review]
v1
Actually, this fails when mCurrentWindow is null.
Attachment #272617 -
Flags: superreview?(jst)
Attachment #272617 -
Flags: superreview-
Attachment #272617 -
Flags: review?(jst)
Attachment #272617 -
Flags: review-
Comment 21•17 years ago
|
||
nit: Peter, you should actually remove the flags instead of minusing it, since you was not the reviewer here.
Assignee | ||
Comment 22•17 years ago
|
||
This one works reliably for me.
Attachment #272617 -
Attachment is obsolete: true
Attachment #272645 -
Flags: superreview?(jst)
Attachment #272645 -
Flags: review?(jst)
Comment 23•17 years ago
|
||
Works here with patch v1.1 applied to CVS "checkout finish: Tue Jul 17 11:33:16 EDT 2007"
Updated•17 years ago
|
Attachment #272645 -
Flags: superreview?(jst)
Attachment #272645 -
Flags: superreview+
Attachment #272645 -
Flags: review?(jst)
Attachment #272645 -
Flags: review+
Comment 24•17 years ago
|
||
Thanks Peter, works great on linux for me.
Updated•17 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta1
Updated•17 years ago
|
Flags: in-testsuite?
Comment 25•17 years ago
|
||
I'm not sure this is the correct fix. By my LXRing the only caller of nsFocusController::GetControllers is nsXBLWindowKeyHandler and it looks to me as if that should prefer to call nsFocusController::GetControllerForCommand directly so that GetControllers could be made private to nsFocusController.
Comment 26•17 years ago
|
||
Attachment #274077 -
Flags: review?(jst)
Updated•17 years ago
|
Attachment #274077 -
Flags: superreview?(peterv)
Attachment #274077 -
Flags: review?(jst)
Attachment #274077 -
Flags: review+
Assignee | ||
Comment 27•17 years ago
|
||
Comment on attachment 274077 [details] [diff] [review]
Alternate approach
Neil: is this patch trying to fix something or is it just cleanup that's unrelated to the bug?
Comment 28•15 years ago
|
||
Comment on attachment 274077 [details] [diff] [review]
Alternate approach
Clearing stale request, question in bug never addressed.
Attachment #274077 -
Flags: superreview?(peterv)
You need to log in
before you can comment on or make changes to this bug.
Description
•