search

rolopogo / ValheimMods

MIT License
20 stars 24 forks source link

Gizmo: pink wheels and reset options #13

Open rysson opened 3 years ago

rysson commented 3 years ago

First, awesome mod, thanks you very much!

  1. One small issue. On your clip (and icon) I see 3 colors, different in every plane. I see something pink/violet (I'm a man, sorry, it's not red :-)).
  2. Second issue is a little annoying. After press V, resetting works in first time. If I rotate in extra planes (+Shift or +Alt), switch off hammer, and again use it resetting rotate object (90° ? - I'm not sure) but not to 0°. I need use +Shift or +Alt to reset to vanilla position. – from README: Reset an axis by holding its modifier and hitting V (by default)

EDIT: afer reset: 20210313-114433-306x280

EDIT 2: I'm using Linux (Debian)

rolopogo commented 3 years ago

Oh that's interesting!

  1. The assetbundle containing the ring materials and shader was compiled for windows, so maybe the shader needs to be retargeted. I'll see if I can rebuild the assetbundle for linux and have the mod select the appropriate one at runtime.

  2. The reset is per axis, hitting V alone resets the vertical/Y/green axis. To reset the other two axes, hold their modifier key. Shift + V resets the X axis and LAlt + V resets Z. This is intended, but maybe unintuitive. I'll add a config option to reset all axes at once versus per axis.

rysson commented 3 years ago

Oh that's interesting!

  1. The assetbundle containing the ring materials and shader was compiled for windows, so maybe the shader needs to be retargeted. I'll see if I can rebuild the assetbundle for linux and have the mod select the appropriate one at runtime.

Thank you, It will be very nice. I read somewhere, that Valheim (or rather Unity) uses D11 (or similar) and Vulcan. I guess, that only Vulcan is avalaible in Linux.

  1. The reset is per axis, hitting V alone resets the vertical/Y/green axis. To reset the other two axes, hold their modifier key. Shift + V resets the X axis and LAlt + V resets Z. This is intended, but maybe unintuitive. I'll add a config option to reset all axes at once versus per axis.

Oh, sure, I miss it. You wrote in README: Reset an axis by holding its modifier and hitting V (by default) It's good idea. Option is even better idea. Maybe two hotkeys in config (per axis / all)?

boildead commented 3 years ago

I have same issue with pink wheels on Windows Vulcan version. I guess this is Vulcan issue.

rysson commented 3 years ago

Any chance to fix this issue for Vulcan?