-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Resize bug #100
Comments
Morning @adriendst, Thank you so much for reporting the issues. I knew about the first one, not the second. I honestly don't know when I will free up time to look into those. |
Okay, let me know when you will get into it ! I did some adjustment in my own code to avoid those bugs for the moment |
I've also experienced both of these issues while working on this PR in a separate repo. It is worth adding that there seems to be a discrepancy between what happens in Chromium- and Firefox-based browsers, which I and the other repo's maintainer have discussed more in the PR I linked. |
Identified the issue. it's a regression bug. may have to revert passive events. anyways, stick to v0.8.2 for now |
found the bug. expect a release soon |
I believe this bug is resolved in version 8.0.6. |
I'll try it, thanks! By the way, are you intentionally sticking with the 8 major versioning? |
The first issue seems to be fixed, but I'm still having an issue which most closely resembles the second issue. I'm not entirely sure what's causing it- as far as I understand, I'm not using any of the sizing parameters in any extraordinary ways- but if I set the initial size at all, then:
|
After a bit more testing, this seems to actually be an issue with reactivity. I'll try and fix the issue in the PR and hopefully I'll then have a better idea of exactly what it is. |
Good morning, Regarding the version, I am sticking to 8.x.x. I made a mistake a few weeks ago, and it was too late to withdraw the package from npm. I plan to port to Svelte 5 as 9.x.x. Regarding the exception, those are good tips. I will investigate. A REPL to reproduce the error would be helpful. Best regards, PS: size should not be null. set it to 0 |
Seems to be going through this code.
considering introducing a |
plz test the latest code changes in master - not release as a version yet. |
Oh sorry, been busy lately because I've just started uni. I'll try and test it ASAP! |
Ok, I tried creating a REPL for this using StackBlitz and now I'm even more confused about the issue. I first tried with your changes in master (repl) where:
But then, to make sure that these differences were because of your changes, I also made a REPL based on the exact code I've been testing with from the svelte-mosaic PR (repl) where:
So somehow, I am now able to reproduce 5 different types of behaviour:
Leodog896 has the same issues as me when running it locally, so I'm confident that this is not just something weird with my specific setup. My first guess as to what could cause the difference between running it locally and on StackBlitz was node versions, since I use Node 20 locally and StackBlitz only supports Node 18, however the difference between viewing it in-editor and in a separate tab makes me think it's more likely to be related to StackBlitz's WebContainers. More local testing could be done to narrow down what is causing these differences (running GitHub master version locally, running both versions locally using Node 18, setting up WebContainers locally) but, with there being so many differences to test now, I'm not really sure where to start, and I'm hoping someone else will have a better idea from the information I've already found. |
First and foremost, thank you for setting up the test.s
I will need a bit more time to identify the root cause, and I plan to release the changes in master as version 8.0.7 for this purpose. Will keep you posted. |
Released 8.0.7 - didn't make any significative changes other than releasing the code from master. Tried it against your second REPL. However, there is an error on the console: TypeError: Failed to construct 'URL': Invalid URL, which might be coming from code. |
Oh, those are more likely to be related to StackBlitz itself; check the browser console with it open in a new tab and you should see the console actually for the svelte-mosaic example page |
Not sure where that IFRAME is coming from, but it is completely blank and deleting it does not make a difference, so I don't think this is the issue |
ok.. I see it now. weird.
No idea what Starblitz is doing, |
StackBlitz has a "Download as ZIP" button at the top of the explorer pane, but the easier way to test locally would be to just clone the svelte-mosaic PR https://github.com/hopperelec/svelte-mosaic/tree/splitpanes |
the git repo above is the bundled version, wont' work trying to download, |
What do you mean by "the bundled version"? |
bundled - you are not using splipanes as a dependency. i don't want to debug code bundling. I can't download this damn project, I hate starblitz |
I am? It is using version 0.8.2 right now, though. The only place I used it bundled was in the first StackBlitz REPL; the git repo I linked to does not do that |
It looks like you cloned the master branch, not the splitpanes branch I linked to. svelte-mosaic used to handle the panes on its own, but the PR I created is to migrate it to using splitpanes. The PR hasn't been merged yet because of the issues, so it still lives on a separate branch.
|
my bad, got it now. |
The issue has been identified, and version 8.0.8 is being released. Thank you for highlighting this issue. The root cause remains unclear, but the problem has been resolved. Please try, confirm and close this ticket. |
Thanks so much for looking into the issue! Sadly, it still doesn't seem to be fully working for me- it is very similar to 8.0.6 However, in Firefox, the initial size now gets precise if the container is resized, which wasn't the case before. To re-iterate for clarity:
I should also add for full clarity that, in both browsers, it shows as 50% width very briefly before going to the initial size I mentioned. This has been the case the whole time and I assume is just part of the rendering process. I mention this since you mentioned the rendering "flickering" earlier. |
Morning, I changed your code as such -
my point is, there is no way the component can render proper if the size is undefined. am I missing something? |
Undefined should be the default value, meaning it should invoke the default behaviour, no? I have used undefined for panes where a specific size is not provided. |
Hello, I am using the library and I noticed some malfunction. I don't know if they are already known, so I wanted to share them to you.
The first one is :
bug.splitpane.1.mp4
In that case, when the top pane reach his minSize, he can still be minimized a little bit. Moreover, the when the top pane is smaller than his minSize, the bottom pane size reduce too. In the DOM we can see that the combined height of the three panes are over 100%.
The second one is :
bug.splitpane.2.mp4
In that case, when the first pane reach the end of the splitPanes component, both the first and last pane shrinks. The combined width of the three panes are less than 100%.
I was wondering if you could look at them ? Don't hesitate to ask me for more information :) (both the videos have been recorded from https://orefalo.github.io/svelte-splitpanes/ )
The text was updated successfully, but these errors were encountered: