This version looks alot better than the first one.
* SH_FileTimerProc [Internal] * * is invoked every 100ms to check if the property sheet should be closed * A timer is required because the property sheet pages are per default * modeless and the property sheetpage does not have a window procedure
This doesn't seem right to me. The property sheet should destroy all its pages when it is closed.
Also, instead of attaching the new file, you can do: diff -u /dev/null dlls/shell32/fprop.c >> fp-051029.diff so that everything is in one diff. Mike