Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Live Preview : Duplicates elements on copy/paste of entire document #10634

Open
gideonthomas opened this issue Feb 19, 2015 · 3 comments
Open

Live Preview : Duplicates elements on copy/paste of entire document #10634

gideonthomas opened this issue Feb 19, 2015 · 3 comments
Labels
F Live Preview MultiBrowser F Live Preview

Comments

@gideonthomas
Copy link

Hi,

It seems like when you copy and paste an entire html document into the editor (that already has content), DOM elements in the body are duplicated in the Live Preview. This happens regardless of whether the new body content you paste in is the same or different.

I am running the latest version of Brackets on Mac OS X Mavericks.

Here's a screencap of it:
livepreviewfail

Thanks :)
@gideonthomas
Copy link
Author

I dug into the code a bit and I think I tracked it down to this function

This statement results in null because the new <html> tag gets a new tagID which cannot be mapped to a corresponding <html> tag in the old DOM structure.

So, it fails the if(oldElement) check and executes this. In the function generateChildEdits, it fails this condition, not entering the loop where the delete/insert operation (delete the old html tag with the old tagID and insert the new one) happens. It instead only does the insert operation.

Seems like the problem comes from passing in null as the first parameter into generateChildEdits. But not sure how to fix it though.

@humphd humphd mentioned this issue Feb 19, 2015
Closed
@peterflynn
Copy link
Member

Possibly related to #5336

xyin96 added a commit to xyin96/brackets that referenced this issue Feb 25, 2015
@xyin96
for adobe#10634
b79bdd0
xyin96 added a commit to xyin96/brackets that referenced this issue Feb 25, 2015
@xyin96
for adobe#10634
a7736fe
xyin96 added a commit to xyin96/brackets that referenced this issue Feb 25, 2015
@xyin96
for adobe#10634
d7111b5
@xyin96 xyin96 mentioned this issue Feb 25, 2015
Open
@gideonthomas gideonthomas mentioned this issue Feb 25, 2015
Closed
gideonthomas pushed a commit to gideonthomas/brackets that referenced this issue Mar 13, 2015
@xyin96 @gideonthomas
for adobe#10634
e951e1e
gideonthomas pushed a commit to gideonthomas/brackets that referenced this issue Mar 13, 2015
@xyin96 @gideonthomas
for adobe#10634
aee4a73
gideonthomas pushed a commit to gideonthomas/brackets that referenced this issue Mar 13, 2015
@xyin96 @gideonthomas
for adobe#10634
eee9cfa
gideonthomas added a commit to mozilla/brackets that referenced this issue Mar 13, 2015
@gideonthomas
Fix #44 - Landing upstream patch adobe#10653 to fix adobe#10634
a110763
gideonthomas added a commit to gideonthomas/brackets that referenced this issue Mar 13, 2015
@gideonthomas
Fix adobe#44 - Landing upstream patch adobe#10653 to fix adobe#10634
53c578e
humphd pushed a commit to mozilla/brackets that referenced this issue Mar 18, 2015
@xyin96 @humphd
for adobe#10634
d72b612
humphd pushed a commit to mozilla/brackets that referenced this issue Mar 18, 2015
@xyin96 @humphd
for adobe#10634
f99cac9
humphd pushed a commit to mozilla/brackets that referenced this issue Mar 18, 2015
@xyin96 @humphd
for adobe#10634
3603319
@redmunds redmunds mentioned this issue Mar 27, 2015
Closed
@busykai busykai mentioned this issue Apr 1, 2015
Closed
humphd added a commit to mozilla/brackets that referenced this issue Apr 3, 2015
@humphd
Revert "for adobe#10634"
5b9d47a 
This reverts commit 3603319.
humphd added a commit to mozilla/brackets that referenced this issue Apr 3, 2015
@humphd
Revert "for adobe#10634"
d34f553 
This reverts commit f99cac9.
humphd added a commit to mozilla/brackets that referenced this issue Apr 3, 2015
@humphd
Revert "for adobe#10634"
cfdbcba 
This reverts commit d72b612.
humphd pushed a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@xyin96 @humphd
for adobe#10634
e7b958d
humphd pushed a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@xyin96 @humphd
for adobe#10634
3f55462
humphd pushed a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@xyin96 @humphd
for adobe#10634
a36caab
humphd added a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@humphd
Revert "for adobe#10634"
74ee9c3 
This reverts commit 3603319.
humphd added a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@humphd
Revert "for adobe#10634"
7e1ac03 
This reverts commit f99cac9.
humphd added a commit to mozilla/brackets that referenced this issue Apr 15, 2015
@humphd
Revert "for adobe#10634"
25f13c6 
This reverts commit d72b612.
@busykai busykai mentioned this issue Apr 17, 2015
Closed
humphd pushed a commit to mozilla/brackets that referenced this issue May 15, 2015
@xyin96 @humphd
for adobe#10634
e32e713
humphd pushed a commit to mozilla/brackets that referenced this issue May 15, 2015
@xyin96 @humphd
for adobe#10634
9fb5fc3
humphd pushed a commit to mozilla/brackets that referenced this issue May 15, 2015
@xyin96 @humphd
for adobe#10634
2931718
humphd added a commit to mozilla/brackets that referenced this issue May 15, 2015
@humphd
Revert "for adobe#10634"
0c3fea0 
This reverts commit 3603319.
humphd added a commit to mozilla/brackets that referenced this issue May 15, 2015
@humphd
Revert "for adobe#10634"
e3a1477 
This reverts commit f99cac9.
humphd added a commit to mozilla/brackets that referenced this issue May 15, 2015
@humphd
Revert "for adobe#10634"
a78a016 
This reverts commit d72b612.
humphd pushed a commit to mozilla/brackets that referenced this issue May 20, 2015
@xyin96 @humphd
for adobe#10634
e7b1551
humphd pushed a commit to mozilla/brackets that referenced this issue May 20, 2015
@xyin96 @humphd
for adobe#10634
1ec58f0
humphd pushed a commit to mozilla/brackets that referenced this issue May 20, 2015
@xyin96 @humphd
for adobe#10634
c656002
humphd added a commit to mozilla/brackets that referenced this issue May 20, 2015
@humphd
Revert "for adobe#10634"
b087ab0 
This reverts commit 3603319.
humphd added a commit to mozilla/brackets that referenced this issue May 20, 2015
@humphd
Revert "for adobe#10634"
36385d2 
This reverts commit f99cac9.
humphd added a commit to mozilla/brackets that referenced this issue May 20, 2015
@humphd
Revert "for adobe#10634"
da23e06 
This reverts commit d72b612.
humphd pushed a commit to mozilla/brackets that referenced this issue May 26, 2015
@xyin96 @humphd
for adobe#10634
a7a8d48
humphd pushed a commit to mozilla/brackets that referenced this issue May 26, 2015
@xyin96 @humphd
for adobe#10634
227b777
humphd pushed a commit to mozilla/brackets that referenced this issue May 26, 2015
@xyin96 @humphd
for adobe#10634
38e52a8
humphd added a commit to mozilla/brackets that referenced this issue May 26, 2015
@humphd
Revert "for adobe#10634"
479922c 
This reverts commit 3603319.
humphd added a commit to mozilla/brackets that referenced this issue May 26, 2015
@humphd
Revert "for adobe#10634"
107064f 
This reverts commit f99cac9.
humphd added a commit to mozilla/brackets that referenced this issue May 26, 2015
@humphd
Revert "for adobe#10634"
81c945e 
This reverts commit d72b612.
This was referenced Nov 1, 2015
Closed
Open
@Denisov21 Denisov21 mentioned this issue Nov 5, 2015
Closed
@MohabAli
Copy link

solved ... just disable (StaticServer)
go to ... Extension manager >> Default >> disable (StaticServer)
and use an alternative (Autosave) extension then enable (auto save (400 ms))
it will be in (File) after installation
https://github.com/nmaxcom/brackets-autosaver
run (live preview) enjoy

This was referenced Aug 31, 2021
Open
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
F Live Preview MultiBrowser F Live Preview
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@peterflynn @busykai @gideonthomas @MohabAli