-
Notifications
You must be signed in to change notification settings - Fork 291
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
integrate devtools into preview #1813
Comments
This is something that I think would work well as a plugin. |
What would this add that just installing dev tools would not? |
In my opinion: none, it just brings workflow improvements. This being said, it'd be very nice to avoid the annoyance of having to enable/disable for each course, and to potentially add some other AT-specific features. If we can do this as an installable plugin, then it doesn't need to be something that goes into core. |
My concern would be that preview would no longer show exactly what the course will look like. Its also worth pointing out dev tools is not an |
When we trigger a global event with a handle to the previewWindow, Plugin authors can hook into and inject scripts into the preview Window. |
We all agree that, preview and some kind of live editing functionality is missing in the current versions. In my opinion the devtools offer some very interesting and powerfull concepts that can improve a course authors workflow and experience. therefore I think we should think about adding some concepts from this plugin into the core. @canstudios-louisem we can add a button that loads the plugin only when requested. So sideeffects should be rather low. |
Affected Area
Course Preview
Requested Feature
Integrate some features from adapt-devtools into the preview window.
Use Case
Current Workaround
One must enable the plugin manually and make sure it is removed once the course is deployed.
The text was updated successfully, but these errors were encountered: