Changes between Initial Version and Version 1 of Ticket #4880, comment 10


Ignore:
Timestamp:
Jun 3, 2011, 7:36:35 AM (13 years ago)
Author:
jscipione

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #4880, comment 10

    initial v1  
    1 I applied your patch (DeskBarAutoHideFeature.diff) and it does seem to work well. I look forward to this patch being applied, unfortunately I have no authority to do so. You got rid of the overloaded TBarWindow::DispatchMessage() from BarWindow.cpp which should never have been there in the first place, congrats. I have only 2 small nitpicks. Firstly I would change the case kAutoRaise: in BarApp.cpp from
     1I applied your patch (DeskBarAutoHideFeature.diff) and it does seem to work well. I look forward to this patch being included in the project, unfortunately I have no authority to do so. You got rid of the overloaded TBarWindow::DispatchMessage() from BarWindow.cpp which should never have been there in the first place, congrats. I have only 2 small nitpicks. Firstly I would change the case kAutoRaise: in BarApp.cpp from
    22
    33{{{