Ticket #2099 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

Sugar shouldn't let the two "My Colors" be identical

Reported by: Zack Owned by: marco
Priority: normal Milestone: Trial-3
Component: sugar Version:
Keywords: Cc: marco, Zack, Eben, walter
Action Needed: Verified: no
Deployments affected: Blocked By:
Blocking:

Description

When I first booted 499, sugar's default colors were both the same. I accepted that, and as I expected, most of my icons are not recognizable, due to being one color.

Change History

Changed 7 years ago by marco

Please attach your .sugar/default/config

Changed 7 years ago by jg

  • milestone changed from Untriaged to Trial-2

Changed 7 years ago by marco

To clarify, Sugar choose the colors from a fixed list (which Walter provided), so we need to know which colors you was using exactly to be able to do something about it.

Changed 7 years ago by marco

Zack, can you provide your config please? Can't do much about this without it.

Changed 7 years ago by jg

  • owner changed from dcbw to Zack

Changed 7 years ago by bert

  • cc marco added

In build 526 I got colors that were not identical, but so close the icons still were pretty much unrecognizable: FFFA00,F8E800

Changed 7 years ago by marco

  • owner changed from Zack to walter

Reassigning to Walter. Please let me know how I should update the list of colors.

Changed 7 years ago by coderanger

Adding reporter to CC list

Changed 7 years ago by coderanger

  • cc Zack added

Adding reporter to CC list

Changed 7 years ago by marco

  • cc Eben added

Adding Eben. Eben, should we just drop this color from the list? I'm mostly wondering if this is the only case or there are problem in the way the color list was generated.

Changed 7 years ago by Eben

  • cc walter added

Well, the list of colors is neatly defined in such a way that tossing one from the list isn't really a good idea. There are 5 "groups" of colors, where each group has all possible combinations of stroke and fill assuming 6 distinct hues. That is, there are 6*6*5=180 color pairs in total, and removing any of them will leave a hole in the newly designed color picker.

Walter, do you want to revisit these particular edge cases? Glancing at them again, I see a few that might need tweaking:

(page, fill, stroke) (5, yellow, yellow) (4, orange, yellow) (3, yellow, orange)

Of course, the latter 2 are really the same pair, inverted. The rest seem distinct enough.

Changed 7 years ago by walter

Let's change the yellow yellow pair from FFFA00,F8E800 to FFFA00,EDDE00

Changed 7 years ago by marco

  • owner changed from walter to marco

Changed 7 years ago by marco

  • status changed from new to closed
  • resolution set to fixed

Applied the fix suggested by Walter.

Changed 22 months ago by Micah

  • blockedby 12152 added

Changed 22 months ago by Micah

  • blockedby 12152 removed
Note: See TracTickets for help on using tickets.