Artemystra / eclipsephase

The second edition character sheet for FoundryVTT
18 stars 17 forks source link

Restore Rest (All Players) Button #71

Closed herrozerro closed 8 months ago

herrozerro commented 10 months ago

I get what the feature is trying to do, it will refresh active sheets attached to players. But I was just running a session with multiple fork sheets per person and it was a hassle to reset all of the sheets manually. Perhaps another button called Restore Rest (All PC Sheets)?

Artemystra commented 10 months ago

This is possible, but it would limit the GM to use Player sheets if they want to. But in general this is possible and easily to be added to the urrent system.

herrozerro commented 10 months ago

Okay I can see the issue. What about allowing players to have access to the reset rest button? In general I trust my players and would like them to have full control of their sheets

johnhelmuth commented 9 months ago

I think having a UI element that pops up a window with a list of player sheets each with a checkbox. The current players' sheets would be checked already. The GM could check the additional sheets they want to "Restore rest", and then clicks the action button to do the thing.

Edit: So I shouldn't run off the handle without digging into the actual functionality of the feature in question.

Why would adding all Player Character Actors to the list, and only checking the active player Actors in the list, prevent the GM from using Player Character actors if they want?

I feel like I'm missing something.

Artemystra commented 8 months ago

A fix for this has released with v1.0.2