#459 - status preview before submit #2002

Closed
jaredr wants to merge 2 commits from gitlab-mr-iid-732 into develop
Member

Closes #459

Closes #459
Member

This requires backend changes right? is it merged yet?

This requires backend changes right? is it merged yet?
Member

pleroma!995 seems to be merged

pleroma!995 seems to be merged
Member

this doesn't make a lot of sense, preview of an empty post is an empty post, there's little reason to show error, imo.

this doesn't make a lot of sense, preview of an empty post is an empty post, there's little reason to show error, imo.
Member

not liking the setTimeout hack here

not liking the setTimeout hack here
Member

as already said in previous MR (can you please stop multiplying MRs?) i would pretty much prefer preview be a live preview - post preview showing above/below/to the side of post input instead of hitting preview all the time. I think it would simplify code (just a separate entity updating its content) instead of adding more branching to existing code (are we previewing or posting? are we previewing or posting?)

as already said in previous MR (can you please stop multiplying MRs?) i would pretty much prefer preview be a live preview - post preview showing above/below/to the side of post input instead of hitting preview all the time. I think it would simplify code (just a separate entity updating its content) instead of adding more branching to existing code (are we previewing or posting? are we previewing or posting?)
Member

closing this as it's way behind develop and I made new MR for same issue: #2426

closing this as it's way behind develop and I made new MR for same issue: #2426

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
pleroma/pleroma-fe!2002
No description provided.