A Rendering bug when resizing

[mewadahimansh01-stack]

Description

I am making a pong clone and this bug came to light. In the given game when I click on maximize then maximize again to go back to normal size, the right paddle disappears completely and doesn't reappear until i resize it by dragging the borders, then it stays even if you click on maximize repeatedly. I believe my code should work normally for the first resize. What are your views on it ? The code is provided below, if anyone wants to check you can remove the logo parts and it should still produce the same bug.

My Computer Specifications are:

Operating System: Arch Linux
KDE Plasma Version: 6.6.2
KDE Frameworks Version: 6.23.0
Qt Version: 6.10.2
Kernel Version: 6.19.8-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 4 × Intel® Core™ i7-6600U CPU @ 2.60GHz
Memory: 8 GiB of RAM (7.6 GiB usable)
Graphics Processor 1: Intel® HD Graphics 520
Graphics Processor 2: AMD Radeon R7 M360
Manufacturer: Dell Inc.
Product Name: Latitude E5470

Code

use macroquad::{miniquad::conf::Icon, prelude::*};

#[macroquad::main(window_conf)]
async fn main() {

    let width = screen_width();
    let height = screen_height();

    let paddle = [width / 40., height/ 5.];
    let mut position_left = [
        width / 2. - 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];
    let mut position_right = [
        width / 2. + 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];

    loop {

        if width != screen_width() || height != screen_height() {
            position_left = [
                screen_width() / 2. - 2. * screen_width() / 5. - paddle[0] / 2.,
                screen_height() / 2. - screen_height() / 6.,
            ];
            position_right = [
                screen_width() / 2. + 2. * screen_width() / 5. - paddle[0] / 2.,
                screen_height() / 2. - screen_height() / 6.,
            ];
        }

        clear_background(BEIGE);

        // x, y, w, h, color
        // right paddle
        draw_rectangle(
            position_right[0],
            position_right[1],
            paddle[0],
            paddle[1],
            RED,
        );

        // left paddle
        draw_rectangle(
            position_left[0],
            position_left[1],
            paddle[0],
            paddle[1],
            RED,
        );

        next_frame().await;
    }
}

fn window_conf() -> Conf {
    Conf {
        window_resizable: true,
        window_title: "Pong Clone".to_string(),
        icon: Some(set_window_icon()),
        ..Default::default()
    }
}

fn set_window_icon() -> Icon {
    Icon {
        small: *include_bytes!("logo16.rgba"),
        medium: *include_bytes!("logo32.rgba"),
        big: *include_bytes!("logo64.rgba"),
    }
}

[mewadahimansh01-stack]

You can use this to check

use macroquad::{miniquad::conf::Icon, prelude::*};

#[macroquad::main(window_conf)]
async fn main() {

    let width = screen_width();
    let height = screen_height();

    let paddle = [width / 40., height/ 5.];
    let mut position_left = [
        width / 2. - 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];
    let mut position_right = [
        width / 2. + 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];

    loop {

        if width != screen_width() || height != screen_height() {
            position_left = [
                screen_width() / 2. - 2. * screen_width() / 5. - paddle[0] / 2.,
                screen_height() / 2. - screen_height() / 6.,
            ];
            position_right = [
                screen_width() / 2. + 2. * screen_width() / 5. - paddle[0] / 2.,
                screen_height() / 2. - screen_height() / 6.,
            ];
        }

        clear_background(BEIGE);

        // x, y, w, h, color
        // right paddle
        draw_rectangle(
            position_right[0],
            position_right[1],
            paddle[0],
            paddle[1],
            RED,
        );

        // left paddle
        draw_rectangle(
            position_left[0],
            position_left[1],
            paddle[0],
            paddle[1],
            RED,
        );

        next_frame().await;
    }
}

fn window_conf() -> Conf {
    Conf {
        window_resizable: true,
        window_title: "Pong Clone".to_string(),
        ..Default::default()
    }
}

[DissolveDZ]

Probably an XWayland thing since you're on Wayland and don't hint experimental Wayland support (I also can't reproduce it)

[mewadahimansh01-stack]

@DissolveDZ
So, what can I do to solve the problem? I don't know much about this.

[maxwoolfdev]

Reproduced on Windows, so it's not related to Wayland.

Could relate to the issue #1021 (comment)

[maxwoolfdev]

I just fixed it by adding mutable into width, height, and paddle.

Here you go:

use macroquad::{miniquad::conf::Icon, prelude::*};

#[macroquad::main(window_conf)]
async fn main() {

    let mut width = screen_width();
    let mut height = screen_height();

    let mut paddle = [width / 40., height / 5.];
    let mut position_left = [
        width / 2. - 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];
    let mut position_right = [
        width / 2. + 2. * width / 5. - paddle[0] / 2.,
        height / 2. - height / 6.,
    ];

    loop {

        if width != screen_width() || height != screen_height() {
            width = screen_width();
            height = screen_height();

            paddle = [width / 40., height / 5.];
            position_left = [
                width / 2. - 2. * width / 5. - paddle[0] / 2.,
                height / 2. - height / 6.,
            ];
            position_right = [
                width / 2. + 2. * width / 5. - paddle[0] / 2.,
                height / 2. - height / 6.,
            ];
        }

        clear_background(BEIGE);

        // x, y, w, h, color
        // right paddle
        draw_rectangle(
            position_right[0],
            position_right[1],
            paddle[0],
            paddle[1],
            RED,
        );

        // left paddle
        draw_rectangle(
            position_left[0],
            position_left[1],
            paddle[0],
            paddle[1],
            RED,
        );

        next_frame().await;
    }
}

fn window_conf() -> Conf {
    Conf {
        window_resizable: true,
        window_title: "Pong Clone".to_string(),
        ..Default::default()
    }
}

[DanielJoyce]

Yeah, that makes sense, without the mut and assignement its always comparing against the width and height it had when first launched.

[lofcz]

Fixed in Miniquad, needs to be merged in: not-fl3/miniquad#631