search

haacked / seegit

SeeGit - The Git Repository Visualizer
MIT License
508 stars 107 forks source link

Support browse for repo under WinXP #50

Closed DavidDaly closed 11 years ago

DavidDaly commented 11 years ago

Resolves Issue #48

Therzok commented 11 years ago

Looks good to merge Phil. Thanks David.

haacked commented 11 years ago

This looks like it always uses the Windows Forms dialog. Is that exactly the same? If not, I'd prefer to optimize for non-XP by catching the PlatformNonSupportedException and fallback to this code in that case.

Therzok commented 11 years ago

So, any progress on rewriting for WinXP compatibility in case that exception is thrown?

DavidDaly commented 11 years ago

Sorry guys - haven't looked at it yet but should have something with you later today

DavidDaly commented 11 years ago

I have made the required changes but I could do with someone on Win7 at least running the code to make sure it behaves as expected

haacked commented 11 years ago

Thanks @DavidDaly! I just tried it out on Win8 and works fine. Appreciate the help!

DavidDaly commented 11 years ago

Thanks - pleasure to be involved in a small way!