`Viewfinder.visibleGameSize` is not reapplied when the viewport changes size
I maintainer di solito rispondono entro 1 giorno
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
What happened?
The zoom that visibleGameSize sets depends on the size of the viewport, but it's only recalculated in Viewfinder.onGameResize (viewfinder.dart#L221-L224), not when the size of the viewport itself changes. This breaks in three ways:
- The viewport is resized by the game. When a
FixedSizeViewportis resized at runtime, for example to expand a minimap from 256×160 to 576×352, the zoom stays at 0.44. The level then only fills the middle of the expanded minimap, with 1296×792 units visible instead of 576×352. - The viewport of a camera that the game added itself is replaced.
camera.viewport = ...adds the new viewport after the viewfinder (camera_component.dart#L112-L117), so on a game resize, the viewfinder recalculates the zoom from the old size of the viewport, and only then is the viewport resized. The zoom is one resize behind from then on: after maximizing the window it's still the zoom for the old size, and after restoring it, it's the zoom for the maximized window.FlameGame.cameraisn't affected, sinceFlameGame.onGameResizeresizes the viewport of that camera first (flame_game.dart#L219), but every other camera is, like the cameras added for split-screen or minimaps. visibleGameSizeis set before the viewport has a size. For exampleCameraComponent(world: world)..viewfinder.visibleGameSize = Vector2(100, 100)before the camera is added to the game sets the zoom to 0, which throwszoom level must be positive: 0.0.
https://github.com/user-attachments/assets/72c6672a-39cc-4102-9304-4f15c1b29479
https://github.com/user-attachments/assets/e6adbf48-3354-4ae7-89c0-bc7320124d1f
What do you expect?
The zoom is recalculated whenever the size of the viewport changes, so the region given to visibleGameSize always fits into it, including when the viewport only gets its size later.
How can we reproduce this?
- Add a second camera with a
FixedSizeViewport(256, 160), and set itsvisibleGameSizeto the size of the level, 576×352. The zoom is 0.44. - Set
minimap.viewport.size = Vector2(576, 352). - The zoom stays at 0.44 instead of 1.0.
What steps should take to fix this?
The viewport calls Viewfinder.onViewportResize whenever its size changes (viewport.dart#L88). That will recalculate the zoom, which covers all three cases and makes the onGameResize override unnecessary. _updateZoom will also skip the update while the viewport doesn't have a size yet, instead of setting the zoom to 0.
Do have an example of where the bug occurs?
import 'dart:ui';
import 'package:flame/camera.dart';
import 'package:flame/components.dart';
import 'package:flame/game.dart';
import 'package:flutter/material.dart' show Colors;
import 'package:flutter/widgets.dart';
void main() => runApp(GameWidget(game: ReproGame()));
class ReproGame extends FlameGame {
@override
Future<void> onLoad() async {
final levelSize = Vector2(576, 352);
world.add(
RectangleComponent(
size: levelSize,
paint: Paint()..color = Colors.brown,
),
);
final minimap = CameraComponent(
world: world,
viewport: FixedSizeViewport(256, 160)..position = Vector2.all(16),
);
minimap.viewfinder.position = levelSize / 2;
add(minimap);
minimap.viewfinder.visibleGameSize = levelSize;
// Expand the minimap after two seconds.
add(
TimerComponent(
period: 2,
removeOnFinish: true,
onTick: () => minimap.viewport.size = Vector2(576, 352),
),
);
}
}
Expected: after two seconds, the level fills the expanded minimap. Actual: the zoom stays at 0.44, and the level stays small in the middle of the minimap.
For the second case, add the camera to the game yourself, replace its viewport, and then resize the window:
class ReproGame extends FlameGame {
@override
Future<void> onLoad() async {
final arena = World()
..add(
RectangleComponent(
size: Vector2(330, 196),
anchor: Anchor.center,
paint: Paint()..color = Colors.brown,
),
);
final camera = CameraComponent(
world: arena,
viewport: FixedAspectRatioViewport(aspectRatio: 16 / 9),
);
addAll([arena, camera]);
camera.viewfinder.visibleGameSize = Vector2(330, 196);
// Switch to filling the window, like from a display mode setting.
add(
TimerComponent(
period: 2,
removeOnFinish: true,
onTick: () => camera.viewport = MaxViewport(),
),
);
}
}
Expected: after the switch, the arena fits the window at every window size. Actual: after maximizing the window the arena is too small, and after restoring it the arena is cut off.
Relevant log output
# Only for the third case, otherwise nothing is logged or thrown.
'package:flame/src/camera/viewfinder.dart': Failed assertion: line 61 pos 12: 'value > 0': zoom level must be positive: 0.0
Execute in a terminal and put output into the code block below
[√] Flutter (Channel stable, 3.47.9, on Microsoft Windows [Version 10.0.22631.6199], locale en-US)
• Flutter version 3.47.9 on channel stable
• Framework revision c197ca5199 (4 days ago), 2026-10-29 10:10:20 -0400
• Engine revision 89c39edfdf
• Dart version 3.13.7
• DevTools version 2.60.0
[√] Windows Version (11 Pro 64-bit, 23H2, 2009)
[√] Android toolchain - develop for Android devices (Android SDK version 36.1.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2026 18.11.2)
[√] Connected device (3 available)
[√] Network resources
• No issues found!
Affected platforms
All
Are you interested in working on a PR for this?
- I want to work on this
- Lingua principale
- Dart
- Stelle
- 10.8k
- Fork
- 1.1k
- Merge medio
- 1g 21h
- PR unite (30g)
- 62
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di flame-engine/flame
-
`Viewfinder.visibleGameSize` zooms in too far with a `FixedResolutionViewport`Forse già presa @ufrshubham l’ha presa oggi. Apertabug
flame-engine/flame#4203 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Camera bounds with `considerViewport: true` are not updated when the viewport is resizedForse già presa @ufrshubham l’ha presa oggi. Apertabug
flame-engine/flame#4202 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
`CameraComponent.setBounds` with `considerViewport: true` only works for bounds that start at (0, 0), at zoom 1Forse già presa @ufrshubham l’ha presa oggi. Apertabug
flame-engine/flame#4201 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
`findPostProcessFromContext` returns `null` for components rendered with `CameraComponent.postProcess`Forse già presa @ufrshubham l’ha presa oggi. Apertabug
flame-engine/flame#4200 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
`ComponentPool` returns components to the pool while they are still in use, and loses components removed before mountingForse già presa @ufrshubham l’ha presa 1 giorno fa. Apertabug
flame-engine/flame#4191 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di flame-engine/flame
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
lollipopkit/flutter_server_box#1851 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
fable-compiler/Fable#5252 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
chriskelly/wish-together#114 ·
I maintainer di solito rispondono entro 1 giorno
-
type:feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
mikejr83/rc-soar-score#724 ·
I maintainer di solito rispondono entro 1 giorno