Dave Rayment

Results 68 comments of Dave Rayment

@crutkas Thanks! It's a privilege to be able to contribute, and there's certainly no rush. Best of luck with the upcoming release 😀

@htcfreek Thanks for your comments and suggestions. I've updated the code with an error InfoBar, which is shown when a delete fails: ![image](https://github.com/user-attachments/assets/a19bda28-7414-4b4d-a5c7-b4799845efc9) I've added user-friendly messages for the most...

@jaimecbernardo @htcfreek @Jay-o-Way Thank you for the review and guidance. However, I disagree with displaying a prompt. The default Windows functionality for file deletions is not to show a confirmation...

Thanks, @jaimecbernardo Sorry, yes, you're right! I must have clicked the "Don't Show" checkbox when I initially started using Windows Photo. Please forgive the lapse in memory - it was...

> @daverayment using a `delete` function in most code languages (like `DeleteFile(item)` -> `FO_DELETE`) is very explicit - it doesn't care about a recycling bin. Code languages often have a...

> @daverayment , @jaimecbernardo For best user behavior the setting should be controllable from the dialogue in Peek (disable message) and the settings page (disable and enable) in PT Settings....

@htcfreek Yes, I'm still working on this. Sorry about the delay.

**Update: Please note that I've since simplified this dialog, and removed the workaround, as I'm unsure whether it is actually a WinUI issue or an intentional inconsistency.** I've updated the...

**Update: the bug is now fixed. Please disregard.** In further testing I have found a couple of issues with the new dialog, so please don't merge for the time being....

I have updated the delete confirmation dialog to be simpler: ![image](https://github.com/user-attachments/assets/006bf46d-adb5-4796-b818-8e114b814361) There was no need to include the filename, as it is obvious which file is current. This revision includes...