teamtreedyn / Binoculars

{literally watching your every move*} *not really, just when your graph runs. 🔍
MIT License
28 stars 8 forks source link

Improve and unify the User Interface #19

Open StudioLE opened 5 years ago

StudioLE commented 5 years ago

At present we're calling standard windows dialog boxes but these are stylistically dissimilar to the UI used throughout Dynamo, Project Refinery, Revit, and Dynamo Player.

Rather than using MessageBox we should call our own UI class that outputs dialog boxes using context specific interface elements. Hopefully this will be as simple as calling whichever UI methods Dynamo or Revit use but it might not be this simple.

Binoculars Current UI

2019-04-19 Binoculars T Cs

2019-04-19 Binoculars Privacy

Standard Dynamo UI

2019-04-19 Dynamo Privacy

2019-04-19 Dynamo T Cs

2019-04-19 Dynamo Working Range

2019-04-19 Dynamo About

2019-04-19 Dynamo Packages

Standard Revit UI

2019-04-19 Revit Template

2019-04-19 Revit Transmitted

2019-04-19 Revit Units

2019-04-19 Revit Upgrade

2019-04-19 Revit Warning

Standard Dynamo Player UI

2019-04-19 Dynamo Player Inputs

2019-04-19 Dynamo Player UI

Standard Project Refinery UI

[To be added later]

StudioLE commented 5 years ago

Continuing an earlier conversation:

https://github.com/teamtreedyn/Binoculars/blob/e0cf00e74b95c37f2ac07381bae780b42e9a1c58/BinocularsViewExtension.cs#L48

@radumg: or don't :) the current box does the trick nicely and has no overhead - unless the interaction needs to be more complex, with checkboxes, etc, i wouldn't add anything just for a message box.

To highlight the privacy implications of the data being collected and why this data is useful for diagnostic and business insight we're going to need a lot more space the current dialog box.

Autodesk already provide a good example (see below) of how to do this and we should looks to provide our own version which I don't believe will be possible with the default MessageBox.

2019-04-19 Dynamo Privacy

brencass commented 5 years ago

@StudioLE @WaynePatrickDalton

Have a look at the about window in the following branch i have created in my fork it will create the about window as per the below image within dynamo.

Github Branch: https://github.com/brencass/Binoculars/tree/About-WIndow Main class modification for loading about window: https://github.com/brencass/Binoculars/commit/d0f4d084f3d6f9eba47af019aa49d416e72f61b0#diff-fba541edf4f828f1ffc326db0a23e870

image

StudioLE commented 5 years ago

Thanks for this Brendan, it looks much simpler than I anticipated.

Comparison (for future reference): https://github.com/teamtreedyn/Binoculars/compare/master...brencass:About-WIndow

brencass commented 5 years ago

@StudioLE No worries, though it is probably one of the more simple examples of WPF windows.