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

Hide preview option and indicate that does not support Chrome >= 81 #213

Merged
merged 6 commits into from
Mar 19, 2020

Conversation

yhatt
Copy link
Member

@yhatt yhatt commented Mar 15, 2020

In Chrome 81 that is going to release in Mar 17th, --preview option won't work due to not updated permissions in unmaintained carlo.

The logic of experimental --preview option does not change from the current, but Marp CLI will indicate with an error message that --preview option does not support Chrome >= 81. In addition, -p / --preview are no longer shown in --help too (but it can show via --help --show-hidden).

Related to #211.

@yhatt
Copy link
Member Author

yhatt commented Mar 18, 2020

We had expected to merge this and bump version according to the time of releasing Chrome 81 (yesterday), but it seems to be delayed. I've still got Chrome 80 after the update in yesterday.

@yhatt
Copy link
Member Author

yhatt commented Mar 19, 2020

Upcoming releases for Chrome was paused due to adjusted schedules (It was probably affected by COVID-19). Stopping evolution of Web is feeling like a bit of sadness but this fact luckily gets time to deprecate preview option during the stable Chrome is 80.

We can reduce warning level from ERROR to WARN, and note about an error in future release of Chrome.

@yhatt yhatt merged commit e71a306 into master Mar 19, 2020
@yhatt yhatt deleted the hide-preview-option branch March 19, 2020 19:47
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.

1 participant