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

SUF.mapLayers is now awkward #737

Open
davesmith00000 opened this issue May 9, 2024 · 0 comments
Open

SUF.mapLayers is now awkward #737

davesmith00000 opened this issue May 9, 2024 · 0 comments

Comments

@davesmith00000
Copy link
Member

mapLayers was introduce to SceneUpdateFragment so that you can ...map over the layers... and modify common attributes.

With the recent changes, you are now mapping over LayerEntrys and the Layer ADT, which is far less convenient.

This is probably a more subtle problem than it seems, for instance if you make a mapContentLayers method, should it map over top level content layers or nested ones too?

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

No branches or pull requests

1 participant