Spade-Editor / Spade

Cross-platform raster graphics editor inspired by Paint.NET
GNU General Public License v3.0
41 stars 10 forks source link

Brush and Eraser previews + Color Selection Square #59

Closed BurntPizza closed 10 years ago

BurntPizza commented 10 years ago

Draws in XOR mode for view-ability, but doesn't show up over transparent background for some reason. Extensible for arbitrary tools now as well ;)

BurntPizza commented 10 years ago

Hope I didn't break anything...

HeroesGrave commented 10 years ago

@-mention me when it's ready.

BurntPizza commented 10 years ago

@HeroesGrave Ready for merge.

HeroesGrave commented 10 years ago

@BurntPizza

Problem:

Try removing the old colour-display panel, move the buttons to the side, and have the square take up the top-left corner.

Reverted the merge, but I can't re-open the pull request. You'll have to open a new one once you fix things.

Other than that, nice work.

BurntPizza commented 10 years ago

@HeroesGrave Fixed, hopefully. Changed from BorderLayout (rigid) to a more flexible SpringLayout which should keep the square in the right place, regardless of "Color Square" font size. Here's how it looks for me: colorchooser

I would like to keep the color display, it's useful because you can see a nice swatch of both colors you have selected. Tell me if the ui continues to be flaky.

Making new pull request.

HeroesGrave commented 10 years ago

The positioning is a little bit awkward, but I guess it can be fixed later and it's better than nothing.