Remove all dependencies from websocket index.html #1573
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check list:
developmentbranch999-SNAPSHOTversion of Quarkus[...]
====================================
Hello,
I followed the guide on Getting started with WebSocket Next. It all worked fine but I kept wondering about the linked index.html. This one HTML file comes with 3 external JavaScript dependencies + 2 CSS dependencies.
Besides the problem that this makes the example less self contained, IMO the file becomes more complicated than it must be. So I would like to propose an alternative - that looks less nice - which does basically the same but with bare bone HTML/JS.
Pro:
Con:
Let me know if you think this change is worth it. I could also add "some" CSS to make it less ugly, but honestly from my point of view it should focus on the WebSocket stuff and not on a pleasing UI.