unsoluble / smalltime

A small FoundryVTT module for displaying and controlling the current time of day.
MIT License
24 stars 15 forks source link

Issue with visibility #87

Closed remiverdel closed 2 years ago

remiverdel commented 2 years ago

Hi,

Thanks for the great module. Just saw a problem on the visibility side of things. I checked visible for player to 'none' but when I try to connect as a player, I still see the smalltime window and can move it around. I sure can't interact with it, but the functionality doesn't seem to correctly hide it for the players.

Cheers,

unsoluble commented 2 years ago

Hmm! Just to be sure, did you only change the default for this setting in the Module Settings screen perhaps, and not the actual scene-specific setting in the config for the scene itself?

remiverdel commented 2 years ago

Hi John,

I just changed the default option

Le 17 sept. 2021 à 18:46, John Weston @.***> a écrit :

Hmm! Just to be sure, did you only change the default for this setting in the Module Settings screen perhaps, and not the actual scene-specific setting in the config for the scene itself?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/unsoluble/smalltime/issues/87#issuecomment-921937714, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFOV557CGBJFMVQ6ZTQLGDTUCNWGFANCNFSM5EH5QMCQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

unsoluble commented 2 years ago

Yeah so that default setting will apply to any new scenes you create from this point forward, but any existing scenes that you'd like to behave this way you'll need to go into their scene settings and explicitly choose this view permission.

remiverdel commented 2 years ago

Oh thanks! I didn’t know these settings were there. My bad! Thanks again for such a module

Cheers,

Le 17 sept. 2021 à 18:48, John Weston @.***> a écrit :

Yeah so that default setting will apply to any new scenes you create from this point forward, but any existing scenes that you'd like to behave this way you'll need to go into their scene settings and explicitly choose this view permission.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/unsoluble/smalltime/issues/87#issuecomment-921939138, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFOV556EK2KN6GLZQDI4CZLUCNWPJANCNFSM5EH5QMCQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

unsoluble commented 2 years ago

No prob! Glad you're liking it. :)