shmuelzon / home-assistant-floor-plan

Home Assistant Floor Plan Generator Plugin For Sweet Home 3D
MIT License
264 stars 8 forks source link

Layout: Small issue on the dialog window #34

Closed byDMA closed 3 months ago

byDMA commented 3 months ago

Hi, a small layout issue, this field for high spinner is to big. I#m using FlatLaf dark mode, it looks very well https://www.sweethome3d.com/support/forum/viewthread_thread,12872_lastpage,yes#62795

GridBagConstrains-LayoutIssue

probably Panel.java:

lines 324-326 add(heightSpinner, new GridBagConstraints( 3, 2, 1, 1, 0, 0, GridBagConstraints.LINE_START, GridBagConstraints.HORIZONTAL, new Insets(0, 0, 0, 0), 0,0));`

should be add(heightSpinner, new GridBagConstraints( 3, 2, 1, 1, 0, 0, GridBagConstraints.LINE_START, GridBagConstraints.HORIZONTAL, componentInsets, 0, 0));

shmuelzon commented 3 months ago

Thanks for that! First time developing in Java in ~20 years, let alone GUI. That was probably a result of copy-pasting from various examples...