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

Preview button appears only if wiki syntax is enabled #230

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

Conversation

moisesbr-dw
Copy link
Contributor

When wiki syntax is disabled, the preview simply repeats the comment.
Therefore the preview button is useless and can be removed from the form.

@michitux
Copy link
Member

I'm not sure about this change. While you are right that the comment is really just repeated, it might still be nice for a user to see in the preview that there is actually no formatting. I.e., if you expected that there was some support for formatting and you used that and then you press preview and see that formatting is ignored, you can fix your comment to remove the formatting.

@moisesbr-dw
Copy link
Contributor Author

moisesbr-dw commented Jun 18, 2017

I think there can be no confusion.
The messages are different and the toolbar appears when wikitext is enabled.
Please compare these two screenshots:

Wikitext enabled
message-with-wikitext

Wikitext disabled
message-without-wikitext

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

Successfully merging this pull request may close these issues.

2 participants