crnormand / gurps

Implementing a GURPS 4e game aid for Foundry VTT
MIT License
105 stars 49 forks source link

Feature request - Customisable and floating Roll 3d6 button #1942

Closed Xain3 closed 5 days ago

Xain3 commented 3 months ago

The Roll 3d6 button is the most GURPSy thing that GURPS for Foundry has - I am fully ready to admit that. However, sometimes, when you use a lot of modules like I do, the button becomes a bit cumbersome. It would be great if the button could be resized, moved, or maybe (Steve Jackson forgive me for what I am saying) hide it altogether. Less crucial, but since we're going down this road why not also give the option to customise the button image? This way we could maybe adapt it to the setting and the tone of the campaign?

chrismaille commented 2 weeks ago

Current solution:

Settings:

image

Positions are:

Bucket at left (default position, custom 3d6 dice):

image

Bucket at right, defaut 3d6 icon

image

IHappyEndI commented 2 weeks ago

@chrismaille Hi! Thanks for implementing the idea, but there is one small bug. If you use a user interface module that centers the macro panel, then with these changes applied, this does not happen, and the macro panel remains in its normal position

chrismaille commented 2 weeks ago

Hi @IHappyEndI , I did not found any v12 certified modules which changes the toolbar position. Which one your're using?

IHappyEndI commented 2 weeks ago

@chrismaille for example RPG Styled UI

chrismaille commented 2 weeks ago

I see. RPG Styled UI is a v11 module that mainly hardcode the hotbar CSS in the new position. I think it's risky to try to support this changing foundry interface CSS like they do. But I might look at this module again if they port it to v12.

As a workaround, my suggestion is to use the right option. Please let me know if this helps.

IHappyEndI commented 2 weeks ago

@chrismaille I'm using a fork of this module, I don't know how much suitable it is for use in V12 - https://github.com/Oxy949/rpg-styled-ui . Unfortunately, using different positions does not change anything ; ( . But anyway, thanks for the new feature! image