notedeck icon indicating copy to clipboard operation
notedeck copied to clipboard

Post confirmation and preview

Open jb55 opened this issue 1 year ago • 7 comments

We should have an optional post confirmation screen that shows a preview of the rendered post before you click send.

jb55 avatar Jul 01 '24 14:07 jb55

Maybe we should have this on iOS as well ? @danieldaquino

rock suggested a separate preview button so you can skip it if you don't want to preview

jb55 avatar Jul 01 '24 14:07 jb55

@jb55 what is the underlying issue that we are trying to resolve with this post preview?

danieldaquino avatar Jul 03 '24 17:07 danieldaquino

@danieldaquino I pasted a URL and accidently added an extra character on the end, a preview would have caught that. Also when you are tagging people or pasting npubs, it can get screwed up if you backspace too many times. It's just a quick "check to make sure I didn't screw up something" before you post.

Alternatively we can just try to ensure we always have a rich text preview, but rich text previews can obfuscate the underlying text sometimes.

jb55 avatar Jul 04 '24 15:07 jb55

On iOS we don't have 'preview post' feature request. However, we have 10 second undo post functionality:

https://github.com/damus-io/damus/issues/801

alltheseas avatar Jul 25 '24 12:07 alltheseas

@robagreda @karnagebitcoin what is a best UI practice?

  1. preview post, or
  2. 10 second undo?

alltheseas avatar Jul 25 '24 13:07 alltheseas

I would say 10 second undo!

robagreda avatar Jul 25 '24 14:07 robagreda

@robagreda @karnagebitcoin what is a best UI practice?

  1. preview post, or
  2. 10 second undo?

Hard to decide. 10 sec is a long time to show a notice. I imagine you wouldn't do that, and only have it in note options - but that might not be obvious the first time. I kinda like the idea of a preview button, but might be more cumbersome on mobile. 10 sec undo in options might be the easier option to implement and solves the occasional mis-formatted post issue

karnagebitcoin avatar Jul 26 '24 13:07 karnagebitcoin