LogFX 1.3.0 - #35
Merged
Merged
Conversation
Replace Label with StackPane + Text in LogLine to avoid JavaFX's internal CSS cache leak caused by setStyle(). Programmatic setBackground() and setFill() persist on StackPane/Text since no CSS rules target those properties, working correctly in JavaFX 21+. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Set.copyOf() discards iteration order from the backing LinkedHashSet, so files were written to the config in arbitrary hash order. Use LinkedHashSet copy constructor to preserve insertion order. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
SerialGC does not proactively clear soft/weak references used by JavaFX's internal CSS caches, causing apparent memory growth in the jlink image. Use the default G1GC instead. Also remove --vm=server which was removed from jlink in JDK 14. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Delete existing build/image/ directory before running jlink (forcing files writable to handle protected files). After creating the image, zip it to build/logfx-<version>.zip. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Download jb and Install jb steps to build-mac and build-win jobs, and replace ./gradlew packageImage with jb/bin/jb -d jlink to match the build-linux job. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace three duplicate jobs (build-linux, build-mac, build-win) with a single matrix job, fixing the unresolved matrix.oskind reference. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace three duplicate jobs (build-linux, build-mac, build-win) with a single matrix job, parameterizing runs-on and the test report check name. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add jb_platform matrix variable with correct asset names (linux, macos-arm64, windows) for downloading jb releases. Also manually fixed the jb download and build steps in gradle.yml. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
build-properties.yaml file. Removed generation of Java code and Constants class.
Ensure the jb jlink task actually generates the splash screen image. Claude Code used to implement everything, including the "slick animation" of the splash screen!
The Java NIO zip filesystem does not preserve POSIX file permissions, causing executables like bin/java and bin/logfx to lose the executable bit. Use the system `zip -r` command on Unix which natively preserves permissions, falling back to the Java approach on Windows. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Allow providing a custom splash image path via the LOGFX_SPLASH_IMAGE environment variable, falling back to $JAVA_HOME/bin/logfx-logo.png. Also fix NPE when the splash image doesn't exist by checking that the stage has a scene before attempting the fade-out animation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Updated information about running LogFX. Updated .gitignore to ignore jb test build folder.
The project has migrated to jb as its build system. Remove all Gradle build files, wrapper scripts, and cache. Rename the GitHub Actions workflow from gradle.yml to jb.yml to reflect the change. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Small bug fixes.