ADT Explorer bugfixes and updates

[ccrowley96]

ADO listed Bug fixes

• Fixes REST error when landing in explorer for the first time. Users can no longer cancel out of the ADT URL picker dialogue before selecting an instance URL. Also removed the call to query models on initial load when no URL is selected. (ADO item)
• Model Graph inheritance lines no longer display implied inheritance. Added logic to slice only the first base model when drawing inheritance lines on the model graph. (ADO item)
• Updated model graph 'Run Layout' button to 'Choose layout' with correct icon. (ADO item)
• Fixed bug where model refresh button did nothing. This was happening when caching was enabled. The cached models would be hit and no refresh was executed. These changes add a forced cache bypass to this button - forcing a refresh. (ADO item)
• Fixed issue where clicking 'Run query' from the model graph had no effect and was confusing to users. Now, when a user runs a query from the model graph, it snaps back to the twin graph and executes the query. (ADO item)
• Fixed issue where ADT URL configuration was not saving when local storage cleared. URL configuration is now also stored in app state and only seeded by local storage - this should fix the multi-tab configuration issue users are facing. (ADO item)
• Updated caching to default OFF (ADO item)

Comparison of model graph inheritance and component line changes

Before

After

GitHub issue bug fixes

• fixes [Bug] Refresh model doesn't work #228
• fixes Hover pop-up for twin shows description from top level extended model #226
• fixes Model graph shows redundant parent's Components link #198
• fixes Model graph shows inheritances to parent, parent's parent, etc #182
• fixes manifest.json referencing "favicon.ico" which is missing, it should be "favicon.png" instead #178

New features

Model upload confirmation

Added model upload confirmation message bar and info dialogue. Clicking view details on the success message bar opens the dialogue showing the details of the uploaded models. The success message bar dismisses after 15 seconds.

Visual changes

Updated panel tabs to Fluent style with lighter background. This helps to clarify what tab is selected and makes their tab nature more apparent.

Fixed high-contrast mode toggle UI.

Before

After

[niusoff]

Overall looks great! The one thing I noticed when playing around with it is an accessibility bug that appears to be present in prod as well, so, it's not a regression - there appears to be some issues with tabbing into/out of the "twin graph" and "model graph" tab buttons. Can repro by clicking on one of them, hitting tab, then shift tab and you can see the focus jump from the "model graph" button to the previous thing in the tab order. Not sure what's triggering this

[ccrowley96]

Overall looks great! The one thing I noticed when playing around with it is an accessibility bug that appears to be present in prod as well, so, it's not a regression - there appears to be some issues with tabbing into/out of the "twin graph" and "model graph" tab buttons. Can repro by clicking on one of them, hitting tab, then shift tab and you can see the focus jump from the "model graph" button to the previous thing in the tab order. Not sure what's triggering this

I think the Fluent UI Pivot Items (tabs) and command bars use arrow keys to navigate within their tab group rather than tab / shift tab. So this might not be a bug, not positive I'm reproing what you hit though.