Close Dialogs on a Timer about adarkroom

[jernaumorat]

I don't honestly see why that is an issue. They're there to be bought, let the people buy them. As long as it isn't enabling duplication or another kind of exploitation, it isn't an actual issue.

[Continuities]

I'm with @vermilingua on this one. Closing the dialog automatically is a no-go, since ADR is based around idling. We can't punish the player for taking too long to revisit the tab. I don't think this issue is gamebreaking, and this is probably a case of the cure being worse than the disease.

[anubisthejackle]

I agree with not closing it automatically. One change to this could be to capture the fur count with the first click of a purchase button, so when the dialog appears they can choose to idle and collect fur, and purchase when they're ready.

On the flip side of this, I'd also be 100% behind making the purchase boxes reactivate if the user idles after they've grayed out. I just feel the way it currently is, we're half in and half out.

[Continuities]

That's a good point, @anubisthejackle. I'm totally cool with reactivating them as your resources replenish.

[jernaumorat]

Could you clarify? To me at least, it isn't entirely clear what you're getting at. Do you want only a set amount of goods bought, based off the initial count of fur; or do you want the button to de/reactivate based on the current fur count?

[Continuities]

The second one. The buttons already disable when you can't afford the purchase, but they don't enable again as your resources increase while the dialog is open.

[jernaumorat]

Ah right,I wasn't aware of that, I always bought/$SMed the trading Post in
whenever it was needed.
On 16/07/2014 12:05 AM, "Michael Townsend" [email protected] wrote:

The second one. The buttons already disable when you can't afford the
purchase, but they don't enable again as your resources increase while the
dialog is open.

—
Reply to this email directly or view it on GitHub
https://github.com/Continuities/adarkroom/issues/108#issuecomment-49036017
.

[anubisthejackle]

Issue fixed with Pull request #120 so closing ticket out.

[groteworld]

as a bit of Github-Fu if you say something like resolves #120 or closes #120, it will auto close it on merge. Thought that will help!