[Utility] Clock Adjuster

[nephranka]

Yeah, in my branch to maintain FGC compatibility, I changed those to be defaults set in options. Sorry if there was anything accidentally pulled in from my branch.

So we can set them in the options now? That would work as well. My experience is that they reset to zero each load. I will have to take a look.

[MrDDT]

To make it more obvious since the chat messages are easy to miss?

Exactly, as a DM you have a lot of stuff going on and might miss it. But pop ups are hard to miss.

[bmos]

Yeah, in my branch to maintain FGC compatibility, I changed those to be defaults set in options. Sorry if there was anything accidentally pulled in from my branch.

ah! that's what that was about.
Looks like the options themselves didn't come over. nephranka/anyone, do you have a preference between fields that hold onto their numbers or a default in options?

[nephranka]

ah! that's what that was about.
Looks like the options themselves didn't come over. nephranka/anyone, do you have a preference between fields that hold onto their numbers or a default in options?

I think the long/short rest time fields stay set to the last entry so it would make sense the adj hours/mins fields behave the same way. That said if they all are options or all use "stay set" works equally well for me.

Edit: So in the new version the long/short rest time field are now not saved as well. I see now what is going on. So either as options or last set fields works for me.

[JustinFreitas]

The reason I did options in my FGC branch is because I usually always have my long/short rest the same at 8hr / 60min. I tried a 'last used' implementation but ended up back at options in the end. If you need me to look at anything when you pull whatever in, just add me as a reviewer to the PR and I'll give it a look!

[bmos]

Justin posted a pull request to re-enable saving those values (both methods actually, very generous -- i went with the 'last used' method) and I made a few more improvements to clean things up.
https://www.fantasygrounds.com/forum...l=1#post608785

[nephranka]

Justin posted a pull request to re-enable saving those values (both methods actually, very generous -- i went with the 'last used' method) and I made a few more improvements to clean things up.
https://www.fantasygrounds.com/forum...l=1#post608785

I just happen to be here and saw this. When I follow the link and click the ext to download, I get:
"Invalid Attachment specified. If you followed a valid link, please notify the administrator"

[JustinFreitas]

I just happen to be here and saw this. When I follow the link and click the ext to download, I get:
"Invalid Attachment specified. If you followed a valid link, please notify the administrator"

Link works for me but I’m just on my phone right now and haven’t made it to my computer today. I’ll try it there in a few minutes.

[nephranka]

Just tried again and it downloaded. All good. Thanks for this update!

[JustinFreitas]

Link works for me but I’m just on my phone right now and haven’t made it to my computer today. I’ll try it there in a few minutes.

Yeah, it's working on the desktop too. That's weird, nephranka. Maybe try again from an Incognito tab or something?

Edit: Ah, it's fixed. Nice!