Skip to content

Coordinate display#15

Merged
duplexsystem merged 2 commits into
PolyhedralDev:masterfrom
booleanbyte:CoordinateDisplay
Jun 1, 2025
Merged

Coordinate display#15
duplexsystem merged 2 commits into
PolyhedralDev:masterfrom
booleanbyte:CoordinateDisplay

Conversation

@booleanbyte
Copy link
Copy Markdown
Contributor

Adds display of the coordinates under the mouse cursor when hovering over the preview in the Render tab.

image

Copy link
Copy Markdown
Member

@solonovamax solonovamax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this smells like an LLM wrote it, however I'm not super sure on that.

just to confirm, have you used an LLM (or any other form of AI-based code generation tool) for this?

OP confirmed it was not written by an LLM

Comment on lines +119 to +129
JTabbedPane tabbedPane = new JTabbedPane();
tabbedPane.addTab("Render", noise);

pane.addTab("Render 3D", noise3d);
pane.addTab("Render Voxel", noise3dVox);
tabbedPane.addTab("Render 3D", noise3d);
tabbedPane.addTab("Render Voxel", noise3dVox);

pane.addTab("Settings", settingsPanel);
tabbedPane.addTab("Settings", settingsPanel);

pane.addTab("Statistics", statisticsPanel);
tabbedPane.addTab("Statistics", statisticsPanel);

pane.addTab("Distribution", distributionPanel);
tabbedPane.addTab("Distribution", distributionPanel);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why was this renamed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clarity. I found "pane" is a bit too generic for a UI beyond the simplest example with only one pane. "tabbedPane" at least gives a good hint at which pane it is.

While fixing the layout, I also tried to integrate a split pane to make the text/preview resizable. Along the way I also ended up trying to consolidate some related parts in the code a bit more, in an attempt to make it easier to follow. The name change came along with the effort.

I didn't keep the change with the split pane, it didn't work well in practice with how the render doesn't shrink again, so I reverted to grid layout again. But I kept the rest because I thought it was an improvement.

It's not LLM.
It's closer to well-intended arrogance if anything.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not LLM.
It's closer to well-intended arrogance if anything.

fair enough

I just wanted to make sure it wasn't

@duplexsystem duplexsystem merged commit 5371a39 into PolyhedralDev:master Jun 1, 2025
1 check passed
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.

3 participants