wsy4665 / ulipad

Automatically exported from code.google.com/p/ulipad
0 stars 0 forks source link

A patch with explanation #167

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Limodou, there is a reason I must insist that the strings must be
'Open Command Prompt' and 'Open Windows Explorer'
and *not* 'Open Command Prompt Here' and 'Open Windows Explorer Here'
or any other variation with the word 'Here' added.

There is a reason for this. The Command Prompt is *not* opened *inside* the
Directory Browser, it is *not* *embedded* into the Directory Browser when
you click that context-menu item. The Command Prompt window opens
separately from the Directory Browser, so it is *not* opened 'here' meaning
'inside the Directory Browser pane'.

Limodou, please accept my patch and apply it into the latest SVN version of
UliPad.

This is good patch. I will be very sad if this gets rejected.

Original issue reported on code.google.com by vinet...@gmail.com on 3 Dec 2008 at 8:47

Attachments:

GoogleCodeExporter commented 9 years ago
Why I use here? Because it doesnot mean the "Directory Browser Window" but "this
position", so it'll open DOS prompt or explorer according current file path or
directory path. And if you know there is a tool of microsoft, you can find it at
http://www.microsoft.com/windowsxp/downloads/powertoys/xppowertoys.mspx, and 
there is
a tool named Open Command Window Here, I learnt from it.

Original comment by limo...@gmail.com on 4 Dec 2008 at 12:48

GoogleCodeExporter commented 9 years ago

Original comment by limo...@gmail.com on 7 Dec 2008 at 7:43