Bug 66296 - When selected a text witn't delete it with 'delete' nor 'backspace' nor any other character to overwrite it.
Summary: When selected a text witn't delete it with 'delete' nor 'backspace' nor any ...
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml forms (other bugs)
Version First Reported In: unspecified
Platform: Compiled Sources Linux
: NOR grave
Target Milestone: ---
Assignee: Konqueror Bugs
URL:
Keywords:
: 66872 68272 69823 70027 70303 71120 72096 72244 72763 73248 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-10-20 20:58 UTC by Engin AYDOGAN
Modified: 2004-02-01 12:00 UTC (History)
10 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Engin AYDOGAN 2003-10-20 20:58:41 UTC
Version:           KDE: 3.1.92 (CVS >= 20031019) (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc version 3.2.2 20030222 (Red Hat Linux 3.2.2-5) CFLAGS=-march=pentium4 -O3 -pipe -fomit-frame-pointer
OS:          Linux

When you select a text with your mouse ( eg. double clicking on a word, selecting by drag-n-drop ) you can't modify it. For instance when you hit 'backspace' or 'delete' key after selecting the text, the text won't be deleted. Also when you hit any key the selection won't be overwritten, nothing happends. This does not apply to the texts selected by keybord -- for instance by ctrl+shift+right arrow.
Comment 1 Stephan Binner 2003-10-20 22:03:00 UTC
You're talking about web pages and wonder that you cannot edit them? :-)
Comment 2 Engin AYDOGAN 2003-10-20 22:09:32 UTC
I'm sorry I need to clarify this.

I'm talking about text fields on web pages (<input> ). Textareas are fine.
Comment 3 Stephan Binner 2003-10-23 15:16:10 UTC
Works for me with current CVS, can you confirm?
Comment 4 Engin AYDOGAN 2003-10-23 19:31:29 UTC
Yes seems ok now. 
Comment 5 Maksim Orlovich 2003-10-30 02:55:01 UTC
*** Bug 66872 has been marked as a duplicate of this bug. ***
Comment 6 Teemu Rytilahti 2003-11-06 13:51:16 UTC
Well, this works usually, but not always..

For example, try to report a new bug.. Press Continue when you're asked about duplicate reports.. And then there's input named "shortDesc", try to edit/backspace/whatever it.. Doesn't work here.
Comment 7 Sean Lynch 2003-11-13 06:07:55 UTC
I'm also noticing this.  Seems to only occur when you type in text, click outside the input field, then hightlight the text that you typed.  BTW, I'm running CVS 031112 and it still seems to be present.
Comment 8 Engin AYDOGAN 2003-11-13 07:52:07 UTC
Yes, this bug is seems to be present again. But I couldn't find a certain case to regenerate this bug. I'll post it if I can.
Comment 9 Maksim Orlovich 2003-11-15 14:50:29 UTC
*** Bug 68272 has been marked as a duplicate of this bug. ***
Comment 10 Stephan Kulow 2003-12-06 18:30:57 UTC
I think it's fixed. Please reopen if you got concrete step to step how to reproduce with HEAD
Comment 11 Maksim Orlovich 2003-12-08 00:29:03 UTC
*** Bug 69823 has been marked as a duplicate of this bug. ***
Comment 12 Engin AYDOGAN 2003-12-08 00:34:39 UTC
I've updated my kdelibs now and compiled. This bug is still present unfortunetly. It's hard to give a test case. It sometimes happens.
Comment 13 Maksim Orlovich 2003-12-13 17:26:26 UTC
*** Bug 70027 has been marked as a duplicate of this bug. ***
Comment 14 Maksim Orlovich 2003-12-13 17:26:49 UTC
*** Bug 70303 has been marked as a duplicate of this bug. ***
Comment 15 Maksim Orlovich 2003-12-13 17:27:09 UTC
Still happens sometimes
Comment 16 Stephan Binner 2003-12-13 17:36:28 UTC
Always reproducible testcase: Alt+F2 "leo:testcase" Return. Works fine when the page is reloaded or the search term is entered in the URL bar of a running Konqueror window.
Comment 17 Jesper Juhl 2003-12-14 22:51:07 UTC
I can reliably reproduce this with KDE 3.2Beta2 using the procedure described in Bug 69823
Comment 18 Jarl Gjessing 2004-01-03 01:12:15 UTC
I noticed this error while developing some webpages that had inputs fields like text.
I assumed it was because I had defined a maxlen and that since the maxlen had been used by the inputed text, the input from the keyboard was ignored when the text was marked.
It is "solved" now, since Konqueror now ignores maxlen!!!
I have just tried the pages that I had problems with some days ago. Just before Christmas. I'm 100% sure that I could not mark the text and then enter new text if maxlen was fulfilled, but now I can and I can enter just as many characters as I want to. Konqueror completely ingores maxlen
Comment 19 Maksim Orlovich 2004-01-09 17:13:58 UTC
*** Bug 72096 has been marked as a duplicate of this bug. ***
Comment 20 Maksim Orlovich 2004-01-09 17:14:15 UTC
*** Bug 72244 has been marked as a duplicate of this bug. ***
Comment 21 Maksim Orlovich 2004-01-11 19:04:45 UTC
*** Bug 71120 has been marked as a duplicate of this bug. ***
Comment 22 András Manţia 2004-01-16 15:47:50 UTC
*** Bug 72763 has been marked as a duplicate of this bug. ***
Comment 23 András Manţia 2004-01-16 16:26:45 UTC
Subject: 
Just a note, that it is still reproducable, see the #72763 duplicate.

Comment 24 Jesper Juhl 2004-01-20 23:05:39 UTC
This bug is still very much alive and is greatly annoying. Around 1 out of 4 times when I highlight some text, with the mouse, in an input field with the intention to type some new text to replace it or press delete or backspace to remove it nothing happens.
You have to go and place the cursor in the input field so no text is highlighted and then use backspace or delete to remove it before you can type in the replacement.
This is with KDE 3.2RC1 , but 3.2Beta2 had the same problem. Never saw this issue with KDE 3.1.x
Comment 25 Leo Savernik 2004-01-21 22:49:24 UTC
Subject: kdelibs/khtml

CVS commit by savernik: 

ensure that embedded widgets don't lose their focus if there's no suitable editable element.

CCMAIL: 66296-done@bugs.kde.org


  M +11 -1     khtmlview.cpp   1.610


--- kdelibs/khtml/khtmlview.cpp  #1.609:1.610
@@ -2368,6 +2368,16 @@ void KHTMLView::ensureNodeHasFocus(NodeI
   if (!node) firstAncestor = 0;
 
+  DocumentImpl *doc = m_part->xmlDocImpl();
+  // ensure that embedded widgets don't lose their focus
+  if (!firstAncestor && doc->focusNode() && doc->focusNode()->renderer()
+        && doc->focusNode()->renderer()->isWidget())
+    return;
+
   // Set focus node on the document
-  m_part->xmlDocImpl()->setFocusNode(firstAncestor);
+#if DEBUG_CARETMODE > 1
+  kdDebug(6200) << k_funcinfo << "firstAncestor " << firstAncestor << ": "
+        << (firstAncestor ? firstAncestor->nodeName().string() : QString::null) << endl;
+#endif
+  doc->setFocusNode(firstAncestor);
   emit m_part->nodeActivated(Node(firstAncestor));
 }


Comment 26 Jesper Juhl 2004-01-21 23:00:10 UTC
Great to see a fix go in. This one caused me a lot of grief.
I'm wondering (since I have not been able to test this fix yet), if Bug #70447 is related and possibly fixed by this as well?
Comment 27 Stephan Binner 2004-01-22 17:22:29 UTC
*** Bug 73248 has been marked as a duplicate of this bug. ***
Comment 28 Tomas 2004-02-01 12:00:15 UTC
Agree, bug still present in KDE3.2 RC1
See #72096

Didn't try to apply the patch from Leo

Tomas