[Utility] Clock Adjuster

Quote:

---

Originally Posted by JustinFreitas

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.

Quote:

---

Originally Posted by bmos

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.

Quote:

---

Originally Posted by JustinFreitas

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?

Quote:

---

Originally Posted by bmos

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.

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!

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

Quote:

---

Originally Posted by 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

---

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"

Quote:

---

Originally Posted by nephranka

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.

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

Quote:

---

Originally Posted by 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!