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

ScrollView stops working after exporting its content to png using export_to_png #3589

Closed
wzet opened this issue Aug 19, 2015 · 2 comments · Fixed by #4121
Closed

ScrollView stops working after exporting its content to png using export_to_png #3589

wzet opened this issue Aug 19, 2015 · 2 comments · Fixed by #4121
Labels
Component: graphics kivy/graphics Status: Confirmed Confirmed as real issue

Comments

@wzet
Copy link

wzet commented Aug 19, 2015

Example to reproduce the error:
http://pastebin.com/jv3vu0iW

Output:
Exception ValueError: 'list.remove(x): x not in list' in 'kivy.graphics.instructions.Instruction.rremove' ignored

@kived kived added Component: graphics kivy/graphics Status: Confirmed Confirmed as real issue labels Aug 19, 2015
@tshirtman
Copy link
Member

The issue likely comes from the self.parent.canvas.remove line, since ScrollView manipulates its canvas (as seen in its add_widget overriding), and indeed, a dirty workaround is to add an intermediate widget between the widget to capture and the ScrollView (i used another GridLayout, but anything correctly containing the current GridLayout would do).

This doesn't give an easy solution, though, maybe ScrollView should avoid adding an indirection to its canvas, or export_to_png should use a function or AliasProperty to get the actual canvas, and ScrollView would override it.

@tshirtman
Copy link
Member

hm, actually, checking that `canvas_parent_index' is not -1 does seem to fix the issue, not sure it's 100% harmless though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: graphics kivy/graphics Status: Confirmed Confirmed as real issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants