Thread: 5E - Token Height Indicator
-
August 7th, 2021, 01:48 #361
This is a version of 4.8 with some debug. This shouldn't be used by anybody unless they're having issues - the version on the forge is what should be used otherwise.
-
August 9th, 2021, 13:21 #362
- Join Date
- Aug 2019
- Posts
- 2,025
Small problem: Clicking on options always need one extra click to cycle back to the first option.
Big problem: Using the scroll-wheel without key-modifier does not zoom in/out of the map when the mouse-pointer hovers over a token.
-
August 9th, 2021, 13:51 #363
GKEnialb, I may be able to help you with that issue since I already solved that in my extension; actually Moon Wizard solved this issue
(though I am guessing now, I did not yet test your extensions)
Change your onWheel function to the following:
Code:function onWheel(tokenCT, notches) if Input.isAltPressed() then TokenHeight.updateHeight(tokenCT, notches, true); return true; end end
This method improves compatibility with the actual code and also won't overwrite the onWheel code when just the mouse wheel is used while hovering over a token. Then Weissrolf's issue should be solved since the original code actually already adds zooming while hovering over a token and using the mouse wheel
Your method executes always return true, leading to that the code for a pure mouse wheel without using shortcuts while hovering over a token is overwritten. With Moon Wizard's method you then do not need to figure out adding the zooming while hovering over a token (and neither do you need to maintain the code for the other shortcuts which can be a mess; your shift-if-clause may for example not work completely on hex grids or other type of grids in general since adjacent fields are not always 8. So, this issue may be then solved, too )Last edited by Kelrugem; August 9th, 2021 at 14:06.
My extensions for 3.5e and Pathfinder
Bug reports please here
-
August 9th, 2021, 14:25 #364
- Join Date
- Aug 2019
- Posts
- 2,025
This seems to have done the trick. Thanks Kelrugem!
-
August 9th, 2021, 14:46 #365
Solution:
Code:function registerOptions() OptionsManager.registerOption2 ( "THIALLOWUSERADJUST", false, "option_header_height_indicator", "option_label_allow_player_mod", "option_entry_cycler", { labels = "option_val_no", values = "no", baselabel = "option_val_yes", baseval = "yes", default = "yes" } ) OptionsManager.registerOption2 ( "THIPOSITION", false, "option_header_height_indicator", "option_label_height_position", "option_entry_cycler", { labels = "option_val_bottom_left|option_val_left|option_val_top_left|option_val_top|option_val_top_right|option_val_right|option_val_bottom_right", values = "bottom left|left|top left|top|top right|right|bottom right", baselabel = "option_val_bottom", baseval = "bottom", default = "bottom" } ) OptionsManager.registerOption2 ( "THIFONT", false, "option_header_height_indicator", "option_label_font", "option_entry_cycler", { labels = "option_val_large", values = "large", baselabel = "option_val_medium", baseval = "medium", default = "medium" } ) OptionsManager.registerOption2 ( "THIFONTCOLOR", false, "option_header_height_indicator", "option_label_font_color", "option_entry_cycler", { labels = "option_val_medium|option_val_light", values = "medium|light", baselabel = "option_val_dark", baseval = "dark", default = "dark" } ) OptionsManager.registerOption2 ( "THIDIAGONALS", false, "option_header_height_indicator", "option_label_variant_diagonals", "option_entry_cycler", { labels = "option_val_long", values = "long", baselabel = "option_val_short", baseval = "short", default = "short" } ) OptionsManager.registerCallback("THIALLOWUSERADJUST", setPlayerControl) OptionsManager.registerCallback("THIPOSITION", changeOptions) OptionsManager.registerCallback("THIFONT", changeOptions) OptionsManager.registerCallback("THIFONTCOLOR", changeOptions) setFont() end
bmos' extensions
he/them
-
August 10th, 2021, 01:53 #366
Thanks, Kelrugem and bmos! Uploaded v4.9 to the first page and the Forge (still processing in the Forge as I write this) with both of those fixes. Wish I had known about the onWheel thing at the beginning - could never quite figure out that behavior. And also didn't know about not duplicated the baselabel/baseval. Great to learn new things.
-
August 10th, 2021, 22:02 #367
-
August 10th, 2021, 22:41 #368
It is live on the forge as of now. It took long enough to process last night that I got distracted by a shiny object and didn't check later, so thanks for the reminder.
-
August 16th, 2021, 20:52 #369
- Join Date
- Jul 2016
- Posts
- 40
I recently updated to 1.2 of Kent Height and 4.9 of Token Height and when i put a token 20' in the air and next to another token it is rolling with disadvantage with a ranged attack. It was working previously.
-
August 16th, 2021, 22:41 #370
Thread Information
Users Browsing this Thread
There are currently 1 users browsing this thread. (0 members and 1 guests)
Bookmarks