Combine the Control Panel with the Canvas window

[artreadcode]

Describe the bug

Quantum Brush has three different windows. While Stroke Manager supports specific tasks such as brush execution and stroke management, there is no clear reason for Canvas and Control Panel stay separated. This was an engineering decision based on Processing(Java)'s main window drawing mechanism, but this is not ideal on the user's side.

To reproduce

Open the Quantum Brush app, and observe the workflow between different windows.

Expected behavior

1. Like other graphics tools, inside of the single window, current Control Panel and the Canvas should be grouped altogether, rather than being separated.
2. This must not break Control Panel's user parameter input method and Canvas' brush stroke display with undo/redo + zoom in/out, etc. Thus, all established features of Control Panel and the Canvas must not be broken after the update.

NOTE: The source code for the app can be found under this branch (Branch name: source)

[artreadcode]

This issue is only for UnitaryHACK event.

[codexbountylab]

Hi @artreadcode, I opened PR #64 with an implementation for this issue. It combines the canvas and control panel into a single workspace while keeping the existing painting controls available.

Happy to adjust the approach if you prefer a different UI structure.

[artreadcode]

@codexbountylab

Hello, thank you for your interest. Please provide the screenshot, working GIF, and the built Java package file (.jar) for the test.

[codexbountylab]

@codexbountylab

Hello, thank you for your interest. Please provide the screenshot, working GIF, and the built Java package file (.jar) for the test.

Added the requested screenshot, GIF, and zipped Java test build in PR #64.

[artreadcode]

@codexbountylab The issue is almost solved.

There are some bugs before I assign you on this issue and close it. The canvas size logic is recently improved by one of the UnitaryHACK participant. (Refer to this issue: #46 ) While refering this, your deployment can be significantly improved without the UI issues, and this issue can finally be closed.

Also, the placeholder text on the canvas while there's no image is loaded, is not located in the middle when you open up the Quantum Brush application. Thus, these two bugs will be the ones that must be fixed before the bounty!

[codexbountylab]

Hi @artreadcode, I pushed an update to #64.

I adapted the canvas sizing logic based on the approach from #46, while accounting for the unified workspace layout in this PR. I also fixed the empty-canvas placeholder so it is centered in the visible canvas area when the app opens.

[artreadcode]

@codexbountylab Thank you for this. Can you provide the compiled .jar file in your PR thread like before, so that I could easily test it out?

I saw you also worked on the de-serialisation. If you could provide the compiled .jar package with solves both of the issues, (on both issue thread) you could get two bounties.

Only solving this window issue is also fine. Don't need to stretch out yourself too much.

[artreadcode]

The issue of combining windows are solved now and the bounty is yours. @codexbountylab
This issue is now closed.