fix: prevent crash opening screenshots with no recent

Author: HamaIndustries

src/client/java/dev/spiritstudios/snapper/Snapper.java

@@ -61,7 +61,12 @@ public void onInitializeClient() {
                 client.player.sendMessage(Text.translatable("text.snapper.panorama_success", SCREENSHOT_MENU_KEY.getBoundKeyLocalizedText()), true);
             }
             while (RECENT_SCREENSHOT_KEY.wasPressed()) {
-                File latestScreenshot = ScreenshotActions.getScreenshots(client).getFirst();
+                List<File> screenshots = ScreenshotActions.getScreenshots(client);
+                if (screenshots.size() == 0) {
+                    if (client.player != null) client.player.sendMessage(Text.translatable("text.snapper.screenshot_failure_open"), true);
+                    continue;
+                }
+                File latestScreenshot = screenshots.getFirst();
 
                 client.setScreen(new ScreenshotViewerScreen(
                         ScreenshotImage.of(latestScreenshot, client.getTextureManager()),

src/client/resources/assets/snapper/lang/en_us.json

@@ -27,6 +27,7 @@
   "text.snapper.rename_invalid": "New name for screenshot invalid",
   "text.snapper.rename_invalid_png": "Name must end with '.png'",
   "text.snapper.screenshot_instructions": "%s. View by pressing %s",
+  "text.snapper.screenshot_failure_open": "Take a screenshot in order to view it",
   "panorama.snapper.failure": "Couldn't save panorama: %s",
   "panorama.snapper.success": "Saved screenshot as %s"
 }