Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate edit and preview view for posts and comments #611

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lindskog
Copy link
Contributor

@lindskog lindskog commented Feb 17, 2021

Fixes #537

What's the problem you solved?

Message preview seems to submit the contents of the preview, even if additional edits have been made (issue #537)

What solution are you recommending?

Instead of the recommended solution in the issue I discussed this with some people and suggest that edit and preview of posts and comments are separated into two different views, to create a more natural and less confusing user/data flow.

@Powersource
Copy link
Collaborator

Thanks for the PR!

Tried it out and seems to work well.

But the user flow when mentioning someone gets a bit weird since the people you might mention are showed to you on the preview screen and if you want to mention them you have to copy the mention and then click Edit to go back and use the mention. But maybe this is fine for now, overall an improvement and we should merge for now, what do you think @christianbundy ? In another PR we could maybe add another button next to the Preview button, that says "Get mentions" or something, that keeps you on the edit screen but lists the possible mentions for you?

image

@christianbundy
Copy link
Member

Thanks for working on this issue. I agree that this is a step backward for mentions, is it possible to implement this in a way that doesn't break that feature? If you'd like, I'm happy to discuss on GitHub so that we can come up with a solution that'll work. I didn't see any conversation in #537.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

message Preview seems to submit the contents of the preview, even if additional edits have been made
3 participants