ucsb-cs56-projects / cs56-discretemath-towers-sierpinski

-
1 stars 8 forks source link

Update Color Picker Program #34

Open ShadowAries opened 10 years ago

ShadowAries commented 10 years ago

Right now, the color picker program run with

ant EX_DEF

has 5 color selections to choose from. There appears to be some slight typos with the name and associated colors. For example, choosing the "Winter" color theme appears to actually result in the Rainbow theme. The "Rainbow" color theme results in a red/orange/yellow theme which doesn't seem to be "Rainbow."

In addition to the issue above, add in 5 new, interesting color schemes.

~estimated 100

jstaahl commented 10 years ago

This is a good issue.

kjorg50 commented 10 years ago

I think the labels for "Autumn" and "Rainbow" are still mixed up. For instance when I select Autumn and click the preview I see these colors:

autumn

And then when I click Rainbow and select the preview I see these colors: rainbow

ShadowAries commented 10 years ago

@kjorg50

Thanks for pointing out the color schemes in the preview, I did not check that feature. However, as I noted in the code, I'm not too certain what the last 2 colors are used for, as they don't seem to change the color scheme at all when you actually run the program and view the XML file in a browser. It appears that the first 3 colors are the only ones of use. Therefore, the first 3 colors in "Autumn" are Red, Orange, and Yellow while the first 3 colors in "Rainbow" are Red, Green, and Blue. The last 2 colors seem to have no effect.

ashedden commented 8 years ago

W16 ok!

asyung commented 8 years ago

All colors are used as number of disks increase (ex. 5) to build the triangle. The colours for Autumn and Rainbow should be updated to reflect their titles.

F16 OK (50 points)