emilk / egui

egui: an easy-to-use immediate mode GUI in Rust that runs on both web and native
https://www.egui.rs/
Apache License 2.0
22.25k stars 1.6k forks source link

Variable Sized Items in Scroll View #1376

Open dignifiedquire opened 2 years ago

dignifiedquire commented 2 years ago

I am trying to render 10.000+ items in a scroll view, which all have different heights, including multiline text and images. Currently the examples only show how to render rows with the same height when only rendering what is visible. But in lrder for me to know the height of the elements I have to lay them out first. Any pointers or thoughts on hoe to add support for this?

Thank you

emilk commented 2 years ago

The only way to support this is if you know the heights of the elements beforehand. For instance, if you are showing a bunch of images of different sizes, you will (presumably) know they size of each image beforehand. Or you lay out your widgets once and then store the bounds of each in a vector (i.e. the top/bottom for vertical scrolling). You can then use ScrollArea::show_viewport and do a binary search in your vector to figure out which elements need to be rendered. A helper for such a thing could potentially be created and added to egui. Do you feel like working on it?

dignifiedquire commented 2 years ago

The challenge I have is that (a) I can't load all items from the database due to performance issues and (b) because there is variable sized text in there, I don't know how many rows to calculate without laying it out first.

For context, this is the messages view for a chat application.

It would be totally acceptable if the scrollbar is "wrong" and I just load elements as I reach the start/end of the scroll area, and the srollbar adjusts accordingly to what is now rendered. But I am not sure how I could achieve that either with the current setup.

coderedart commented 2 years ago

Discord only shows a few messages at a time. And when you scroll too far back, it gives you a button to manually press asking whether you want to load previous messages. Same with reddit. You could try to batch the messages too like Google search results or other forums and only have like 40 or so messages at a time in scroll area.

dignifiedquire commented 2 years ago

@coderedart yes that’s what I am aiming for, just without manually pressing a button, but rather through scrolling.

xNxExOx commented 2 years ago
let count = my_things_to_draw.len();
egui::ScrollArea::vertical().auto_shrink([false;2]).show_indexed_viewport(ui, count, |ui, viewport|{
    let max_height = viewport.height();
    let mut used_rect = egui::Rect::NOTHING;
    for drawable in my_things_to_draw.iter().skip(viewport.start_index()) {
        used_rect = used_rect.union(drawable.draw(ui));
        if used_rect.height() > max_height {
            break;
        }
    }

    ui.allocate_rect(used_rect, egui::Sense::hover());
});

based on the examples I came up with this idea show_indexed_viewport with start_index on the viewport. using usize instead of f32 would allow easy implementation on the callers side, and only potential issue would be if single item would not fit to the view, but for sizes up to 8 388 607 it can solved by using f32 internally and apply some relative shift (that part would need additional function to define a size of an first item).

xNxExOx commented 2 years ago

@emilk the main page state:

Extensible: easy to write your own widgets for egui

but when I am trying to implement item scrolling area, I am hitting private functions: ui.ctx().frame_state(), and ui.advance_cursor_after_rect(...), is that intended? Are libraries supposed to use different functions? If so which?

Thanks in advance for the answer.

emilk commented 2 years ago

@xNxExOx: some things should probably be made pub. Open a PR

…but also: containers, such as scroll areas, are not always easy to implement in egui :)

xNxExOx commented 2 years ago

ok I will take another look at it and open PR when if I have something (if I figure how these scroll bars work)

xNxExOx commented 2 years ago

I decided to do it differently this time, and write it within the repo, so no issues with private types and functions this time.

I have it almost working, but I am quite stuck on meaning content_is_too_small in relation to show_scroll_this_frame

Does content_is_too_small means that the content is bigger than the area it is shown on? (That is only explanation that worked for me, in which case I think it should be called content_is_too_large instead)

But at this point I do not think I will make it as far as PR, because I am missing few important details 😞

Not finished:

  1. When scrolling by dragging the handle it is jumping too much, and I am not sure why. (happens for about 70% of scroll bar)
  2. When scrolling by dragging the content (opposite direction), it is bit inconsistent and sometimes jumping on last item. (fine otherwise) (jumpy if last item small, not possible to scroll true if last item too big)
  3. Not nice alignment when scrolling by mouse wheel. (would be ideal it is would snap to item boundaries from reasonable distance, to not show half line of the first shown item that often)
  4. Scrolling by mouse wheel feels too fast
  5. Scrolling over the last item shrinks the scroll handle (intended, because I do not know a better solution, so this point is to improve, not an unfinished / bugged part)
  6. The already mentioned content_is_too_small calculation and related show_scroll_this_frame
  7. auto_shrink is disabled for the axis with scroll bar
  8. not finished comments and examples
  9. Calculations of fraction offset when scrolling to new item
  10. Consistent IDs. (Not sure if needed to do anything, or not)
  11. No inner response, because I decided it does not make sense, but maybe it can return range that was drawn, or something?
  12. Lot of shared logic with ScrollArea so would be probably worth considering sharing the logic instead of duplicating for the final code.

What works:

If you have any questions why I decided to do some of the changes the way I did, feel free to ask. If you have any suggestions how should I continue (what to look at, to fix the issues listed above) I am open to suggestions. If you think it is worth adding with so many TODOs feel free to do so. (I am kind of too perfectionist, to even consider PR at this stage)

Code This is really copy paste of `scroll_area.rs` with attempt to do as little changes as possible. So you can just compare the two files to see changes. ```rs //! Coordinate system names: //! * content: size of contents (generally large; that's why we want scroll bars) //! * outer: size of scroll area including scroll bar(s) //! * inner: excluding scroll bar(s). The area we clip the contents to. #![allow(clippy::needless_range_loop)] use crate::*; #[derive(Clone, Copy, Debug)] #[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))] #[cfg_attr(feature = "serde", serde(default))] pub struct State { /// Positive offset means scrolling down/right pub offset: Vec2, show_scroll: [bool; 2], /// Momentum, used for kinetic scrolling #[cfg_attr(feature = "serde", serde(skip))] pub vel: Vec2, /// Mouse offset relative to the top of the handle when started moving the handle. scroll_start_offset_from_top_left: [Option; 2], /// Is the scroll sticky. This is true while scroll handle is in the end position /// and remains that way until the user moves the scroll_handle. Once unstuck (false) /// it remains false until the scroll touches the end position, which reenables stickiness. scroll_stuck_to_end: [bool; 2], first_to_show: usize, previous_total_items: usize, first_shown_item_size: Vec2, } impl Default for State { fn default() -> Self { Self { offset: Vec2::ZERO, show_scroll: [false; 2], vel: Vec2::ZERO, scroll_start_offset_from_top_left: [None; 2], scroll_stuck_to_end: [true; 2], first_to_show: 0, previous_total_items: 0, first_shown_item_size: Vec2::ZERO, } } } impl State { pub fn load(ctx: &Context, id: Id) -> Option { ctx.data().get_persisted(id) } pub fn store(self, ctx: &Context, id: Id) { ctx.data().insert_persisted(id, self); } } pub struct ScrollAreaOutput { /// [`Id`] of the [`ItemScrollArea`]. pub id: Id, /// The current state of the scroll area. pub state: State, /// Where on the screen the content is (excludes scroll bars). pub inner_rect: Rect, } /// Add vertical and/or horizontal scrolling to a contained [`Ui`]. /// /// ``` /// # egui::__run_test_ui(|ui| { /// egui::ItemScrollArea::vertical(1).show_items(ui, |ui, i| { /// // Add a lot of widgets here for `i`-th item. /// }); /// # }); /// ``` /// /// You can scroll to an element using [`Response::scroll_to_me`], [`Ui::scroll_to_cursor`] and [`Ui::scroll_to_rect`]. #[derive(Clone, Debug)] #[must_use = "You should call .show()"] pub struct ItemScrollArea { /// Do we have horizontal/vertical scrolling? has_bar: [bool; 2], auto_shrink: [bool; 2], max_size: Vec2, min_scrolled_size: Vec2, always_show_scroll: bool, id_source: Option, offset_x: Option, offset_y: Option, /// If false, we ignore scroll events. scrolling_enabled: bool, /// If true for vertical or horizontal the scroll wheel will stick to the /// end position until user manually changes position. It will become true /// again once scroll handle makes contact with end. stick_to_end: [bool; 2], total_items: usize, } impl ItemScrollArea { /// Create a horizontal scroll area. pub fn horizontal(total_items: usize) -> Self { Self::new([true, false], total_items) } /// Create a vertical scroll area. pub fn vertical(total_items: usize) -> Self { Self::new([false, true], total_items) } /// Create a scroll area where both direction of scrolling is disabled. /// It's unclear why you would want to do this. pub fn neither(total_items: usize) -> Self { Self::new([false, false], total_items) } /// Create a scroll area where you decide which axis has scrolling enabled. /// For instance, `ScrollAre::new([true, false])` enable horizontal scrolling. // not pub, because both does not make sense fn new(has_bar: [bool; 2], total_items: usize) -> Self { Self { has_bar, auto_shrink: [true; 2], max_size: Vec2::INFINITY, min_scrolled_size: Vec2::splat(64.0), always_show_scroll: false, id_source: None, offset_x: None, offset_y: None, scrolling_enabled: true, stick_to_end: [false; 2], total_items, } } /// The maximum width of the outer frame of the scroll area. /// /// Use `f32::INFINITY` if you want the scroll area to expand to fit the surrounding [`Ui`] (default). /// /// See also [`Self::auto_shrink`]. pub fn max_width(mut self, max_width: f32) -> Self { self.max_size.x = max_width; self } /// The maximum height of the outer frame of the scroll area. /// /// Use `f32::INFINITY` if you want the scroll area to expand to fit the surrounding [`Ui`] (default). /// /// See also [`Self::auto_shrink`]. pub fn max_height(mut self, max_height: f32) -> Self { self.max_size.y = max_height; self } /// The minimum width of a horizontal scroll area which requires scroll bars. /// /// The [`ItemScrollArea`] will only become smaller than this if the content is smaller than this /// (and so we don't require scroll bars). /// /// Default: `64.0`. pub fn min_scrolled_width(mut self, min_scrolled_width: f32) -> Self { self.min_scrolled_size.x = min_scrolled_width; self } /// The minimum height of a vertical scroll area which requires scroll bars. /// /// The [`ItemScrollArea`] will only become smaller than this if the content is smaller than this /// (and so we don't require scroll bars). /// /// Default: `64.0`. pub fn min_scrolled_height(mut self, min_scrolled_height: f32) -> Self { self.min_scrolled_size.y = min_scrolled_height; self } /// If `false` (default), the scroll bar will be hidden when not needed/ /// If `true`, the scroll bar will always be displayed even if not needed. pub fn always_show_scroll(mut self, always_show_scroll: bool) -> Self { self.always_show_scroll = always_show_scroll; self } /// A source for the unique [`Id`], e.g. `.id_source("second_scroll_area")` or `.id_source(loop_index)`. pub fn id_source(mut self, id_source: impl std::hash::Hash) -> Self { self.id_source = Some(Id::new(id_source)); self } /*/// Set the horizontal and vertical scroll offset position. /// /// See also: [`Self::vertical_scroll_offset`], [`Self::horizontal_scroll_offset`], /// [`Ui::scroll_to_cursor`](crate::ui::Ui::scroll_to_cursor) and /// [`Response::scroll_to_me`](crate::Response::scroll_to_me) pub fn scroll_offset(mut self, offset: Vec2) -> Self { self.offset_x = Some(offset.x); self.offset_y = Some(offset.y); self } /// Set the vertical scroll offset position. /// /// See also: [`Self::scroll_offset`], [`Ui::scroll_to_cursor`](crate::ui::Ui::scroll_to_cursor) and /// [`Response::scroll_to_me`](crate::Response::scroll_to_me) pub fn vertical_scroll_offset(mut self, offset: f32) -> Self { self.offset_y = Some(offset); self } /// Set the horizontal scroll offset position. /// /// See also: [`Self::scroll_offset`], [`Ui::scroll_to_cursor`](crate::ui::Ui::scroll_to_cursor) and /// [`Response::scroll_to_me`](crate::Response::scroll_to_me) pub fn horizontal_scroll_offset(mut self, offset: f32) -> Self { self.offset_x = Some(offset); self }*/ /*/// Turn on/off scrolling on the horizontal axis. pub fn hscroll(mut self, hscroll: bool) -> Self { self.has_bar[0] = hscroll; self } /// Turn on/off scrolling on the vertical axis. pub fn vscroll(mut self, vscroll: bool) -> Self { self.has_bar[1] = vscroll; self } /// Turn on/off scrolling on the horizontal/vertical axes. pub fn scroll2(mut self, has_bar: [bool; 2]) -> Self { self.has_bar = has_bar; self }*/ /// Control the scrolling behavior /// If `true` (default), the scroll area will respond to user scrolling /// If `false`, the scroll area will not respond to user scrolling /// /// This can be used, for example, to optionally freeze scrolling while the user /// is inputing text in a [`TextEdit`] widget contained within the scroll area. /// /// This controls both scrolling directions. pub fn enable_scrolling(mut self, enable: bool) -> Self { self.scrolling_enabled = enable; self } /// For each axis, should the containing area shrink if the content is small? /// /// If true, egui will add blank space outside the scroll area. /// If false, egui will add blank space inside the scroll area. /// /// Default: `[true; 2]`. pub fn auto_shrink(mut self, auto_shrink: [bool; 2]) -> Self { self.auto_shrink = auto_shrink; self } /// The scroll handle will stick to the rightmost position even while the content size /// changes dynamically. This can be useful to simulate text scrollers coming in from right /// hand side. The scroll handle remains stuck until user manually changes position. Once "unstuck" /// it will remain focused on whatever content viewport the user left it on. If the scroll /// handle is dragged all the way to the right it will again become stuck and remain there /// until manually pulled from the end position. pub fn stick_to_right(mut self) -> Self { self.stick_to_end[0] = true; self } /// The scroll handle will stick to the bottom position even while the content size /// changes dynamically. This can be useful to simulate terminal UIs or log/info scrollers. /// The scroll handle remains stuck until user manually changes position. Once "unstuck" /// it will remain focused on whatever content viewport the user left it on. If the scroll /// handle is dragged to the bottom it will again become stuck and remain there until manually /// pulled from the end position. pub fn stick_to_bottom(mut self) -> Self { self.stick_to_end[1] = true; self } } struct Prepared { id: Id, state: State, has_bar: [bool; 2], auto_shrink: [bool; 2], /// How much horizontal and vertical space are used up by the /// width of the vertical bar, and the height of the horizontal bar? current_bar_use: Vec2, always_show_scroll: bool, /// Where on the screen the content is (excludes scroll bars). inner_rect: Rect, content_ui: Ui, scrolling_enabled: bool, stick_to_end: [bool; 2], total_items: usize, } impl ItemScrollArea { fn begin(self, ui: &mut Ui) -> Prepared { let Self { has_bar, auto_shrink, max_size, min_scrolled_size, always_show_scroll, id_source, offset_x, offset_y, scrolling_enabled, stick_to_end, total_items, } = self; let ctx = ui.ctx().clone(); let id_source = id_source.unwrap_or_else(|| Id::new("scroll_area")); let id = ui.make_persistent_id(id_source); ui.ctx().check_for_id_clash( id, Rect::from_min_size(ui.available_rect_before_wrap().min, Vec2::ZERO), "ItemScrollArea", ); let mut state = State::load(&ctx, id).unwrap_or_default(); state.offset.x = offset_x.unwrap_or(state.offset.x); state.offset.y = offset_y.unwrap_or(state.offset.y); let max_scroll_bar_width = max_scroll_bar_width_with_margin(ui); let current_hscroll_bar_height = if !has_bar[0] { 0.0 } else if always_show_scroll { max_scroll_bar_width } else { max_scroll_bar_width * ui.ctx().animate_bool(id.with("h"), state.show_scroll[0]) }; let current_vscroll_bar_width = if !has_bar[1] { 0.0 } else if always_show_scroll { max_scroll_bar_width } else { max_scroll_bar_width * ui.ctx().animate_bool(id.with("v"), state.show_scroll[1]) }; let current_bar_use = vec2(current_vscroll_bar_width, current_hscroll_bar_height); let available_outer = ui.available_rect_before_wrap(); let outer_size = available_outer.size().at_most(max_size); let inner_size = { let mut inner_size = outer_size - current_bar_use; // Don't go so far that we shrink to zero. // In particular, if we put a [`ItemScrollArea`] inside of a [`ItemScrollArea`], the inner // one shouldn't collapse into nothingness. // See https://github.com/emilk/egui/issues/1097 for d in 0..2 { if has_bar[d] { inner_size[d] = inner_size[d].max(min_scrolled_size[d]); } } inner_size }; let inner_rect = Rect::from_min_size(available_outer.min, inner_size); let mut content_max_size = inner_size; if true { // Tell the inner Ui to *try* to fit the content without needing to scroll, // i.e. better to wrap text and shrink images than showing a horizontal scrollbar! } else { // Tell the inner Ui to use as much space as possible, we can scroll to see it! for d in 0..2 { if has_bar[d] { content_max_size[d] = f32::INFINITY; } } } let fraction = |d| { if state.previous_total_items == 0 { 0. } else { let scroll_item_size = inner_rect.height() / (state.previous_total_items as f32); if state.first_to_show != ((state.offset[d] / scroll_item_size) as usize) { // TODO figure out handling for this case, to work with drag-scrolling //dbg!(state.first_to_show, state.offset[d] / scroll_item_size); 0. } else { let a = state.offset[d] - state.first_to_show as f32 * scroll_item_size; state.first_shown_item_size[d] * a / scroll_item_size } } }; let fraction = if has_bar[0] { vec2(fraction(0), 0.) } else if has_bar[1] { vec2(0., fraction(1)) } else { vec2(0., 0.) }; let content_max_rect = Rect::from_min_size(inner_rect.min - fraction, inner_size); let mut content_ui = ui.child_ui(content_max_rect, *ui.layout()); let mut content_clip_rect = inner_rect.expand(ui.visuals().clip_rect_margin); content_clip_rect = content_clip_rect.intersect(ui.clip_rect()); // Nice handling of forced resizing beyond the possible: for d in 0..2 { if !has_bar[d] { content_clip_rect.max[d] = ui.clip_rect().max[d] - current_bar_use[d]; } } content_ui.set_clip_rect(content_clip_rect); state.first_to_show = if has_bar[0] { let item_height_on_scroll = inner_size[0] / (total_items as f32); (state.offset[0] / item_height_on_scroll) as usize } else if has_bar[1] { let item_height_on_scroll = inner_size[1] / (total_items as f32); (state.offset[1] / item_height_on_scroll) as usize } else { 0 }.min(total_items - 1); Prepared { id, state, has_bar, auto_shrink, current_bar_use, always_show_scroll, inner_rect, content_ui, scrolling_enabled, stick_to_end, total_items, } } /// Efficiently show only the visible part of a large number of rows. /// /// ``` /// # egui::__run_test_ui(|ui| { /// let total_rows = 10_000; /// egui::ItemScrollArea::vertical(total_rows).show_items(ui, |ui, row| { /// let text = format!("Row {}/{}", row + 1, total_rows); /// ui.label(text); /// }); /// # }); /// ``` pub fn show_items( self, ui: &mut Ui, mut add_contents: impl FnMut(&mut Ui, usize), ) -> ScrollAreaOutput { let mut prepared = self.begin(ui); let id = prepared.id; let inner_rect = prepared.inner_rect; // TODO Make sure we get consistent IDs? add_contents(&mut prepared.content_ui, prepared.state.first_to_show); prepared.state.first_shown_item_size = prepared.content_ui.min_size(); for i in (prepared.state.first_to_show + 1)..prepared.total_items { add_contents(&mut prepared.content_ui, i); if !inner_rect.y_range().contains(&prepared.content_ui.next_widget_position().y) { break; } } let state = prepared.end(ui); ScrollAreaOutput { id, state, inner_rect, } } } impl Prepared { fn end(self, ui: &mut Ui) -> State { let Prepared { id, mut state, inner_rect, has_bar, auto_shrink, mut current_bar_use, always_show_scroll, content_ui, scrolling_enabled, stick_to_end, total_items, } = self; // TODO is there an better way? (to prevent shrinking when last item is smaller) let auto_shrink = if state.first_to_show > 0 { if has_bar[0] { [false, auto_shrink[1]] } else { [auto_shrink[0], false] } } else { auto_shrink }; if state.previous_total_items != total_items { for d in 0..2 { if has_bar[d] { if state.previous_total_items != 0 { if total_items != 0 { let old_item_height = inner_rect.height() / (state.previous_total_items as f32); let new_item_height = inner_rect.height() / (total_items as f32); state.offset[d] -= state.first_to_show as f32 * old_item_height; state.offset[d] += state.first_to_show as f32 * new_item_height; } else { state.offset[d] = 0.; } } else { state.offset[d] = 0.; } } else { state.offset[d] = 0.; } } } let content_size = content_ui.min_size(); for d in 0..2 { if has_bar[d] { // We take the scroll target so only this ItemScrollArea will use it: let scroll_target = content_ui.ctx().frame_state().scroll_target[d].take(); if let Some((scroll, align)) = scroll_target { let min = content_ui.min_rect().min[d]; let clip_rect = content_ui.clip_rect(); let visible_range = min..=min + clip_rect.size()[d]; let start = *scroll.start(); let end = *scroll.end(); let clip_start = clip_rect.min[d]; let clip_end = clip_rect.max[d]; let mut spacing = ui.spacing().item_spacing[d]; let delta = if let Some(align) = align { let center_factor = align.to_factor(); let offset = lerp(scroll, center_factor) - lerp(visible_range, center_factor); // Depending on the alignment we need to add or subtract the spacing spacing *= remap(center_factor, 0.0..=1.0, -1.0..=1.0); offset + spacing - state.offset[d] } else if start < clip_start && end < clip_end { -(clip_start - start + spacing).min(clip_end - end - spacing) } else if end > clip_end && start > clip_start { (end - clip_end + spacing).min(start - clip_start - spacing) } else { // Ui is already in view, no need to adjust scroll. 0.0 }; if delta != 0.0 { state.offset[d] += delta; ui.ctx().request_repaint(); } } } } let inner_rect = { // At this point this is the available size for the inner rect. let mut inner_size = inner_rect.size(); for d in 0..2 { inner_size[d] = match (has_bar[d], auto_shrink[d]) { (true, true) => inner_size[d].min(content_size[d]), // shrink scroll area if content is small (true, false) => inner_size[d], // let scroll area be larger than content; fill with blank space (false, true) => content_size[d], // Follow the content (expand/contract to fit it). (false, false) => inner_size[d].max(content_size[d]), // Expand to fit content }; } let mut inner_rect = Rect::from_min_size(inner_rect.min, inner_size); // The window that egui sits in can't be expanded by egui, so we need to respect it: for d in 0..2 { if !has_bar[d] { // HACK for when we have a vertical-only scroll area in a top level panel, // and that panel is not wide enough for the contents. // This code ensures we still see the scroll bar! let max = ui.input().screen_rect().max[d] - current_bar_use[d] - ui.spacing().item_spacing[d]; inner_rect.max[d] = inner_rect.max[d].at_most(max); // TODO(emilk): maybe auto-enable horizontal/vertical scrolling if this limit is reached } } inner_rect }; let original_inner_rect = inner_rect; let outer_rect = Rect::from_min_size(inner_rect.min, inner_rect.size() + current_bar_use); // TODO should I modify this if we scrolled to the end? (meaning there is empty space after last item) let content_is_too_small = [ content_size.x > inner_rect.width() || (has_bar[0] && state.first_to_show != 0), content_size.y > inner_rect.height() || (has_bar[1] && state.first_to_show != 0), ]; if content_is_too_small[0] || content_is_too_small[1] { // Drag contents to scroll (for touch screens mostly): let sense = if self.scrolling_enabled { Sense::drag() } else { Sense::hover() }; let content_response = ui.interact(inner_rect, id.with("area"), sense); if content_response.dragged() { for d in 0..2 { if has_bar[d] { state.offset[d] -= ui.input().pointer.delta()[d]; state.vel[d] = ui.input().pointer.velocity()[d]; state.scroll_stuck_to_end[d] = false; } else { state.vel[d] = 0.0; } } } else { let stop_speed = 20.0; // Pixels per second. let friction_coeff = 1000.0; // Pixels per second squared. let dt = ui.input().unstable_dt; let friction = friction_coeff * dt; if friction > state.vel.length() || state.vel.length() < stop_speed { state.vel = Vec2::ZERO; } else { state.vel -= friction * state.vel.normalized(); // Offset has an inverted coordinate system compared to // the velocity, so we subtract it instead of adding it state.offset -= state.vel * dt; ui.ctx().request_repaint(); } } } let max_offset = original_inner_rect.size(); if scrolling_enabled && ui.rect_contains_pointer(outer_rect) { for d in 0..2 { if has_bar[d] { let mut frame_state = ui.ctx().frame_state(); let scroll_delta = frame_state.scroll_delta; let scrolling_up = state.offset[d] > 0.0 && scroll_delta[d] > 0.0; let scrolling_down = state.offset[d] < max_offset[d] && scroll_delta[d] < 0.0; if scrolling_up || scrolling_down { state.offset[d] -= scroll_delta[d]; // Clear scroll delta so no parent scroll will use it. frame_state.scroll_delta[d] = 0.0; state.scroll_stuck_to_end[d] = false; } } } } // TODO remove ` || (has_bar[?] && state.first_to_show != 0)` after `content_is_too_small` is correct let show_scroll_this_frame = [ (content_is_too_small[0] || always_show_scroll) || (has_bar[0] && state.first_to_show != 0), (content_is_too_small[1] || always_show_scroll) || (has_bar[1] && state.first_to_show != 0), ]; //dbg!(content_is_too_small[1], show_scroll_this_frame[1]); let max_scroll_bar_width = max_scroll_bar_width_with_margin(ui); // Avoid frame delay; start showing scroll bar right away: if show_scroll_this_frame[0] && current_bar_use.y <= 0.0 { current_bar_use.y = max_scroll_bar_width * ui.ctx().animate_bool(id.with("h"), true); } if show_scroll_this_frame[1] && current_bar_use.x <= 0.0 { current_bar_use.x = max_scroll_bar_width * ui.ctx().animate_bool(id.with("v"), true); } for d in 0..2 { let animation_t = current_bar_use[1 - d] / max_scroll_bar_width; if animation_t == 0.0 { continue; } let scroll_item_size = inner_rect.size()[d] / (total_items as f32); // margin between contents and scroll bar let margin = animation_t * ui.spacing().item_spacing.x; let min_cross = inner_rect.max[1 - d] + margin; // left of vertical scroll (d == 1) let max_cross = outer_rect.max[1 - d]; // right of vertical scroll (d == 1) let min_main = inner_rect.min[d]; // top of vertical scroll (d == 1) let max_main = inner_rect.max[d]; // bottom of vertical scroll (d == 1) let outer_scroll_rect = if d == 0 { Rect::from_min_max( pos2(inner_rect.left(), min_cross), pos2(inner_rect.right(), max_cross), ) } else { Rect::from_min_max( pos2(min_cross, inner_rect.top()), pos2(max_cross, inner_rect.bottom()), ) }; // maybe force increase in offset to keep scroll stuck to end position if stick_to_end[d] && state.scroll_stuck_to_end[d] { state.offset[d] = content_size[d] - inner_rect.size()[d]; } let handle_rect = if d == 0 { Rect::from_min_max( pos2(min_main + state.offset.x, min_cross), pos2(min_main + state.offset.x + scroll_item_size, max_cross), ) } else { Rect::from_min_max( pos2(min_cross, min_main + state.offset.y), pos2(max_cross, min_main + state.offset.y + scroll_item_size), ) }; let interact_id = id.with(d); let sense = if self.scrolling_enabled { Sense::click_and_drag() } else { Sense::hover() }; let response = ui.interact(outer_scroll_rect, interact_id, sense); if let Some(pointer_pos) = response.interact_pointer_pos() { let scroll_start_offset_from_top_left = state.scroll_start_offset_from_top_left[d] .get_or_insert_with(|| { if handle_rect.contains(pointer_pos) { pointer_pos[d] - handle_rect.min[d] } else { let handle_top_pos_at_bottom = max_main - handle_rect.size()[d]; // Calculate the new handle top position, centering the handle on the mouse. let new_handle_top_pos = (pointer_pos[d] - handle_rect.size()[d] / 2.0) .clamp(min_main, handle_top_pos_at_bottom); pointer_pos[d] - new_handle_top_pos } }); let new_handle_top = pointer_pos[d] - *scroll_start_offset_from_top_left; state.offset[d] = remap(new_handle_top, min_main..=max_main, 0.0..=content_size[d]); // some manual action taken, scroll not stuck state.scroll_stuck_to_end[d] = false; } else { state.scroll_start_offset_from_top_left[d] = None; } let unbounded_offset = state.offset[d]; state.offset[d] = state.offset[d].max(0.0); state.offset[d] = state.offset[d].min(max_offset[d]); if state.offset[d] != unbounded_offset { state.vel[d] = 0.0; } if ui.is_rect_visible(outer_scroll_rect) { // Avoid frame-delay by calculating a new handle rect: let mut handle_rect = if d == 0 { Rect::from_min_max( pos2(min_main + state.offset.x, min_cross), pos2(min_main + state.offset.x + scroll_item_size, max_cross), ) } else { Rect::from_min_max( pos2(min_cross, min_main + state.offset.y), pos2(max_cross, min_main + state.offset.y + scroll_item_size), ) }; let min_handle_size = ui.spacing().scroll_bar_width; if handle_rect.size()[d] < min_handle_size { handle_rect = Rect::from_center_size( handle_rect.center(), if d == 0 { vec2(min_handle_size, handle_rect.size().y) } else { vec2(handle_rect.size().x, min_handle_size) }, ); } let visuals = if scrolling_enabled { ui.style().interact(&response) } else { &ui.style().visuals.widgets.inactive }; ui.painter().add(Shape::rect_filled( outer_scroll_rect, visuals.rounding, ui.visuals().extreme_bg_color, )); ui.painter().add(Shape::rect_filled( handle_rect, visuals.rounding, visuals.bg_fill, )); } } ui.advance_cursor_after_rect(outer_rect); if show_scroll_this_frame != state.show_scroll { ui.ctx().request_repaint(); } let available_offset = original_inner_rect.size(); state.offset = state.offset.min(available_offset); state.offset = state.offset.max(Vec2::ZERO); // Is scroll handle at end of content, or is there no scrollbar // yet (not enough content), but sticking is requested? If so, enter sticky mode. // Only has an effect if stick_to_end is enabled but we save in // state anyway so that entering sticky mode at an arbitrary time // has appropriate effect. state.scroll_stuck_to_end = [ (state.offset[0] == available_offset[0]) || (self.stick_to_end[0] && available_offset[0] < 0.), (state.offset[1] == available_offset[1]) || (self.stick_to_end[1] && available_offset[1] < 0.), ]; state.previous_total_items = total_items; state.show_scroll = show_scroll_this_frame; state.store(ui.ctx(), id); state } } /// Width of a vertical scrollbar, or height of a horizontal scroll bar fn max_scroll_bar_width_with_margin(ui: &Ui) -> f32 { ui.spacing().item_spacing.x + ui.spacing().scroll_bar_width } ```
xNxExOx commented 2 years ago

@dignifiedquire can you try ItemScrollArea from my PR? I fix the scrolling to not be so jumpy (in most cases).

emilk commented 2 years ago

Oops, yes it seems content_is_too_small is named the opposite of what it means 🤦. Will fix

xNxExOx commented 2 years ago

Thanks for confirming @emilk that it was just wrongly named, it was really confusing at first, but then I figured out, that it is either wrong name, or some crazy inversion of logic to referring to the area being to small for the content.

I updated the name in my PR too.

dignifiedquire commented 2 years ago

@xNxExOx thanks, will try it out tomorrow

xNxExOx commented 2 years ago

@dignifiedquire how well it worked for your use case?

dignifiedquire commented 2 years ago

It renders similarly for the moment, current challenges are

What I am unclear about is, how do I do dynamic loading, is the idea to give it the total number of known items and fetch them when the show_items closure is called?

xNxExOx commented 2 years ago

Or for your specific case you can just load lets say 100 messages, and scroll to the end, if new message appear, you load them it will automatically scroll to it, and check the response after a draw, if state.offset.y == 0.0 you load another 100 messages before, no need for new functionality I think.

emilk commented 2 years ago

Dropping it here for visibility: egui_extras uses a pretty simple API for heterogeneously sized rows: https://docs.rs/egui_extras/latest/egui_extras/struct.TableBody.html#method.heterogeneous_rows. It is still O(N) in the number of rows, but is very fast.

mikedilger commented 1 year ago

FYI I have addressed this problem in my project with this algorithm:

  1. If the line has no calculated line height, actually render it (even if offscreen) and save the rendered height into a map.
  2. If it does have a height in the map, and it is offscreen, just add_space(height)
  3. otherwise actually render it, and save the height into the height map.

This causes a bit of a pause on the first frame since a very long list is actually rendered to get the heights. But then it's fast after that with accurate scrolling.

jackzhp commented 1 year ago

I have no problem with items with different heights.

My problem is the stick_to_end, I found that it is some kind of stick_to_begin. It is not stick to the bottom. As in the demo "Scrolling" -> "stick_to_end", when a new row is added, "stick_to_bottom"("end") should ensure the last few rows to be shown, rather than the beginning rows to be shown.

I will open a new issue for this: #2897. Any idea about this?

necauqua commented 1 year ago

Or you lay out your widgets once and then store the bounds of each in a vector (i.e. the top/bottom for vertical scrolling). You can then use ScrollArea::show_viewport and do a binary search in your vector to figure out which elements need to be rendered.

I've implemented exactly this (thanks for reminding me that binary search exists :joy:) for my similar chat-like thing - and it works beautifully, and stick_to_end works great as well - 150k rows are smooth as butter, only take a few seconds for the initial render :upside_down_face:

For anyone who also stumbles upon this issue in their search (hope some form of this gets merged in the lib or extras by someone)

My thing cut out a little ```rust #[derive(Default)] struct ScrollList { rows: Vec, pending_rows: Vec, heights: Option>, } impl Deref for ScrollList { type Target = [T]; fn deref(&self) -> &Self::Target { &self.rows } } impl ScrollList where for<'a> &'a T: Widget, { fn new(rows: Vec) -> Self { Self { rows, pending_rows: vec![], heights: None, } } fn push(&mut self, row: T) { self.pending_rows.push(row); } fn mark_dirty(&mut self) { self.heights = None; } fn show(&mut self, ui: &mut Ui, area: ScrollArea) -> ScrollAreaOutput<()> { area.show_viewport(ui, |ui, viewport| { ui.vertical(|ui| match &mut self.heights { // re-render everything to recalculate heights None => { self.rows.append(&mut self.pending_rows); let spacing_y = ui.style().spacing.item_spacing.y; let mut heights = Vec::with_capacity(self.rows.len()); let mut last = 0.0; for message in &self.rows { last += ui.add(message).rect.height() + spacing_y; heights.push(last); } self.heights = Some(heights); } Some(heights) => { let top = viewport.top(); let bottom = viewport.bottom(); let from = heights .binary_search_by(|h| h.total_cmp(&top)) .unwrap_or_else(|i| i); let to = heights .binary_search_by(|h| h.total_cmp(&bottom)) .unwrap_or_else(|i| i); if from != 0 { ui.allocate_space(egui::vec2(0.0, heights[from - 1])); } for row in &self.rows[from..(to + 1).min(self.rows.len())] { ui.add(row); } let spacing_y = ui.style().spacing.item_spacing.y; let mut total = heights.last().copied().unwrap_or_default(); let to = to.min(self.rows.len().saturating_sub(1)); let remaining = total - heights.get(to).copied().unwrap_or_default() - spacing_y; if remaining > 0.0 { ui.allocate_space(egui::vec2(0.0, remaining)); } for row in self.pending_rows.drain(..) { let r = ui.add(&row); total += r.rect.height() + spacing_y; heights.push(total); self.rows.push(row); } } }); }) } } ```

The only real downside is the re-/initial calculation that does a full render, which has to happen whenever the container width changes (speaking of that, would love to have windows to have only a vertical resize).

Another thing which would help with those (that I'll maybe try to do later) is chunked load - where it would actually render and cache heights of only like 1k rows around the current scroll bounds and the rest of the virtual area would be approximated by total_rest_rows * some_average_height - this is a compromise that would have a (hopefully barely noticeable if at all) "incorrect" scrollbar that would jump a little.

lucasmerlin commented 9 months ago

I made egui_virtual_list, which supports items/rows with varying heights, and even custom layouts like here:

image

It works by lazily measuring the row's heights as you scroll down, and caching the heights it's already measured. If you scroll directly to the bottom of the list it will measure 1000 item heights per frame until it knows all heights, so if you have 100000+ items it will take a moment but at least it's not blocking.

My main usecase why I created this crate was to have performant infinite scroll that doesn't get slow if you keep scrolling, so the current version was made with that in mind. But I think it should be possible to create a even smarter algorithm that doesn't need to calculate all heights when you jump somewhere in the list.

For the infinite scroll usecase I also made egui_infinite_scroll as a wrapper of egui_virtual_list. You can try it in the gallery and chat example in the hello_egui demo app