Changes between Initial Version and Version 1 of Ticket #3746, comment 5


Ignore:
Timestamp:
Jan 5, 2017, 6:54:27 PM (7 years ago)
Author:
humdinger

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #3746, comment 5

    initial v1  
    1 You're missing a .AddStrut(B_USE_DEFAULT_SPACING) between the Defaults and Revert buttons.
     1You're missing a .AddStrut(B_USE_DEFAULT_ITEME_SPACING) between the Defaults and Revert buttons.
    22
    33And the Defaults button should pretty much always be enabled. Now it's disabling when the user presses "Apply" and only becomes enabled again when the colour picker is manipulated. One could use the DesktopColor() call on the current BScreen to see if the defaults are already set and disable the Defaults button if that's the case. But personally, I wouldn't be bothered if the button were just always enabled...