andrewfiorillo / sketch-palettes

A Sketch plugin for exporting and importing fill presets. It supports colors, gradients, and pattern fills.
MIT License
2.27k stars 134 forks source link

Include palettes in package #9

Closed JesseBilsten closed 8 years ago

JesseBilsten commented 8 years ago

If a user overwrites their default color palette they can set it back to the defaults in Sketch.

https://www.dropbox.com/s/0br2a8dba6fjzni/Default.sketchpalette?dl=0

andrewfiorillo commented 8 years ago

I'm trying to be really judicious about what I add to the menu. Do you think it is common that people want to get back to that default set? You could also imagine, alternatively, replacing the "clear palette" command with "reset palette" command.

JesseBilsten commented 8 years ago

I think adding "reset" or back to defaults works and appreciate the judicious nature of adding something to the menu. I could imagine someone being upset after they spent a lot of time setting up a palette and then couldn't get back to it.

On Tue, Nov 17, 2015 at 12:57 PM, Andrew Fiorillo notifications@github.com wrote:

I'm trying to be really judicious about what I add to the menu. Do you think it is common that people want to get back to that default set? You could also imagine, alternatively, replacing the "clear palette" command with "reset palette" command.

— Reply to this email directly or view it on GitHub https://github.com/andrewfiorillo/sketch-palettes/issues/9#issuecomment-157504721 .

andrewfiorillo commented 8 years ago

Well, if someone loads a new palette, I'm not sure we could get back a previous palette that was never saved (unless that temporary unsaved palette data was somehow being autosaved somewhere). But we could reset to a hard-coded default set. I was thinking more about problem of inadvertently overwriting your palette though. Perhaps a double confirm or option to append is the solution there.

JesseBilsten commented 8 years ago

An MVP solution would be to provide a link in the README to the Default palette hosted as a gist: https://gist.github.com/JesseBilsten/f182bb124ad58278156d

I moved mine from the Dropbox link there so it's easier to link to/maintain.

On Tue, Nov 17, 2015 at 2:39 PM, Andrew Fiorillo notifications@github.com wrote:

Closed #9 https://github.com/andrewfiorillo/sketch-palettes/issues/9.

— Reply to this email directly or view it on GitHub https://github.com/andrewfiorillo/sketch-palettes/issues/9#event-467299110 .

bzx commented 8 years ago

+1 for adding an option to reset to a default color palette - this is what I wanted to do immediately after clearing the colors and noticing that I can't revert it back in any new document.

andrewfiorillo commented 8 years ago

I didn't add an option to the menu, but I've bundled some palettes with the plugin now, and included the default Sketch palette.