Hide private members should be a setting, not a tool

Question

Hide private members should be a setting, not a tool

GeertvanHorrik opened this issue 9 years ago · comments

I think that if VisibilitySettings.HidePrivateMembers is a setting (and thus can be determined dynamically), it works much better than using it as a tool (where clicking the button does the calculation of a moment in time).

What do you think?

Geert van Horrik · Answer 1 · Mon May 18 2015 16:28:03 GMT+0800 (China Standard Time)

I see it's also stored as a property. Not sure why the need for explicitly writing the identifiers out is needed.

Gerrit Gazic · Answer 2 · Mon May 18 2015 23:34:21 GMT+0800 (China Standard Time)

In fact HidePrivateMembers is a property which gets checked during the parse step. They are included in the ExcludedIdentifiers property for the correct visualization in the UI at the moment. It is an old mechanism and could be replaced in the future.