Dev #55
Merged
Dev #55
Conversation
This file contains 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
This could conflict with other imgui application in same folder, and the Server doesn't really need any Imgui settings saved
-Invalid GPU resource release on Com Thread -TextureId cast error -Unneeded Windows specific include
Default is unused. Prevents having 2 running NetImgui Servers listening on same port. Seems like the 'force listen' isn't needed on Windows/Linux, but can still toggle it on if needed.
-Small changes done to inputs
Possible error for platorm using top bit in 64bits in VirtualAddress space
updated msbuild setup
Update to dotnet installed (needed for sharpmake)
* First Prototype for Custom Texture Work not completed, but give an overview of the feature direction * Custom Texture Sample update Working on a CustomTexture basic example, for reference * Finalizing Custom Texture handling and sample -Moved custom texture code to new file -Added sample to showcase usage -Removed console command for safety * Replace Server texture list with hashmap -Faster lookup when handling many textures -Fixed relasing HAL texture objects when client disconnect
Signed-off-by: Kostiantyn Chernenok <[email protected]>
-Server and Client now are monitor DPI aware. User can adjust how much text scaling should occurs to compensate for small text on high resolution screens -Was unable to save config when file was set to 'readonly'. This situation could happen when file is submitted to a source control. Now create a second _user config file, if default one is readonly -Fixed crash when closing Server application. Tied to Server Listening thread with socket never timeout and unable to exit
- NetImgui Server application now support monitor DPI awareness
- By default, use FontGlobalScale to draw text in bigger font size
- If user provide a callback when initiating a connection, will use this callback to request a font update with bigger font sizes
- This callback allows for sharper text renderer at desired size
- Added support for 'Shared Config' client
- Enabling this option in the client config, will make this client available no matter where the NetImgui Server Application is launched from
- This is achieved by saving in the user OS shared config folder (instead of working directory)
- Also detect if the config file is readonly and create a second one that's writable if this happen
- This is to handle people checkin in the netimgui.cfg in a source control and leaving it readonly, we want user to still be able to add clients
- Can be useful distribute default settings to users, then they can add their own clients/settings
- In compatibility test files, stop downloading Imgui versions when already there - Fixed a clang compilation warning
- Added copy/paste of text accross Server/Client - Fixed a keys reception issue cause letter drop when quickly typed.
- Instead of scaling up rendering, create new font texture with DPI aware characters. Gives sharper results
- Updated sample code to handle DPI - Fixed some texture update issue on server
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Moving Dev to Master for next release