[Feature] Add Windows Explorer context menu to file tabs
Reported by Jocelyn Legault | November 24th, 2009 @ 11:58 PM | in v2.0.0
Add Windows Explorer context menu to file tabs, in the same way the Light Explorer plugin has it when you right click on a file.
Comments and changes to this ticket
-
Jocelyn Legault November 25th, 2009 @ 08:22 PM
- State changed from open to needs_ack
(from [caadb41966b5bf529663f5ff3521c543af50629c]) Explorer Context menu: add contextual menus on tabs.
- Explorer context menu added to tab context menus.
- Code mainly leveraged from NppLightExplorer.
- Can be enabled / disabled through the preference panel.
Additions to the code from NppLightExplorer:
- Make sure that no separators show up at the top or the bottom of the Explorer Menu.
- List of unwanted menu items is larger. Some can be done by ID, others can't and are done textually.
- Comparisons with unwanted menu items is done without the '&'
Potential feature addition:
- User could specify more items he does not want to see there in the Preference panel.
[#18 state:needs_ack]
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/joce/npp-community/commit/caadb41966b5bf529663f5f... -
Jocelyn Legault November 25th, 2009 @ 09:05 PM
- Assigned user changed from Jocelyn Legault to npp-community
-
Thell Fowler November 28th, 2009 @ 08:09 PM
- Tag cleared.
- Assigned user changed from npp-community to Jocelyn Legault
Why are ExplorerContentMenu and explorerMenu being used instead of ExplorerContextMenu and explorerContextMenu?
Thell
-
Thell Fowler November 29th, 2009 @ 03:31 PM
Joce,
After doing a merge of this and then building the debug build using vc8 I was able to get the explorer menu to show up just fine, but then I clicked on the Setting|Preferences menu and started receiving 'Unhandled Default Case' messages (which have garbage characters in the message box title), so I attached the debugger thinking that would cause a break. It didn't. After setting a break point on the only line in the source container 'Unhandled' I still found myself unable to have the BP hit. Several messages later the Preferences dialog finally came up.
I'll do some more testing, but thought you'd want to know.
Thell
-
Thell Fowler November 29th, 2009 @ 04:51 PM
See comment for ticket #8 regarding DEBUG vs _DEBUG
That is the culprit of the msgs I witnessed.
-
Jocelyn Legault December 2nd, 2009 @ 10:29 PM
Why are ExplorerContentMenu and explorerMenu being used instead of ExplorerContextMenu and explorerContextMenu?
Ooops. Oversight for my part. Will be fixing that.
-
Jocelyn Legault December 3rd, 2009 @ 05:36 AM
- Assigned user changed from Jocelyn Legault to npp-community
(from [0b71c5e77e12558136fcac7ef0f5b400dd154c1b]) Explorer menu variables: rename to clearer names.
- Trivial variable name changes to better reflect the nature of some variables used in the context of the ExplorerContextMenu.
[#18 state:needs_ack responsible:npp-community]
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/joce/npp-community/commit/0b71c5e77e12558136fcac7... -
Jocelyn Legault December 3rd, 2009 @ 07:49 PM
- Milestone set to v2.0.0
-
Thell Fowler December 4th, 2009 @ 03:18 PM
ExporerConextMenu class is still being referred to as ExplorerContentMenu...
Also, in ExplorerContentMenu::CleanUpMenu() you clean-up the Send To menu item. What's the reasoning for this?
Thell
-
Jocelyn Legault December 4th, 2009 @ 06:12 PM
Because it does not come up looking right. It only shows an empty submenu.
I guess we could try to find out why and fix it. -
Thell Fowler December 6th, 2009 @ 06:19 PM
- Assigned user changed from npp-community to Jocelyn Legault
Because it does not come up looking right. It only shows an empty submenu. I guess we could try to find out why and fix it.
Nah. I was mostly just curious.
The class name still needs to be fixed from ExplorerContentMenu to ExplorerContextMenu though.
-
Thell Fowler December 6th, 2009 @ 09:20 PM
- State changed from needs_ack to acked
Joce,
Perhaps there is something different between win7 and XP in regards to sub-menu items. I decided to do a change of ExplorerContentMenu to ExplorerContextMenu so this could get on pu and when I loaded and opened N++ this is what I saw:
1) A working Send-To submenu::
2) A non-working Share-With submenu::
Even though I'm sure you will want to do more fine tuning on this, I'm going ahead with adding it onto pu as LH-18v3 since I think it is a feature others will want to have easy access to.
I've also pushed this as a stand-alone branch for you to just grab @:
http://github.com/almostautomated/npp-community/tree/jl/LH-18v3/con...
Thell
-
npp-community December 7th, 2009 @ 06:14 AM
- State changed from acked to needs_ack
(from [f5fa068f552d0f4a364431e1cf3f1eb4d817b0bc]) Explorer Context menu: add contextual menus on tabs.
- Explorer context menu added to tab context menus.
- Code mainly leveraged from NppLightExplorer.
- Can be enabled / disabled through the preference panel.
Additions to the code from NppLightExplorer:
- Make sure that no separators show up at the top or the bottom of the Explorer Menu. - List of unwanted menu items is larger. Some can be done by ID, others can't and are done textually. - Comparisons with unwanted menu items is done without the '&'Potential feature addition:
- User could specify more items he does not want to see there in the Preference panel.[#18 state:needs_ack]
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/npp-community/npp-community/commit/f5fa068f552d0f... -
Thell Fowler December 7th, 2009 @ 06:14 AM
- State changed from needs_ack to proposed
(from [e857535280887c2555689b334a8efb183f61d69e]) Merge branch 'jl/LH-18v3/context-menu-on-tabs' into pu
- jl/LH-18v3/context-menu-on-tabs: Explorer menu variables: rename to clearer names. Explorer Context menu: add contextual menus on tabs.
Conflicts:
PowerEditor/src/Notepad_plus.cpp PowerEditor/visual.net/notepadPlus.8.vcproj PowerEditor/visual.net/notepadPlus.9.vcproj
[#18 state:proposed]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/npp-community/npp-community/commit/e857535280887c... -
npp-community December 7th, 2009 @ 06:14 AM
- State changed from proposed to needs_ack
- Assigned user changed from Jocelyn Legault to npp-community
(from [5c3d735bd3ad5587cbede394c2f31d7988f42d17]) Explorer menu variables: rename to clearer names.
- Trivial variable name changes to better reflect the nature of some variables used in the context of the ExplorerContextMenu.
[#18 state:needs_ack responsible:npp-community]
Acked-by: Thell Fowler git@tbfowler.name
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/npp-community/npp-community/commit/5c3d735bd3ad55... -
Thell Fowler December 7th, 2009 @ 06:23 AM
- State changed from needs_ack to proposed
The last two automatic state changes from Github where applied in the wrong order... state is proposed and it is on pu.
-
Thell Fowler December 7th, 2009 @ 11:02 PM
(from [7ed450c6cb650b97ce2d323c11f89b0da152bd69]) Merge branch 'jl/LH-18v3/context-menu-on-tabs' into pu
- jl/LH-18v3/context-menu-on-tabs: Explorer menu variables: rename to clearer names. Explorer Context menu: add contextual menus on tabs.
Conflicts:
PowerEditor/src/Notepad_plus.cpp PowerEditor/visual.net/notepadPlus.9.vcproj
[#18 state:proposed]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/npp-community/npp-community/commit/7ed450c6cb650b... -
Thell Fowler December 10th, 2009 @ 10:22 PM
On Windows 7 when using the context menu on a .bat file with PowerCmd loaded an access violation would happen. It looks like the cause is that the info stuct wasn't getting cleared.
By moving the definition for info and info.cbsize into the for loop in ExplorerContentMenu::CleanUpMenu() so everything is fresh for each iteration resolves this issue.
-
Thell Fowler December 22nd, 2009 @ 03:53 PM
(from [9ed37f1a2f95bfd3cffb17c463acdae6a7f91a0e]) Merge branch 'jl/LH-18v3/context-menu-on-tabs' into pu
- jl/LH-18v3/context-menu-on-tabs: Explorer menu variables: rename to clearer names. Explorer Context menu: add contextual menus on tabs.
Conflicts:
PowerEditor/src/Notepad_plus.cpp PowerEditor/src/Parameters.h PowerEditor/src/menuCmdID.h PowerEditor/visual.net/notepadPlus.8.vcproj PowerEditor/visual.net/notepadPlus.9.vcproj
[#18]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/almostautomated/npp-community/commit/9ed37f1a2f95... -
Thell Fowler January 19th, 2010 @ 06:07 AM
(from [07fc1df4a7ce597a598ec07b1c328318f1874661]) Merge branch 'jl/LH-18v3/context-menu-on-tabs' into pu
- jl/LH-18v3/context-menu-on-tabs: Explorer menu variables: rename to clearer names. Explorer Context menu: add contextual menus on tabs.
Conflicts:
PowerEditor/src/Notepad_plus.cpp PowerEditor/src/Parameters.h PowerEditor/src/menuCmdID.h PowerEditor/visual.net/notepadPlus.8.vcproj PowerEditor/visual.net/notepadPlus.9.vcproj
[#18]
Acked-by: Thell Fowler git@tbfowler.name
http://github.com/npp-community/npp-community/commit/07fc1df4a7ce59... -
Thell Fowler February 14th, 2010 @ 08:16 PM
(from [93abcc638815d6a6aa003b9684bb57456472411c]) ExplorerContextMenu.cpp: make cbsize refresh
- On Windows 7 some extensions (ie: .bat) could cause an access violation during menu cleanup. It looks like the cause is that the info stuct wasn't getting cleared. Moving the definition for info and info.cbsize into the for loop in ExplorerContentMenu::CleanUpMenu() so everything is fresh for each iteration resolves this issue.
[#18]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/almostautomated/npp-community/commit/93abcc638815... -
Thell Fowler February 14th, 2010 @ 08:18 PM
- State changed from proposed to needs_ack
-
Jocelyn Legault February 16th, 2010 @ 05:58 PM
- State changed from needs_ack to acked
-
Thell Fowler February 20th, 2010 @ 02:57 AM
- State changed from acked to proposed
-
Thell Fowler February 27th, 2010 @ 05:21 PM
(from [9ea329862f8024ce153660e00bd6b98239f81923]) Merge branch 'jl/LH-18v4/context-menu-on-tabs' into pu
- jl/LH-18v4/context-menu-on-tabs: ExplorerContextMenu.cpp: make cbsize refresh Explorer menu variables: rename to clearer names. Explorer Context menu: add contextual menus on tabs.
Conflicts:
PowerEditor/src/Notepad_plus.cpp PowerEditor/src/Parameters.h PowerEditor/src/menuCmdID.h PowerEditor/visual.net/notepadPlus.8.vcproj PowerEditor/visual.net/notepadPlus.9.vcproj
[#18]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/npp-community/npp-community/commit/9ea329862f8024... -
Jocelyn Legault October 31st, 2010 @ 05:21 PM
- State changed from proposed to needs_ack
- Milestone order changed from 0 to 0
(from [d250249234be51b7366e30deb72e76d89d599e91]) Explorer menu variables: rename to clearer names.
- Trivial variable name changes to better reflect the nature of some variables used in the context of the ExplorerContextMenu.
[#18 state:needs_ack responsible:npp-community]
Acked-by: Thell Fowler git@tbfowler.name
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/joce/npp-community/commit/d250249234be51b7366e30d... -
Jocelyn Legault October 31st, 2010 @ 05:21 PM
(from [5fa8eff3704c282d8bbb67b20b71aa1db4bd513e]) Fix the tab options GUI items alignement
Vertical alignment was off and fixed.
[#18]
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/joce/npp-community/commit/5fa8eff3704c282d8bbb67b... -
Thell Fowler October 31st, 2010 @ 05:21 PM
(from [f9ac3edfd0aa590145597b1a42516f32f5339eb0]) ExplorerContextMenu.cpp: make cbsize refresh
- On Windows 7 some extensions (ie: .bat) could cause an access violation during menu cleanup. It looks like the cause is that the info stuct wasn't getting cleared. Moving the definition for info and info.cbsize into the for loop in ExplorerContentMenu::CleanUpMenu() so everything is fresh for each iteration resolves this issue.
[#18]
Signed-off-by: Thell Fowler git@tbfowler.name
http://github.com/joce/npp-community/commit/f9ac3edfd0aa590145597b1... -
Jocelyn Legault October 31st, 2010 @ 05:21 PM
(from [1dc68dde70c79d64da7391428307312801972895]) Explorer Context menu: add contextual menus on tabs.
- Explorer context menu added to tab context menus.
- Code mainly leveraged from NppLightExplorer.
- Can be enabled / disabled through the preference panel.
Additions to the code from NppLightExplorer:
- Make sure that no separators show up at the top or the bottom of the Explorer Menu. - List of unwanted menu items is larger. Some can be done by ID, others can't and are done textually. - Comparisons with unwanted menu items is done without the '&'Potential feature addition:
- User could specify more items he does not want to see there in the Preference panel.[#18 state:needs_ack]
Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/joce/npp-community/commit/1dc68dde70c79d64da73914... -
Thell Fowler October 31st, 2010 @ 08:52 PM
- Assigned user changed from npp-community to Jocelyn Legault
I checked out and built your topic branch and this is what I get for a context menu on a tab. Will look into this soon.
-
Thell Fowler October 31st, 2010 @ 11:13 PM
- State changed from needs_ack to proposed
Figured out what was going on there. When a tab has a file that has no path the Explorer Context menu has no path to latch onto so no context menu is build for explorer! Opened an existing file and it worked great!
I would put this on next as a candidate, but as you requested 'proposed' I'll put it there instead.
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Notepad++ Community Release
People watching this ticket
Referenced by
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack responsible:npp-community]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:proposed]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack responsible:npp-community]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:proposed]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack responsible:npp-community]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18]
- 18 [Feature] Add Windows Explorer context menu to file tabs [#18 state:needs_ack]