Re: [webkit-dev] Edit mode bug inserting <br> ?

6 Jan 2009

      Did you try the WebKit nightly to see if the issue is fixed there?

Dave

________________________________
From: Diego Taylor <diego.taylor@gmail.com>
To: David Kilzer <ddkilzer@webkit.org>
Sent: Tuesday, January 6, 2009 5:35:00 AM
Subject: Re: [webkit-dev] Edit mode bug inserting <br> ?

Hi Dave,

The Safari where I am testing is: "Version 3.2.1 (5525.27.1)"
Is there any workaround for it?

Thank You,
Diego

On Tue, Jan 6, 2009 at 1:25 AM, David Kilzer <ddkilzer@webkit.org> wrote:

Which version of Safari did you try?

There have been some HTML editing fixes recently in WebKit.  Have you tried a nightly WebKit build?  <http://nightly.webkit.org/>

Dave

________________________________
 From: Diego Taylor <diego.taylor@gmail.com>
To: webkit-dev@lists.webkit.org
Sent: Monday, January 5, 2009 1:49:23 PM
Subject: [webkit-dev] Edit mode bug inserting <br> ?

Hi,

I am intercepting the return key in edit mode and adding a <br> in the cursor position, but the first time I press the return key at the end of the text it doesn't move the cursor after the inserted br. Any help is really appreciated, this is the source code (tested on Safari):

---------- o ----------
<html>
    <head>
      <meta http-equiv="content-type" content="text/html; charset=utf-8" />
   </head>
   <body>
      <script language='javascript' type='text/javascript'>
         document.body.contentEditable = true;
         document.designMode = 'on';
         function onKeyPress(ev) {
            console.log('Key Pressed');
            if(ev.keyCode==13) {
               insertBR();
               ev.preventDefault();
            }
         }
         document.addEventListener('keypress', onKeyPress);
         function insertBR()
         {
            var s = window.getSelection();
            var r = s.getRangeAt(0);

            var br = document.createElement('br');
            r.insertNode(br);
            var newRange = document.createRange();
            newRange.setStartAfter(br);
            newRange.setEndAfter(br);

            s.removeAllRanges();
            r.detach();
            s.addRange(newRange);
         }
      </script>
   <div>Here is the place to play.</div>
   </body>
</html>
---------- o ----------

Thanks,
Diego