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

Add showLayerDataTable method #31

Merged
merged 5 commits into from
Mar 20, 2025
Merged

Conversation

tomhicks
Copy link
Collaborator

@tomhicks tomhicks commented Mar 17, 2025

Adds showLayerDataTable and hideLayerDataTable methods with sorting capability

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@tomhicks tomhicks force-pushed the tom/app-7281-toggle-data-table-state branch 2 times, most recently from 53af206 to d024bec Compare March 17, 2025 15:30
| Parameter | Type |
| ----------------- | ----------------------------------------------------------------------------- |
| `params` | \{ `layerId`: `string`; `sorting`: [`SortConfig`](../Shared/SortConfig.md); } |
| `params.layerId` | `string` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we make this optional, since felt ui supports showing the table without any layer

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

you can't select layers in view mode so i don't think you could do anything with that option?

Copy link
Collaborator

Choose a reason for hiding this comment

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

presumably people using the sdk might have their own legend / layer selection mechanism, so it'll allow them to keep the "map view" in a stable state

no strong opinion though, whatever you think makes more sense

@tomhicks tomhicks force-pushed the tom/app-7281-toggle-data-table-state branch 2 times, most recently from cea2760 to 8f6f951 Compare March 19, 2025 09:27
Copy link
Collaborator

@noisyneuron noisyneuron 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

@tomhicks tomhicks force-pushed the tom/app-7281-toggle-data-table-state branch from 8f6f951 to ffb05e9 Compare March 20, 2025 08:58
@tomhicks tomhicks merged commit 3c045a0 into main Mar 20, 2025
2 of 3 checks passed
@tomhicks tomhicks deleted the tom/app-7281-toggle-data-table-state branch March 20, 2025 09:00
tomhicks added a commit that referenced this pull request Mar 20, 2025
* feat: add showLayerDataTable method with sorting capability

* docs(changeset): Add showLayerDataTable method

* Add hide method

* Move to UiController

* Allow opening table pane with no layer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants