Changes between Initial Version and Version 1 of Ticket #21206, comment 40

Timestamp:
08/26/12 21:10:03 (9 months ago)
Author:
downstairsdev
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #21206, comment 40

    initial v1  
    44 
    55Perhaps the only option we have is "default" (when a default it supplied), and otherwise don't have a button there? 
     6 
     7--- 
     8 
     9Update: 
     10 
     11@mbijon's Can you give a use case for clear? 
     12 
     13Generally for a theme option, I'd check whether the default color was selected.  If it is, I wouldn't output an inline style, otherwise I would. 
     14 
     15But I could think that other folks might have always set up their options to output whatever value was returned from the colorpicker options (without checking if it was a default)- so if a user suddenly now has the ability to clear that, it will break certain theme implementations.