Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Widget and Feature Enhancements #244

Merged
merged 8 commits into from
Oct 24, 2023

Conversation

TimPurdum
Copy link
Collaborator

Widgets

  • Supports rendering Widgets anywhere in Razor markup, using a new MapView reference parameter.
  • ExpandWidget now supports custom HTML child content in Razor markup, and can combine a child widget and custom HTML

Layers

  • Added FeatureLayer properties
    • ApiKey
    • blendMode
    • legendEnabled
    • popupEnabled
  • Added method FeatureLayer.GetCapabilities
  • Added VisualVariables for Color and Opacity

MapView

  • Exposed AddWidget and RemoveWidget as public methods.

Pro Support

  • Added checks in MapView.AddLayer and Layer.Load for Pro JS module
  • Added support for registering ProProperties in MapComponent

Significant new unit tests surrounding changes

@@ -352,6 +380,11 @@ public async Task<FeatureEditsResult> ApplyEdits(FeatureEdits edits, FeatureEdit
return await JsLayerReference!.InvokeAsync<Domain?>("getFieldDomain", fieldName, feature);
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs a comment

{
try
{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if there is not a pro module, will the try block throw an exception every time?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will, however, this method is only reached if you are trying to nest an otherwise invalid component, as the inherited override implementations correctly register all known child components.

Copy link
Collaborator

@AndersenBell AndersenBell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@TimPurdum TimPurdum merged commit 4d80196 into develop Oct 24, 2023
@TimPurdum TimPurdum deleted the feature/feature-enhancements-pro-58 branch October 24, 2023 02:35
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