New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chrome aw-snap [works on version 4.3.7] #354
Comments
I have tried every version all the way down to 4.3.7 and is the last release where it works without this crash. originally believed it was the vue-filepond #101 |
Does this happen with the plain JavaScript FilePond version (not running in Vue)? This could have to do with the MutationObserver added to the Vue wrapper (but that would not explain why it doesn't occur with max files of 5). |
Im using latest vue-filepond, with version 4.3.7 of this repo, its working good. The accordian is a vue component too, if thats a case of memory usage or leaking idk, never tried the non-vue version. Working with any amount of files again :) |
Can you try with latest vue-filepond version? |
"filepond": "4.4.12", "filepond": "4.3.7", |
So 4.3.8 causes the issue? Or one of the versions in between? |
4.3.8 has the crash and everything above :O, thanks for the response. |
Might have something to do with ? I have tested it with this, maybe I used it wrong, still dosn't explain why only crashes later then then version 4.3.7 |
@marky291 I'm not sure, if you know the amount of instances you're creating then this sounds like a good idea instead of constantly destroying and creating new ones. |
@marky291 Is this still an issue? |
Will update to latest shortly and test |
Works on latest version Much appreciated support :) |
Fantastic, glad to hear that! I'll close the issue then. |
Sorry, but i still got this error with all versions in plain javascript when opening modal or using bootstrap collapse. Even when i open the element inspector in chrome and hover over the html it crashes. when removing filpond application works normally and never crashes. i tested 4.6.1 and latest 4.9.2 |
@mverstegen Is the FilePond instance created when the collapse opens? and then destroyed when it closes? |
@mverstegen Please create a public test case on codesandbox.io, that makes it easier to debug. |
I can't create a full sandbox for this, it goes wrong in my internal application, maybe we can do a screen sharing session?
Verzonden door BlueMail
Op 17 dec. 2019 10:08, om 10:08, Rik <notifications@github.com> schreef:
…
@mverstegen Please create a public test case on codesandbox.io, that
makes it easier to debug.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#354 (comment)
|
@mverstegen I really need to a public test case to debug this. |
Same problem here ✋ Using Collapse Bootstrap crashes in chrome My config "filepond": "^4.6.1", obs: Safari and Firefox not reproduce the crash |
My setup works with. "filepond": "^4.6.1", Try back date each version and test your react to see which works. |
Hi, Same problem here. Chrome 79 crashes a few seconds after page is loaded, but at least on firefox works. I have these versions: |
Hope someone has fixed this issue in the meanwhile and want to share his fix here. |
@webjohan created a demo here, and has a quick fix, any ideas/suggetsions are welcome. |
So far I've only managed to prevent the "aw snap" by hiding the Note that this is not a fix, it's simply a debug result Hiding the |
I added .filepond--root .filepond--drop-label{ display: none } to my css. .filepond--root *:not(text) { it shows same behaviour as before when i do something (open/close) with the collapse or hover over inspector code, whatever. so for me it's even no quick fix. sorry. |
I think the drop-label is just the final thing in this perfect storm. |
So back on a large screen again. It seems I was wrong, giving the input elements unique names doesn't stop chrome from crashing. |
@webjohan The slide animation updates the I've forked the codepen here: https://codepen.io/rikschennink/full/WNbPQLj It now animates every second. I can reproduce the issue consistently by trying to inspect the page: Note that yesterday it took a bit more time, inspecting the body element worked, but then when hovering over elements in the inspector it would crash. |
One of my colleges ran chrome with logging and there was a GPU Cache error message which seems resonable when we're dealing with this many redraws. Still chrome should not crash. |
@webjohan I think in your codepen this automatically happens because of all the clicks on the link. |
I use the vanilla JS version 4.9.4 and it crashes with collapsible panels from Bootstrap. I was seemingly able to fix the crashing by adding this CSS: .panel-collapse.collapse .filepond--root,
.panel-collapse.collapsing .filepond--root {
display: none;
}
.panel-collapse.collapse.in .filepond--root {
display: block;
} Not 100% sure though, the crashing is a bit inconsistent for me, but I haven't had a crash when the filepond input remain the same height. |
Filed an issue with the Chromium team: https://bugs.chromium.org/p/chromium/issues/detail?id=1044895 Not sure if "staring" the issue will help up the priority but ... |
Alright, this is not ideal, but seems to be a temporary fix. I tried setting different properties on the label container element but while setting This seems to work. Wrap your FilePond in an additional element with a fixed height and an overflow hidden style. See below for example applied to the @webjohan codepen, please confirm if this resolves the issue on your environment. |
@rikschennink this prevents chrome from crashing with my machine on codepen. |
Update: We might have discovered two Chrome bugs here. Looks like the chromium team is working on a solution. |
@webjohan nice! Good stuff |
Good work guys! @webjohan in special! |
@mverstegen Glad to hear that, thanks for confirming 😊 |
Great news! The bug has been fixed in Chrome Canary 81.0.4042.0 and above The Chromium team did a fantastic job! |
@webjohan Waw! That is fast, didn't expect that to be honest. :-) |
@rikschennink I think they might have been working on a related issue for some time, but if not I'm super impressed ... still am though |
will close as is chrome bug |
This is great that chrome is updating it's code to fix the issue, but what if the user has an outdated version of chrome, is there a workaround to fix this? Is the solution provided by "rikschennink commented on Jan 24" can be considered as a workaround? |
@boulepick as it solves the issue, I believe so. |
thank you |
I am building a web service where a user can upload and configure a 'card' which inherits a live preview of the vue instance as you modify the configurator component, the uploading works great throughout however I have narrowed it down to a total of 5 images allowed, where anything more will crash chrome on accordian tab change, here is a preview of the tab.
A preview of the uploading
Once complete we can swap tabs
Swapping tab back to the upload images accordian tab, crashes after showing the content!
If the max-files setting is set to 5, it works fine with no crashing, it only happens when 5+ images are used, which leads me to believe it could be a resulting issue with filepond
Safari does not produce a crash.
The text was updated successfully, but these errors were encountered: