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

Slintpad fixes #6575

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

hunger
Copy link
Member

@hunger hunger commented Oct 16, 2024

Fix loading of additional slint source files as well as of urls in @image-url.

This makes the current slintpad render the printerdemo just as slintpad.com does... which still looks like it is missing the tab below the icons. Was that changed?

Do not extract the file loading into a fuinction: That keeps 
`self` around which is part of a borrowed state. That is a
problem as during the load we end up in `ensure_document_loaded`
again, which will try to borrow_mut that same state.
Slintpad uses URLs to images. Do not fail when we "embed"
those so that we find the list of resources on the Documents
later.

This fixes image loading in slintpad again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant