`request_anim_frame()` prevents redrawing in multi-window

[std-microblock]

I have a custom widget that has an animation. After opening two windows with it inside, I found only the focused one was redrawn. The other ones won't redraw even if there's a data change. After deleting request_anim_frame(), it would redraw if there's a data change, but as a result, it's impossible for me to update my animation. Why would this happen? Is multi-window simply just doesn't supports animation or are there any options for me to enable it?

[std-microblock]

( discussion in zulipchat )

[std-microblock]

Also when I resize one of the windows, all of the other windows redraw... I really cannot figure out why...

[std-microblock]

this is message log of these two windows when I resized one of them

[xStrom]

Are you using 0.7.0 or master?

[std-microblock]

master, iirc

[std-microblock]

( more precisely, d0629622f0a57b41f3e0a4decd8502ad276980e5)

[xStrom]

I confirmed this as a bug by combining the anim and multiwin examples.

[xarvic]

I tested this on Linux. This version of the anim.rs example works fine on Linux. In all windows the animation can draw in parralel.

// Copyright 2019 The Druid Authors.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//     http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

//! An example of an animating widget. It is just a widget that
//! requests an animation frame when it needs to, and draws the frame in the
//! `paint` method.
//! Once the animation is over it simply stops requesting animation frames.
//! Usually we would put the state in the `Data`, but for things like animation
//! we don't. This is because the animation state is not useful to know for the
//! rest of the app. If this is something the rest of your widgets should know
//! about, you could put it in the `data`.

// On Windows platform, don't show a console when opening the app.
#![windows_subsystem = "windows"]

use std::f64::consts::PI;

use druid::kurbo::{Circle, Line};
use druid::widget::prelude::*;
use druid::{AppLauncher, Color, LocalizedString, Point, Vec2, WidgetExt, WindowDesc};
use druid::widget::{Button, Flex};

struct AnimWidget {
    t: f64,
}

impl Widget<()> for AnimWidget {
    fn event(&mut self, ctx: &mut EventCtx, event: &Event, _data: &mut (), _env: &Env) {
        match event {
            Event::MouseDown(_) => {
                self.t = 0.0;
                ctx.request_anim_frame();
            }
            Event::AnimFrame(interval) => {
                ctx.request_paint();
                self.t += (*interval as f64) * 1e-9;
                if self.t < 6.0 {
                    ctx.request_anim_frame();
                } else {
                    // We might have t>1.0 at the end of the animation,
                    // we want to make sure the line points up at the
                    // end of the animation.
                    self.t = 0.0;
                }
            }
            _ => (),
        }
    }

    fn lifecycle(&mut self, _ctx: &mut LifeCycleCtx, _event: &LifeCycle, _data: &(), _env: &Env) {}

    fn update(&mut self, _ctx: &mut UpdateCtx, _old_data: &(), _data: &(), _env: &Env) {}

    fn layout(
        &mut self,
        _layout_ctx: &mut LayoutCtx,
        bc: &BoxConstraints,
        _data: &(),
        _env: &Env,
    ) -> Size {
        bc.constrain((100.0, 100.0))
    }

    fn paint(&mut self, ctx: &mut PaintCtx, _data: &(), _env: &Env) {
        let t = self.t;
        let center = Point::new(50.0, 50.0);
        ctx.paint_with_z_index(1, move |ctx| {
            let ambit = center + 45.0 * Vec2::from_angle((0.75 + t) * 2.0 * PI);
            ctx.stroke(Line::new(center, ambit), &Color::WHITE, 1.0);
        });

        ctx.fill(Circle::new(center, 50.0), &Color::BLACK);
    }
}

fn create_win() -> WindowDesc<()> {
    let root = Flex::row()
        .with_child(AnimWidget { t: 0.0 })
        .with_default_spacer()
        .with_child(
            Button::new("New Window")
                .on_click(|ctx, _, _|ctx.new_window(create_win()))
        );

    WindowDesc::new(root).title(
        LocalizedString::new("anim-demo-window-title")
            .with_placeholder("You spin me right round..."),
    )
}

pub fn main() {
    AppLauncher::with_window(create_win())
        .log_to_console()
        .launch(())
        .expect("launch failed");
}

[std-microblock]

I tested this on Linux. This version of the anim.rs example works fine on Linux. In all windows the animation can draw in parralel. code...

tried on Windows 11, same problem