Opened 16 years ago

Closed 15 years ago

#4196 closed Bug (fixed)

[chrome] List gets destoryed after repetitive type changes

Reported by: Tobiasz Cudnik Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Lists Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

In chromium, list gets destoryed after repetitive type changes. Refer to attached video TC.

Find this issue on GitHub

Change History (7)

Changed 16 years ago by Tobiasz Cudnik

Attachment: 4196.ogg added

comment:1 Changed 15 years ago by Garry Yao

Component: GeneralCore : Lists
Keywords: Confirmed added; webkit removed
Version: SVN (CKEditor)

With the help from Tobias, I was managed to find an even easier way to reproduce:

  1. Load the following content with selection:
    <ul>
    	<li>
    		a
    		<ol>
    			<li>
    				b^
    				<ul>
    					<li>
    						c</li>
    				</ul>
    			</li>
    		</ol>
    	</li>
    </ul>
    
    
  2. Press 'Numbered List' button and check the DOM snapshot;
  • Expected Result:
    <ul>
    	<li>
    		a</li>
    	<li>
    		b
    		<ul>
    			<li>
    				c</li>
    		</ul>
    	</li>
    </ul>
    
  • Actual Result:
    <ul>
    	<li>
    		a
            </li>
    	<li>
    		b
    	</li>
    		<ul>
    			<li>
    				c</li>
    		</ul>
    </ul>
    

So it's a general list bug instead of a webkit specific one.

comment:2 Changed 15 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

I've expected the wrong output on previous comment, the correct result should be what we have in v2:

<ul>
	<li>
		a</li>
</ul>
<p>
	b</p>
<ul>
	<li>
		c</li>
</ul>

comment:3 Changed 15 years ago by Garry Yao

I hope it make sense also in the patch to trim text before <br>.

Changed 15 years ago by Garry Yao

Attachment: 4196.patch added

comment:4 Changed 15 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:5 Changed 15 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Ticket Test added at :
http://ckeditor.t/tt/4196/1.html.
Fixed with [4523].

Find this issue on GitHub
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy