Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

When using 'delete and redraft', clicking outside the pop-up #77

Open
strypey opened this issue Oct 4, 2023 · 3 comments
Open

When using 'delete and redraft', clicking outside the pop-up #77

strypey opened this issue Oct 4, 2023 · 3 comments
Labels
💥 broken Something that is broken

Comments

@strypey
Copy link

strypey commented Oct 4, 2023

Description of the issue

When using 'delete and redraft', clicking outside the pop-up causes me to completely lose my draft.

How you found the issue

Steps to reproduce the behavior:

  1. Go to an existing post
  2. Click on the 3 dots menu at the bottom
  3. Select 'delete and redraft'
  4. Click anywhere outside the popup
  5. The popup disappears and the contents on the post with it.

What you expected to happen

  • On a reply; I expect the text to appear as a draft in the reply field, under the post to which it was a reply.
  • On an original post; I expect the text to appear as a draft in the posting field on the home tab
@strypey strypey added the 💥 broken Something that is broken label Oct 4, 2023
@easrng
Copy link

easrng commented Oct 7, 2023

Reopen the compose dialog by scrolling down on the timeline and clicking the new post button.

@strypey
Copy link
Author

strypey commented Oct 9, 2023

@easrng:

Reopen the compose dialog by scrolling down on the timeline and clicking the new post button

That works, but that "new post" button is extremely non-obvious, and there's no obvious reason to expect it would bring back the pop-up. It would be more obvious, and require fewer clicks, if clicking delete-and-redraft on a;

a) Reply; turned the post back into the kind of draft it was before being posted

b) Post; dropped the draft back into the compose window at the top of the timeline, where it was before being posted

I'm not sure why that pop-up modal exists. The only feature that uses it AFAICT is delete-and-redraft, possibly because it was a bolted-on proof-of-concept, that Nolan never got around to replacing. If you modified the UI to work as I describe above, I think you could completely remove the code for that pop-up window.

Bonus; an edit function could be added to the UI by doing the delete-and-redraft without the delete, and plugging the posting logic into the post editing part of the API, instead of the new post one.

@easrng
Copy link

easrng commented Oct 9, 2023

Bonus; an edit function could be added to the UI by doing the delete-and-redraft without the delete, and plugging the posting logic into the post editing part of the API, instead of the new post one.

I already implemented that in my fork :3 (easrng/enafore, enafore.social)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💥 broken Something that is broken
Projects
None yet
Development

No branches or pull requests

2 participants