eternaldensity / Sandcastle-Builder

xkcd: 1190: Time: The Game
Other
81 stars 66 forks source link

More Molpy.Notify changes. #1426

Closed pickten closed 8 years ago

pickten commented 8 years ago

I'm adding on to Molpy.Notify in order to make the notifsilence option more customizable. Would anybody object if I removed the "log" flag, since it's easy to forget and since there's basically no reason to not use it? Likewise, what about "clas" or "details"? Are those two even used anywhere? If they should be kept, does anyone have other suggestions for how a new flag could get passed in (without making Molpy.Notify ugly)?

For those wondering, the idea is to remove them and add an "importance" flag. If it's 0, the notif will only show up on the "all notifs" option. If 1 (for error messages), it'll show up on that or "less notifs", and if it's 2 (for super-important stuff), it'll always show up.

pickten commented 8 years ago

Since it was merged...