Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a sketch for async loading of view sequences.
ViewSequence
currently acts as the interface between views which do layout and strategies for loading views. This PR tries to add a way for layout views likeLinearLayout
to request the parts of aViewSequence
which would be visible while keeping the the identity for a specific view which should stay in focus.It adds a
set_focus
method to theViewSequence
trait. This method can be used by sequences with a variable length to load only the elements which would be visible.The rebuild method is changed to return a
SequencePosition
this marks the index of the element marked inset_focus
in the rebuild sequence. Using this the container could call a method like druidsscroll_area_to_view()
to keep the the element in view.This is a idea i had for some time, which i would like to discuss in the office hours. The implementation for the backend is missing and the design for
SequencePosition
also needs some work.