-
Notifications
You must be signed in to change notification settings - Fork 44
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
Add a way for users to contribute token overlays #20
Labels
Comments
JamzTheMan
added a commit
that referenced
this issue
Mar 29, 2019
* #7: Add PDF Extraction Initial check-in of PDF Extraction code, working beta 1! Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Adjustments for better window resizing. Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Updated travis. Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * Cleanup * Ignore eclipse .settings * Updated PDFBox libs * Updated Gradle Wrapper to 4.7 Task-Url: JamzTheMan/MapTool#37 Signed-off-by: Jamz <[email protected]> * PDF Enhancements * Removed ImageGallery fxml/controller - unneeded complexity * Added PDF Window i18n title * Changed sentry.io logging to ERROR * Added caching of main PDF image render * Fixed auto resizing of main PDF image * Added drag/drop from PDF extract buttons Signed-off-by: Jamz <[email protected]> * #9: Pog filenames substitute %20 for spaces * Properly decode file names for saving to file * minor cleanup on other files Task-Url: http://github.com/JamzTheMan/TokenTool/issues/9 Signed-off-by: Jamz <[email protected]> * #10: Add arrow key nudges * Added 1 pixel nudge on arrow keys Task-Url: http://github.com/JamzTheMan/TokenTool/issues/10 Signed-off-by: Jamz <[email protected]> * #13: Allow user supplied image for background - WIP * Added button to supply background image * updated token preview clipping to use background image Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Updated Travis CI to install latest java via brew * Updated config to use environment variables from Travis config. Should be generic now for upstream repo Task-Url: JamzTheMan#12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Updated Travis CI to install latest java via brew * Updated config to use environment variables from Travis config. Should be generic now for upstream repo * Updated Appveyor config * WIP - working on errors in travis... Task-Url: JamzTheMan#12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Removed DLL from Inno Setup template * Add TokenTool.iss to .gitignore as this is built for every deploy * Moved Vendor to gradle.properties so it's static for the repo Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Version now grabbed from Tag. If no tag, version = SNAPSHOT+Commit Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Added missing platform x64 config for appveyor... Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Changed gradle deploy to run only 'before_deploy' (was always running) Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * Update .appveyor.yml * #8: Bug in pdf-extract-feature branch * Can now drop PDF files to open them * ImageIO lib updated, no longer throws error Task-Url: http://github.com/JamzTheMan/TokenTool/issues/8 Signed-off-by: Jamz <[email protected]> * #14: Add a keyboard shortcut to take a screen capture. * Added shortcut to Capture Screen menu option per request * Updated and added other shortcuts to menu options Task-Url: http://github.com/JamzTheMan/TokenTool/issues/14 Signed-off-by: Jamz <[email protected]> * Layer support WIP * #13: Allow user supplied image for background * Added drag and drop areas to denote if image is portrait or background * Added layer selection menu button to drive scroll/rotate events WIP * Changed Preference saving to handle ImageView's a little easier * Added system.exit to kill any lingering threads from PDF extractions * Updated Paste/Screen Capture/PDF Extract to put image on currently selected layer * All child windows now remember position and size * Swipe touch event should move 1 page at a time now * New menu option to Reset Settings to factory defaults Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * Logging and Build changes * Sentry now logs OS * Sentry now configures as Development on non tagged commits. * Sentry bumped to 1.7.5 * gradle.properties now hole vendor and sentry DSN for more flexibility * Adding a tag now configures Sentry to Production mode, sets Release version and kicks off Deployment in CI Signed-off-by: Jamz <[email protected]> * #13: Allow user supplied image for background * Help -> Reset menu now has confirmation/cancel dialog * FXML/UI tweaked to better handle ImageViews to remain centered on stage resize (background and portrait) and refactored code to simpler binding Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * #13: Allow user supplied image for background * Right click on pdf extract sets background vs portrait * Sentry.io only logs in Production mode now * WIP: Portrait naming Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * #17: Imported PNG overlays coming in at 100x100 * Fixed png thumbs being imported at default thumb size of 100 Task-Url: http://github.com/JamzTheMan/TokenTool/issues/17 Signed-off-by: Jamz <[email protected]> * #19: Add ESC key to cancel Screen Capture * Added esc key event capture * Added highlighting to drag icons Task-Url: http://github.com/JamzTheMan/TokenTool/issues/19 Signed-off-by: Jamz <[email protected]> * #13: Allow user supplied image for background * Final tweaks on new save options, can now save portrait with backgound options on drag and drop along with normal token image Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * New Overlays * A few new overlays added * AppSetup will only install new overlays since last version installed * TODO: i18n properties still need to be updated... Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * Misc * Added some card type overlay bases * Added label under overlay name above overlay tree to dispaly native overlay dimensions, eg 512 x 512 * Updated i18n files with new externalized strings Task-Url: http://github.com/JamzTheMan/TokenTool/issues/13 Signed-off-by: Jamz <[email protected]> * Misc * i18n'd Layer Menu button Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Fixed bug with brew per Homebrew/homebrew-cask#49716 Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction * PDF image extraction now on it's own service thread * Added progress indicator for long image extractions Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #7: Add PDF Extraction * Spotless... * Tagged! Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * getting the appveyor key to work? Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * Another try at env vars Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * #12: Update CI config and install to Java 10 * YAT Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * Still testing AppVeyor... * YAT WTF Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]> * 12: Update CI config and install to Java 10 Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12 Signed-off-by: Jamz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If users want to contribute them, they would currently need to fork the repo and submit a PR. That's a lot of effort for someone who just wants to contribute some graphics assets back to the community.
Ideally, these assets would be stored on RPTools.net and TokenTool could check at launch for new assets and offer to download them (caching anything previously downloaded). Then contributing new ones amounts to adding an upload facility for RPTools.net, perhaps by a WP page that allows users to contribute and our Bard (RPTroll) to "accept" them for "publication" (i.e. approve them to be copied into the distribution directory where users can get them).
We'll need a policy that says we can reject images we don't consider in good taste, and we'll need support both on RPTools.net and in TokenTool...
The text was updated successfully, but these errors were encountered: