Make WordPress Core

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


Ignore:
Timestamp:
08/26/2012 09:11:01 PM (13 years ago)
Author:
downstairsdev
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #21206, comment 40

    v1 v2  
    1313Generally 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.
    1414
    15 But 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.
     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 or blank)- so if a user suddenly now has the ability to clear that, it will break certain theme implementations.