Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workshop Manager update #179

Merged

Conversation

samisalreadytaken
Copy link

General fixes include:

All workshop items are now fetched instead of only 50.

Status of preview image is displayed as text ("no image"/"loading") instead of always showing a black box.

Preview image is dynamically loaded on each key press on its input box, allowing user to easily look through file versions, or paste from clipboard.

Preview image downloads were assumed to be jpeg because regardless of the file type uploaded, Steam re-encodes images as jpeg to display on web pages; however internally stored files are unchanged, so this jpeg assumption would cause png preview images to fail to load. Images are now downloaded to memory and file type is correctly read.

Published files list default sort is fixed.

Non-ASCII input on title, description and change notes is enabled.

Image filters in file browser are grouped up into one filter. This is also the same behaviour of L4D2 & HLVR workshop managers.

Existence of preview image and addon content is now checked before upload. This fixes the issue of creating empty UGC items with invalid parameters.

Workshop item page is opened in Steam on successful upload. L4D2 & HLVR workshop managers open it on an internet browser, but doing it in Steam makes more sense.

Added localisation support. You will need to add the strings to entropyzero2_english.txt yourself: https://gist.github.com/samisalreadytaken/9fb86655a101e467c0c4cbc8b0008e69

Fetch all workshop items instead of only 50
Preview image is always loaded from user input
Download preview image to memory instead of file
Allow typing newline in input boxes
Allow non-ASCII character input in title, description and change notes
Group all image file browser filters into 1 filter
Always check if preview and content exist
Open Steam to file page on successful publish
@samisalreadytaken samisalreadytaken force-pushed the ez2/workshop_manager-patch branch 3 times, most recently from e801fd3 to af51bbe Compare January 28, 2023 15:14
Copy link

@1upD 1upD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I tested this out in-game and I was able to update a workshop addon. Text strings are working. There was an NPE crash when I did not have the localization strings - I intend to fix this myself in an upcoming PR which will enable JPEG decompression. For now, I would like to merge this PR

@1upD 1upD merged commit 43fd7e9 into entropy-zero:ez2/mapbase Jan 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants