jchambers / pushy-console

A simple GUI app for sending APNs (iOS/macOS/Safari) push notifications
MIT License
53 stars 18 forks source link

Selecting recent payloads inserts content instead of replacing #13

Closed genericwoods closed 5 years ago

genericwoods commented 6 years ago

When selecting recent payloads, it will append this content to whatever you’ve already entered. I think this should probably replace the entire payload, as the current content should already be saved in the recent entries list: screen shot 2018-06-20 at 5 06 17 pm

jchambers commented 6 years ago

This was actually a deliberate design decision. I was worried that "just replace" could accidentally lead to Angst™ if we let a user accidentally clobber a payload they'd been writing by hand. Maybe the answer is an undo/redo system, though.

jchambers commented 6 years ago

…also very open to the possibility that my deliberate design decision was, y'know, not the right one ;)

genericwoods commented 6 years ago

Yea I think an undo/redo could be built into the "recent payloads" section. What’s tricky is that because both the original and new payload were similar and large, it actually took me a minute to figure out there was a second payload hanging around above the one I created.