JankariTech / web-app-presentation-viewer

ownCloud Web extension to view markdown presentation slides
Apache License 2.0
4 stars 1 forks source link

[feature] reveal.js syntax checker #26

Closed koebel closed 2 weeks ago

koebel commented 7 months ago

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

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 commented 7 months ago

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

koebel commented 7 months ago

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

koebel commented 2 weeks ago

There are hardly no situations where the .md file can't be rendered Only situations we can think about are empty .md file --> should display a single white slide But even that case wouldn't break the app.

koebel commented 2 weeks ago

Closing this issue because it doesn't seem to be affecting the user experience much