Fitbit.NET icon indicating copy to clipboard operation
Fitbit.NET copied to clipboard

Add `EditorBrowsableState` to advanced FitbitClient constructors

Open mxa0079 opened this issue 9 years ago • 4 comments

Let's hide advance mode constructors through this attribute: https://msdn.microsoft.com/en-us/library/system.componentmodel.editorbrowsablestate.aspx

mxa0079 avatar Feb 08 '16 19:02 mxa0079

I don't think that is a good idea, if we hide them then no one will find them and there will be no use in having them :-1: Discovering a new library and API through intellisense is half the way to determining what you can use

WestDiscGolf avatar Feb 08 '16 22:02 WestDiscGolf

I agree. Do you know a way to influence they way in which Intellisense displays constructors?

mxa0079 avatar Feb 17 '16 18:02 mxa0079

No. As part of the constructor summary descriptions though which gets displayed when intellisense kicks in we could add "Advanced - <description / proposed usages>" so people know that it's a more advanced one?

WestDiscGolf avatar Feb 17 '16 19:02 WestDiscGolf

yeah, let's not hide. Typically, the order is the fewer parameters first in intellisense, so I think we've got that covered. We can add comment headers to that one to make it clear it's more advanced.

aarondcoleman avatar Feb 20 '16 21:02 aarondcoleman