Skip to content
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

Panic in the preview #4081

Closed
Adancurusul opened this issue Dec 5, 2023 · 9 comments
Closed

Panic in the preview #4081

Adancurusul opened this issue Dec 5, 2023 · 9 comments
Labels
a:live preview The Live preview of slintpad or the LSP (mT,bO) bug Something isn't working

Comments

@Adancurusul
Copy link

Here 's the output

thread 'main' panicked at D:\a\slint\slint\internal\core\properties.rs:486:9:
Recursion detected
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[Info  - 3:33:01 PM] Connection to server got closed. Server will restart.
true
The Slint Language Server crashed. This is a bug.
Please open an issue on https://github.com/slint-ui/slint/issues
@Adancurusul
Copy link
Author

System: Windows 10
Software: vscode
Extension: Slint v1.3.2

@FloVanGH
Copy link
Member

FloVanGH commented Dec 5, 2023

Hi and thank you for your report.

Can you provide a code example?

@FloVanGH FloVanGH added the a:lsp slint-lsp and formating (mO,bT) label Dec 5, 2023
@Adancurusul
Copy link
Author

Hi and thank you for your report.

Can you provide a code example?

Code is in https://github.com/Adancurusul/slint_bug
By the way, thanks for the "coop_widget" , it works great!

@FloVanGH
Copy link
Member

FloVanGH commented Dec 5, 2023

By the way, thanks for the "coop_widget" , it works great!

I'm glade you like it.

Can you shortly explain the steps to reproduce the error?

@Adancurusul
Copy link
Author

I just click the “Show Preview” in Vscode
image
image
And then the question arose.
image
image

@FloVanGH
Copy link
Member

FloVanGH commented Dec 5, 2023

Thank you. I can reproduce it. At @hunger can you take a look?

@hunger
Copy link
Member

hunger commented Dec 5, 2023

I can reproduce the issue and this is the backtrace I see:

thread 'main' panicked at internal/core/properties.rs:486:9:
Recursion detected
stack backtrace:
   0: rust_begin_unwind
             at /rustc/87e1447aadaa2899ff6ccabe1fa669eb50fb60a1/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/87e1447aadaa2899ff6ccabe1fa669eb50fb60a1/library/core/src/panicking.rs:72:14
   2: i_slint_core::properties::PropertyHandle::access
             at /var/home/tobias/src/slint/master/internal/core/properties.rs:486:9
   3: i_slint_core::properties::PropertyHandle::update
             at /var/home/tobias/src/slint/master/internal/core/properties.rs:586:22
   4: i_slint_core::properties::Property<T>::get
             at /var/home/tobias/src/slint/master/internal/core/properties.rs:792:18
   5: <field_offset::FieldOffset<Item,i_slint_core::properties::Property<T>,field_offset::AllowPin> as i_slint_core::rtti::PropertyInfo<Item,Value>>::get
             at /var/home/tobias/src/slint/master/internal/core/rtti.rs:119:9
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Looks like yet another issue with the ComponentContainer needing information from the embedded component:-(

@ogoffart
Copy link
Member

ogoffart commented Dec 8, 2023

Is it the same as #3989 ?

(That the layout constraints depends on the window size)

@ogoffart ogoffart changed the title bug of lsp Panic in the preview Dec 18, 2023
@ogoffart ogoffart added the bug Something isn't working label Dec 18, 2023
@ogoffart ogoffart added a:live preview The Live preview of slintpad or the LSP (mT,bO) and removed a:lsp slint-lsp and formating (mO,bT) labels Jan 15, 2024
@ogoffart
Copy link
Member

This should be fixed in the nightly version of the extension (same as #3989)
Please re-open if the problem persist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:live preview The Live preview of slintpad or the LSP (mT,bO) bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants