Genii-Locorum / brp

Foundry VTT implementation for Basic Role Playing
Other
14 stars 2 forks source link

Nice to have: clicking on a skill on the character sheets opens the skill #8

Closed WisdomOfWombats closed 11 months ago

WisdomOfWombats commented 11 months ago

This is really just nice to have:

Lockbox313 commented 11 months ago

Left click will more than likely roll the skill unless it is edit mode.

Genii-Locorum commented 11 months ago

Yeah, my plan was left click to make a skill roll. I'm trying not to make the overall character sheet size bigger so keeping the amount of info that's visible down, but I can look to squeeze in a view icon if that'd help a lot. I also dont want to reduce the font size as I struggle enough already.

Is having a view icon a worthwhile trade off to a larger character sheet (may become a delete icon when the character sheet is unlocked)?

Lockbox313 commented 11 months ago

I would prefer context menu view in the non edit mode. The less clutter the better.

WisdomOfWombats commented 11 months ago

The skill role could be triggered by clicking the percentile number instead of the skill name. Just as an idea. But having a separate icon for opening a view is also a nice idea.

Genii-Locorum commented 11 months ago

I'll think on this. Both options have pros and cons.

Genii-Locorum commented 11 months ago

I'm going to leave things as they are for now. I've started work on Characteristic rolls so skill rolls will follow shortly afterwards. Let's revisit after that. I'll leave this open for now.

Genii-Locorum commented 11 months ago

OK, I've pushed this in to V1.7

Any feedback on this v old approach would be welcomed