Page MenuHome
Differential D763

Reorganize RMB menu and add option "Copy Full Data Path"
ClosedPublic
Actions

Authored by Julian Eisel (Severin) on Aug 26 2014, 2:35 AM.

Details

Reviewers
Jonathan Williamson (carter2422)
Summary

Reorganize RMB menu and add option 'Copy Full Data Path'

Reorganize RMB menu

As per @Campbell Barton (campbellbarton)'s suggestion, this patch adds a submenu "Scripting" to the right click menu. This contains:

  • Edit Source
  • Copy Data Path
  • Copy Full Data Path
  • Online Python Reference

The "Edit Translations" entry will only be visible for non-release builds, also following the suggestion from @Campbell Barton (campbellbarton).

Copy Full Data Path

An operator is added to the RMB-menu „Copy Full Data Path“, to copy the full RNA path to the clipboard. It aims to complement the „Copy Data Path“, which only copies the part of the path that is needed for drivers, but for writing addons, etc. it is useful to have an option that gives the full data path.

The changes made here should be committed separately.

Diff Detail

Event Timeline

Julian Eisel (Severin) updated this revision to Diff 2491.Aug 26 2014, 2:35 AM
Julian Eisel (Severin) retitled this revision from to RMB-menu: 'Copy Full Data Path'.
Julian Eisel (Severin) updated this object.
Julian Eisel (Severin) added a reviewer: Campbell Barton (campbellbarton).
Julian Eisel (Severin) set the repository for this revision to rB Blender.
Julian Eisel (Severin) added a project: User Interface.
Julian Eisel (Severin) added a subscriber: Pablo Vazquez (pablovazquez).
Maverick (maverick) added a subscriber: Maverick (maverick).Aug 26 2014, 12:17 PM
Julian Eisel (Severin) updated this revision to Diff 2500.Aug 27 2014, 10:16 PM
Julian Eisel (Severin) updated this object.
Julian Eisel (Severin) set the repository for this revision to rB Blender.

I just learned that a RNA boolean can be used here to make the code much cleaner. Thanks @venomgfx for the hint :).

Diego Gangl (januz) added a subscriber: Diego Gangl (januz).Oct 9 2014, 11:09 PM
Julian Eisel (Severin) mentioned this in T42280: Python tooltips ellipsis problem.Oct 18 2014, 2:20 AM
Grabz added a subscriber: Grabz.Oct 18 2014, 8:25 AM
Jonathan Williamson (carter2422) accepted this revision.Oct 21 2014, 9:42 PM
Jonathan Williamson (carter2422) added a reviewer: Jonathan Williamson (carter2422).
Jonathan Williamson (carter2422) added a subscriber: Jonathan Williamson (carter2422).

Such a small but useful change! Gets a thumbs up from me.

This revision is now accepted and ready to land.Oct 21 2014, 9:42 PM
Campbell Barton (campbellbarton) edited edge metadata.Oct 23 2014, 11:14 AM

Note, this patch is fine (in principle),

but I really want to see this menu made more manageable before adding more items.

Eg:

  • Move Edit Translations into a submenu, or only enable for non-release builds.
  • Move Online Manual, Online Python Reference into a submenu.

... with the exceptions of Copy to Selected the last 6 options for the RMB menu are quite obscure.

Suggest we have a Scripting submenu.

It can contain...

  • Edit Source
  • Copy Data Path
  • Copy Data Path (Full)
  • Online API Reference.

... Edit Translations could be moved out too... but it doesn't really fit anywhere else AFAIKS.

Pablo Vazquez (pablovazquez) added a comment.Oct 28 2014, 3:54 PM

Suggest we have a Scripting submenu.

It can contain...

  • Edit Source
  • Copy Data Path
  • Copy Data Path (Full)
  • Online API Reference.

... Edit Translations could be moved out too... but it doesn't really fit anywhere else AFAIKS.

+1 for a Scripting menu.

Edit Translations is quite an obscure item to be on such a prominent menu indeed, +1 to only have it on non-release builds.

Julian Eisel (Severin) updated this revision to Diff 2790.Oct 30 2014, 3:05 PM
Julian Eisel (Severin) retitled this revision from RMB-menu: 'Copy Full Data Path' to Reorganize RMB menu and add option "Copy Full Data Path".
Julian Eisel (Severin) updated this object.
Julian Eisel (Severin) edited edge metadata.
Pablo Vazquez (pablovazquez) awarded a token.Oct 30 2014, 7:38 PM
Julian Eisel (Severin) updated this revision to Diff 3527.EditedFeb 15 2015, 3:19 AM
Julian Eisel (Severin) updated this object.
  • Updated for latest master
  • Fix cases where menu entry is greyed out while not being greyed out in master

@Campbell Barton (campbellbarton) It's not so nice that I had to create another flag for this, but it's the only good solution I can think of. We can't search for an already existing flag, we cant search for but->active, ... So think adding a new flag is fine.

Julian Eisel (Severin) set the repository for this revision to rB Blender.Feb 15 2015, 3:20 AM
Pablo Vazquez (pablovazquez) added a comment.Apr 1 2015, 12:36 PM

Still wishing this was on master! Especially the reorganizing of the menus and getting the Edit Translation only on non-release builds.

What's do you think about this @Campbell Barton (campbellbarton) ? Will test this on the gooseberry branch to get feedback.

Julian Eisel (Severin) mentioned this in rB5baff6246f7f: Reorganized RMB menu and new entry "Copy Full Data Path".Apr 1 2015, 1:43 PM
Julian Eisel (Severin) updated this revision to Diff 4775.Jul 30 2015, 11:57 PM

Update patch to master, make some stuff a bit less ugly

Julian Eisel (Severin) added inline comments.Jul 31 2015, 12:06 AM
source/blender/editors/interface/interface_utils.c
347 ↗(On Diff #4775)

Could also add bool for reverse search, not sure if that's useful though

mathieu menuet (bliblubli) added a subscriber: mathieu menuet (bliblubli).Aug 16 2015, 11:23 AM

Nice patch, why isn't it in master?

Jozef Zajac (zajooo) added a subscriber: Jozef Zajac (zajooo).Dec 17 2015, 9:05 PM

Guys, I am looking forward to see it in the master, as soon as possible ;). Many thanx.

Campbell Barton (campbellbarton) added a comment.EditedDec 18 2015, 7:07 AM

@Julian Eisel (Severin), Can you give an example of where the menu is grayed out?
Would be nice to avoid adding extra special-case flags if possible, I tested the patch and it seemed to work properly, even from pop-ups like the color picker. - See: P301

Lapineige awarded a token.Jan 5 2016, 6:36 PM
Lapineige added a subscriber: Lapineige.
Julian Eisel (Severin) updated this revision to Diff 5747.Jan 5 2016, 10:40 PM
  • Merge branch 'master' into arcpatch-D763
  • Fix wrongly grayed out copy data path buttons
  • Minor tweaks
Julian Eisel (Severin) updated this revision to Diff 5749.Jan 5 2016, 10:45 PM
  • Use mixed code & declarations
Julian Eisel (Severin) added a comment.Jan 5 2016, 10:50 PM

Actually, the last patch version wasn't working correctly either (missing break in loop, grrr!). Now you should be able to see the difference in the 3D view properties region, mainly in the location, rotation, scale and dimension buttons.

Campbell Barton (campbellbarton) removed a reviewer: Campbell Barton (campbellbarton).Aug 11 2016, 6:13 AM
Aaron Carlisle (Blendify) added a subscriber: Aaron Carlisle (Blendify).Jan 1 2017, 11:05 PM

@Julian Eisel (Severin) it has been a year what is the status?

Chris Lee (infin8eye) added a subscriber: Chris Lee (infin8eye).Jan 16 2017, 3:38 AM
Julian Eisel (Severin) mentioned this in D2746: Fix Copy Full Data Path feature.Jul 29 2017, 3:44 PM
Aaron Carlisle (Blendify) mentioned this in D10539: UI: Add Copy Full Data Path to RMB menu.Feb 26 2021, 3:12 AM
Aaron Carlisle (Blendify) mentioned this in rB85623f6a5590: UI: Add Copy Full Data Path to RMB menu.Mar 11 2021, 4:11 AM
Julian Eisel (Severin) closed this revision.Apr 7 2021, 6:22 PM

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
interface/
6 lines
8 lines

Diff 2500

View Options

source/blender/editors/interface/interface_handlers.c

Loading...
View Options

source/blender/editors/interface/interface_ops.c

Loading...