plots ignore the ui layout

[melodycatt]

using a bottom up layout, plots just go to the top left corner, and overlap each other if there is more than one. i tried getting the height of the response rect from the first plot, then manually adding space, but the second plot goes above the first, off the screen. here is that code:

egui::CentralPanel::default().show(ctx, |ui| {
            ui.with_layout(egui::Layout::bottom_up(egui::Align::Center), |ui| {
                    let red = hex("#ff0019");
                    let accent = hex("#ff7700");
                    let accent_xl = hex("#ffa600");

                    ui.style_mut().visuals.override_text_color = Some(red);

                    let height = Plot::new("a")
                        .view_aspect(1.75)
                        .width(ui.available_width() * 0.5)
                        .allow_scroll(false)
                        .allow_zoom(false)
                        .allow_drag(false)
                        .allow_axis_zoom_drag(false)
                        .show(ui, |plot_ui| {
                            plot_ui.line(
                                Line::new(
                                    "a",
                                    PlotPoints::from_iter(...),
                                )
                                .color(accent),
                            );
                        })
                        .response
                        .rect
                        .height();

                    ui.add_space(height);
                    let height = Plot::new("temp")
                        .view_aspect(1.75)
                        .width(ui.available_width() * 0.5)
                        .allow_scroll(false)
                        .allow_zoom(false)
                        .allow_drag(false)
                        .allow_axis_zoom_drag(false)
                        .show(ui, |plot_ui| {
                            plot_ui.line(
                                Line::new(
                                    "temp",
                                    PlotPoints::from_iter(...),
                                )
                                .color(accent),
                            );
                        })
                        .response
                        .rect
                        .height();
                    ui.add_space(height);
            })
        });

[tarolling]

able to repro with this snippet (heights were not needed to see the same behavior, also looks better when fix is applied):

use eframe::egui;
use egui_plot::{Line, Plot, PlotPoints};

struct App;

impl eframe::App for App {
    fn ui(&mut self, ui: &mut egui::Ui, _frame: &mut eframe::Frame) {
        egui::CentralPanel::default().show(ui, |ui| {
            ui.with_layout(egui::Layout::bottom_up(egui::Align::Center), |ui| {
                let red = egui::Color32::from_hex("#ff0019").unwrap();
                let accent = egui::Color32::from_hex("#ff7700").unwrap();
                let accent_xl = egui::Color32::from_hex("#ffa600").unwrap();

                ui.style_mut().visuals.override_text_color = Some(red);

                Plot::new("a")
                    .view_aspect(1.75)
                    .width(ui.available_width() * 0.5)
                    .allow_scroll(false)
                    .allow_zoom(false)
                    .allow_drag(false)
                    .allow_axis_zoom_drag(false)
                    .show(ui, |plot_ui| {
                        plot_ui.line(
                            Line::new(
                                "a",
                                PlotPoints::from_iter(
                                    (0..100).map(|i| [i as f64, (i as f64 * 0.1).sin()]),
                                ),
                            )
                            .color(accent),
                        );
                    });

                Plot::new("temp")
                    .view_aspect(1.75)
                    .width(ui.available_width() * 0.5)
                    .allow_scroll(false)
                    .allow_zoom(false)
                    .allow_drag(false)
                    .allow_axis_zoom_drag(false)
                    .show(ui, |plot_ui| {
                        plot_ui.line(
                            Line::new(
                                "temp",
                                PlotPoints::from_iter(
                                    (0..100).map(|i| [i as f64, (i as f64 * 0.1).cos()]),
                                ),
                            )
                            .color(accent_xl),
                        );
                    });
            })
        });
    }
}

fn main() -> eframe::Result {
    let options = eframe::NativeOptions::default();
    eframe::run_native(
        "Issue 237 repro",
        options,
        Box::new(|_cc| Ok(Box::new(App))),
    )
}