Conversation
|
Anyone's free to provide feedback and thoughts on this design, I wouldn't want to create something that people wouldn't like to use or navigate. |
|
I would like to propose a dark mode toggle |
|
It'd be very nice if the new UI showed progress of what it was doing. Downloading converters/scripts/binaries/etc and what step of the conversion route its on. Would also be nice if you could pick the path it takes. Also, options. If converting from SVG to PNG, would be nice to specify the resolution/dpi. Or if a converter which takes a ppt to mp4 and you have the option to specify the frame rate. |
Oh, that's a good idea, I'll check if this could be implemented nicely.
I do agree that these features would be great to have, but I think that these might be outside of the scope of the UI revamp as these would require bigger backend changes. |
|
I like this UI proposal. Concerning "options", I do feel like we should have some kind of "settings" menu, even if it would be underutilized at the moment. We'll likely want that stuff later, so having a space for it is a good idea. |
|
i wanna see a treeview of the path it took to convert the file |
That's a bit ambitious for now. Let's keep this to just a redesign with room for more features in the future. |
|
I think we should make the github link on the bottom corner into a proper link with a logo beside it to make people more likely to notice it and contribute to the project? |
|
Before I begin, as we're using Bun+Vite, would it be fine if I begin to divide and compartmentalize each component? Currently everything is just lumped in |
|
Yeah, sure. Restructuring is fine. |
|
Minor suggestion, it would be great if you could cancel the current conversion without having to refresh the page |
|
There is full info for the file you are converting to ( |
|
I'm not a fan of all the LLM-generated frontend stuff. I'm glad that y'all are eager to help, but vibecoding a frontend is not a sustainable idea. As I understand it, the people who authored this PR are working on their own implementation. Let them finish what they started, we don't need to rush this by sloppinating a prompt. |
|
it would be nice if there was also a swap button to swap which you were converting to vs from |
|
should really ping @headquarter8302 not me :p looks great!! much better imo |
|
Hi. Yes. I'll be participating actively once I've fully migrated my setup to my new laptop. In the mean time, I'll be ready to integrate the PR! |
|
The PR to the repo is here - The UI upgrades are relatively in place (all conversion aspects working fine) but I'll add some more things today or so |
|
Awesome. When you're done with the additions, just let us know here |
….g. ImageMagick), add logging for conversion steps, conversion done screen
|
@headquarter8302 I'm all done! |
|
Awesome. I'll pull the changes locally and review it in a few hours. I'm currently at work and I won't be free until |
…n-Scratch/convert into Explosion-Scratch-frontend-revamp-contrib
|
I've merged your branch, and as such I can proudly proclaim: Conversion has been implemented! This PR is ready to be marked as ready to review and merge @p2r3 |
|
@headquarter8302 I implemented a few other misc. things that would be good to bundle with this revamp, allowing handlers to suggest routes. More details are in the PR here. If this is outside of the scope of this PR lmk, but essentially this drastically improves EPUB and PPTX and allows handlers to do something like this: public suggestedRoutes = [["html", "typ", "pdf"], ["html", "typ"]];Which means that for said handler, it essentially looks to the graph like it directly outputs PDF but in reality it goes html -> typ -> pdf (the format is file extensions or |
|
It looks like it's outside of the scope of this PR ( |











This PR introduces a major UI revamp of the converter application application.
Progress
Anyone is free to provide any feedback on the design or implementation (even small nitpicks are appreciated) as we want to develop the best possible experience for users!
Current version:
Initial draft wireframe:
This PR should: