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

[feature] reveal.js syntax checker #26

Open
koebel opened this issue Feb 27, 2024 · 2 comments
Open

[feature] reveal.js syntax checker #26

koebel opened this issue Feb 27, 2024 · 2 comments
Labels
enhancement New feature or request P2 Medium Priority

Comments

@koebel
Copy link
Contributor

koebel commented Feb 27, 2024

only preview .md files with correct reveal.js syntax

from a usability and UX perspective it would be best for files where the syntax checker fails to

  • either don't offer the preview in presentation viewer option (might be tricky when using the current web extension system)
  • or show an error screen with a message that is easily understood by non technical users why their slides can't be previewed in this app

User Story

As a user
I want to be informed if and why some .md files cant be properly displayed in the presentation viewer
So that I can solve the problem and once fixed preview my slides.

@individual-it
Copy link
Member

The first solution would be problematic for performance reasons as it would require to read (at least part of the file) before displaying the list.
The second option sounds good to me, just need to think about how to detect issues correctly

@grgprarup grgprarup added the P2 Medium Priority label Feb 28, 2024
@koebel
Copy link
Contributor Author

koebel commented Mar 1, 2024

I guess some high level feedback would be enough as a start :)

@koebel koebel added the enhancement New feature or request label Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P2 Medium Priority
Projects
None yet
Development

No branches or pull requests

3 participants