Skip to content

Conversation

70RMUND
Copy link

@70RMUND 70RMUND commented Dec 20, 2018

Hey KN4CK3R,

One of the things i've always wanted in Reclass.NET was the ability to hide nodes for improved visibility and remove the noise on non-relevant padding. I decided to pull out visual studio this morning and just do it =P

Here are the Features in this pull request:

  • "Hide selected Node(s)" context menu item
  • "Unhide child Node(s)" context menu item
  • "Unhide Node(s) below" context menu item
  • "Unhide Node(s) above" context menu item
  • Changes to *.reclass parser to understand/import legacy 'bHidden="1"' attributes
  • Changes to *.rcnet parser to write/read hidden="true" attributes for the modern rcnet schema

I did a bunch of node validation to qualify whether to enable to disable the context menu items. I also wonder if these 4 context menu items should be placed into a submenu labeled "Node Visibility", not sure if that was cleaner and didn't want to explode the size of the main context menu. Feel free to edit all this at your leisure.

I did as much testing as I could, but perhaps may need a little more incase i introduced weird corner cases.

Here is a preview:

image

@KN4CK3R
Copy link
Member

KN4CK3R commented Dec 21, 2018

Thank you, looks great! An early christmas present this year.

@KN4CK3R KN4CK3R merged commit f0317bf into ReClassNET:master Dec 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants