dovrosenberg / foundry-simple-weather

Apache License 2.0
9 stars 5 forks source link

[BUG] Simple Weather fails to show configuration options inside of FoundryVTT Application interface #30

Closed ytnom-ctrl closed 8 months ago

ytnom-ctrl commented 10 months ago

Simple Weather fails to show configuration options inside of FoundryVTT Application interface. Also does not show configuration item on main Simple Calendar toolbar (non-compact)

Application:

image

Chrome WebBrowser:

image

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

dovrosenberg commented 10 months ago

That Simple Weather screen looks correct. By configuration options, do you mean things like whether to output to chat, let players see the dialog, etc? Because those are in the game settings:

image

If that's not what you meant, could you please clarify? Thanks!

ytnom-ctrl commented 10 months ago

Hi! Thanks for looking. The top screenshot from my report is from within the Foundry application. I don't use Chrome browser as a GM because I'm actually sitting at the machine running the Foundry server. It does not have the weather "gear" on the top right of the Calendar, so I cannot set the seasons, terrain, etc. Only from within the web browser. (2nd screenshot)

dovrosenberg commented 10 months ago

Ah. The top screenshot is actually Simple Calendar (the other module). Note that this is a separate app and has a separate box (just black and white vs. the colored Simple Calendar screen). The calendar side of Simple Weather looks similar to the condensed Simple Calendar display, so easy to confuse.

That said, is the Simple Weather box then not showing anywhere when you're logging in from Foundry directly? It should appear in the same place you left it if it was showing (in the same game) in Chrome...

ytnom-ctrl commented 10 months ago

yes, i have no view of simple calendar at all in the foundry Application.

only the configuration options for the server where you set to output to chat.

On Mon, Oct 30, 2023, 4:53 PM Dov Rosenberg @.***> wrote:

Ah. The top screenshot is actually Simple Calendar (the other module). Note that this is a separate app and has a separate box (just black and white vs. the colored Simple Calendar screen). The calendar side of Simple Weather looks similar to the condensed Simple Calendar display, so easy to confuse.

That said, is the Simple Weather box then not showing anywhere when you're logging in from Foundry directly? It should appear in the same place you left it if it was showing (in the same game) in Chrome...

— Reply to this email directly, view it on GitHub https://github.com/dovrosenberg/foundry-simple-weather/issues/30#issuecomment-1786222915, or unsubscribe https://github.com/notifications/unsubscribe-auth/APYHQP72KCZUEV4YSDUHLLLYCA4ZDAVCNFSM6AAAAAA6WV3SL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBWGIZDEOJRGU . You are receiving this because you authored the thread.Message ID: @.***>

dovrosenberg commented 10 months ago

Did it ever work, or has it always been missing? Are you logging in as a gamemaster or player?

ytnom-ctrl commented 10 months ago

Sorry for the delay in responding. Yes, it worked for me in 1.5.1 Logging in as Gamemaster.

dovrosenberg commented 10 months ago

For some other folks it came back in v1.7.2. Could you try upgrading and see if that fixes it? It not, I have some other ideas.

ytnom-ctrl commented 10 months ago

Ok, I'll check it out. On travel, back tomorrow

On Sun, Nov 12, 2023, 6:36 PM Dov Rosenberg @.***> wrote:

For some other folks it came back in v1.7.2. Could you try upgrading and see if that fixes it? It not, I have some other ideas.

— Reply to this email directly, view it on GitHub https://github.com/dovrosenberg/foundry-simple-weather/issues/30#issuecomment-1807388611, or unsubscribe https://github.com/notifications/unsubscribe-auth/APYHQP6UMEOV2MRH32H3GQTYEGBULAVCNFSM6AAAAAA6WV3SL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGM4DQNRRGE . You are receiving this because you authored the thread.Message ID: @.***>

dovrosenberg commented 8 months ago

Can't reproduce. Closing for now.