fix unnecessary bind group recreation during viewport panning

Author: Ayush2k02

desktop/src/render/state.rs

@@ -236,12 +236,20 @@ impl RenderState {
 			return;
 		};
 		let size = glam::UVec2::new(viewport_texture.width(), viewport_texture.height());
+
+		// Track whether the overlay texture is newly created or resized, since only then does the bind group need updating
+		let old_texture_size = self.overlays_texture.as_ref().map(|t| t.size());
+
 		let result = futures::executor::block_on(self.executor.render_vello_scene_to_target_texture(&scene, size, &Default::default(), None, &mut self.overlays_texture));
 		if let Err(e) = result {
 			tracing::error!("Error rendering overlays: {:?}", e);
 			return;
 		}
-		self.update_bindgroup();
+
+		let new_texture_size = self.overlays_texture.as_ref().map(|t| t.size());
+		if old_texture_size != new_texture_size {
+			self.update_bindgroup();
+		}
 	}
 
 	pub(crate) fn render(&mut self, window: &Window) -> Result<(), RenderError> {

node-graph/libraries/wgpu-executor/src/lib.rs

@@ -98,6 +98,11 @@ impl TargetTexture {
 		&self.view
 	}
 
+	/// Returns the size of the texture.
+	pub fn size(&self) -> UVec2 {
+		self.size
+	}
+
 	/// Returns a reference to the underlying texture.
 	pub fn texture(&self) -> &wgpu::Texture {
 		&self.texture