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

Unable to resize image in the token tool window? #19

Closed
bschollnick opened this issue Feb 9, 2019 · 6 comments
Closed

Unable to resize image in the token tool window? #19

bschollnick opened this issue Feb 9, 2019 · 6 comments
Labels

Comments

@bschollnick
Copy link

Maybe I missed something, but I can't find any documentation for Token tool?

If this feature doesn't exist currently, then please consider this a feature enhancement request.

If I import a 800x1029 (eg.
deviantart_264864091_why not zoidberg image, I see no way to resize the image to fit a decent portion of it into the 256x256 token.

Yes I could resize the token larger, but wouldn't I then need to resize the token smaller in Maptool to fit into a 1in sq?

Now, I'm new to this, I just started using maptool a week or two ago, so I'm still learning.

But it seems like we should have some degree of image resizing in token tool, to help us capture the portion of the token that we would like.

@Phergus
Copy link
Contributor

Phergus commented Feb 9, 2019

  1. Use the mouse wheel to resize either up or down.
  2. No, you wouldn't have to resize the token smaller in MapTool. The default is that when you drop an image on to a map in MapTool, it is set to a single square/hex in size. Technically the size is the map's Pixels per Grid setting. You can also set a tokens size via the right-click menu.

Questions like these are easily answered on the forums or the Discord server.

@Azhrei
Copy link
Member

Azhrei commented Feb 9, 2019

But there should be some kind of documentation that tells users to use the mouse wheel. At least in MapTool, there's the tips that show up in the status bar at the bottom of the window... Something to think about for the next iteration of TT.

@JamzTheMan
Copy link
Member

Agreed, we need so doc writers! 😄

Is there a doc for TT 1.x? It's mostly the same instructions with some additions (like meta+mousewheel to rotate as well)

@Azhrei
Copy link
Member

Azhrei commented Feb 9, 2019

What? Documentation? Pfft, we don't need no stinkin' documentation!

Um, no, I don't think there's anything for TT 1.x either. We've had some forum folks say they were going to try to write some docs, but it's a daunting task, particularly going back and updating stuff when the software changes...

@Phergus
Copy link
Contributor

Phergus commented Feb 10, 2019

Wait a sec. There's rotation? Somebody should document this tool.

Apparently it's Shift-Mousewheel.

@Azhrei
Copy link
Member

Azhrei commented Feb 10, 2019

@Phergus "Wait a sec. There's rotation? Somebody should document this tool."

You're hired! 😁

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 <Jamz@Nerps.net>

* #7: Add PDF Extraction

Adjustments for better window resizing.

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #7: Add PDF Extraction

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #7: Add PDF Extraction

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #7: Add PDF Extraction

Updated travis.

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #7: Add PDF Extraction

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #7: Add PDF Extraction

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* Cleanup

 * Ignore eclipse .settings
 * Updated PDFBox libs
 * Updated Gradle Wrapper to 4.7

Task-Url: JamzTheMan/MapTool#37

Signed-off-by: Jamz <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* 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 <Jamz@Nerps.net>

* Misc

 * i18n'd Layer Menu button

Signed-off-by: Jamz <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #7: Add PDF Extraction

 * Spotless...
 * Tagged!

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/7

Signed-off-by: Jamz <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #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 <Jamz@Nerps.net>

* #12: Update CI config and install to Java 10

 * YAT

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12

Signed-off-by: Jamz <Jamz@Nerps.net>

* Still testing AppVeyor...

* YAT WTF

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12

Signed-off-by: Jamz <Jamz@Nerps.net>

* 12: Update CI config and install to Java 10

Task-Url: http://github.com/JamzTheMan/TokenTool/issues/12

Signed-off-by: Jamz <Jamz@Nerps.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants