openMSX / debugger

31 stars 15 forks source link

PaletteDialog: replace apply with restore defaults #184

Closed pvmm closed 1 year ago

pvmm commented 1 year ago

image

Now the palette editor allows the user to restore palette to the default MSX palette. Apply is not as useful as the Defaults button, so it was removed to reduce clutter.

pvmm commented 1 year ago

I also wonder why BitMapViewer and TileViewer each have their own 'currentPalette' array, Maybe this should be a shared array? (Because even if you use a custom palette rather than the current VDP palette, I assume it's useful to have the same custom palette between BitMapViewer and TileViewer)

Then maybe store it in VDPDataStore and call it alternativePalette.