`1bacf68`

fix: use default wgpu limits to support large windows

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 3 months ago

SHA: 1bacf682581488b04cb8be0950e0128ca619038d
Parents: 8d93a2b
Tree: 21cf9b7

1 changed file

Status	File	+	-
M	`garterm/src/render/gpu.rs`	8	5

garterm/src/render/gpu.rsmodified

          let display = xlib_display.display_ptr();
          let screen = xlib_display.default_screen();
 -        // Prefer GL over Vulkan for better X11 compositor integration
 +        // Try Vulkan first (higher texture limits), fall back to GL
 +        // GL has a 2048 texture limit on some systems which breaks full-screen windows
          let instance = wgpu::Instance::new(&wgpu::InstanceDescriptor {
 -            backends: wgpu::Backends::GL,
 +            backends: wgpu::Backends::VULKAN | wgpu::Backends::GL,
              ..Default::default()
          });
 -        tracing::info!("Using OpenGL backend for better compositor compatibility");
          let handle = XlibWindowHandle::new(window, display, screen);
              .await
              .ok_or(GpuError::NoAdapter)?;
 -        tracing::info!("Using GPU adapter: {:?}", adapter.get_info().name);
 +        let adapter_info = adapter.get_info();
 +        tracing::info!("Using GPU adapter: {:?} (backend: {:?})", adapter_info.name, adapter_info.backend);
          let (device, queue) = adapter
              .request_device(
                  &wgpu::DeviceDescriptor {
                      label: Some("garterm"),
                      required_features: wgpu::Features::empty(),
 -                    required_limits: wgpu::Limits::downlevel_webgl2_defaults(),
 +                    // Use default limits (8192 max texture) instead of downlevel (2048)
 +                    // to support full-screen windows on high-res displays
 +                    required_limits: wgpu::Limits::default(),
                      memory_hints: wgpu::MemoryHints::Performance,
                  },
                  None,