Indent command should use one BLOCKQUTOE+PRE instead of multiple PRE
Reported by
cyril.au...@gmail.com,
Jul 15 2016
|
|||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.8 Safari/537.36 Example URL: https://jsfiddle.net/crl/uem16cqf/1/ Steps to reproduce the problem: 1. https://jsfiddle.net/crl/uem16cqf/1/ 2. observe result on chrome, every line is a <pre></pre> after indenting 3. What is the expected behavior? <pre style="margin:40px">..</pre> FF does <blockquote><pre>..</pre></blockquote> What went wrong? <blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"> <pre>let openTab = li => {</pre> <pre> li.classList.add('selected');</pre> <pre> .... Does it occur on multiple sites: N/A Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? Yes Chrome version: 53.0.2785.8 Channel: dev OS Version: 10.0 Flash Version: Shockwave Flash 22.0 r0
,
Jul 25 2016
Both Edge and Firebox insert BLOCKQUOTE+PRE but Chrome inserts BLOCKQUOTE+multipe PRE
,
Jul 25 2016
,
Jul 25 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 31 2017
Repro with simple case:
<div contenteditable id="ce">
<pre>
Trigger try jobs by running webkit-patch rebaseline-cl. This should trigger jobs on tryserver.blink.
Wait for all try jobs to finish.
seline-cl. This should trigger jobs on tryserver.blink.
Wait for all try jobs to finish.
</pre>
</div>
<script>
ce.focus();
document.execCommand('selectAll');
document.execCommand('indent');
</script>
,
Jul 31 2017
,
Oct 4 2017
,
Oct 4
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 5
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by cbiesin...@chromium.org
, Jul 25 2016