Bug #45456
Work Package #45020: Content Editing Stability and Features
HTML editor displays dynamically inserted content rather than the actual node's
| Status: | Closed | Start date: | 2013-02-13 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | - | % Done: | 0% |
|
| Category: | UI | |||
| Target version: | Base Distribution - 1.0 beta 1 |
Description
if you have an HTML content element which inner content is changed by some client side JavaScript, the HTML editor shows this changed content rather than the background node's.
You can say, the HTML editor always shows the current DOM node's content instead of the TYPO3CR Node's.
Related issues
History
#1 Updated by Aske Ertmann over 2 years ago
- Category set to UI
- Priority changed from Should have to Must have
- Target version set to 1.0 beta 1
- Parent task set to #45020
#2 Updated by Aske Ertmann over 2 years ago
Instead of getting the HTML source from the actual dom we should get it from the VIE entity instead.
#3 Updated by Aske Ertmann over 2 years ago
- Status changed from New to Accepted
#4 Updated by Christoph Dähne almost 2 years ago
How to trigger this bug?
Add two HTML-Content-Elements (requires Bugfix 24168 for typo3.neos):
<div id="myfunnydiv"> Hello World </div> This one is changed.
and afterwards
<p>This one changes</p>
<script type="text/javascript">
setInterval(_onTick, 1000);
function _onTick() {
$('#myfunnydiv').html(new Date().getTime());
}
</script>
Now try to edit the first one using the HTML-Editor. If it works, refresh the page and try again.
#5 Updated by Christoph Dähne almost 2 years ago
NOTE: In the above example the interval is started multiple times if the content area is reloaded. Reloading the whole page discards the additional intervals. If the creation of intervals, … in inline java script in html content elements shall be fully supported, might be worth some discussion.
#6 Updated by Gerrit Code Review almost 2 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24183
#7 Updated by Gerrit Code Review almost 2 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24183
#8 Updated by Adrian Föder over 1 year ago
- Status changed from Under Review to Resolved
#9 Updated by Aske Ertmann 9 months ago
- Status changed from Resolved to Closed