Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Add sort key api for FillLayer and LineLayer - #209

Merged
Chaoba merged 1 commit into
masterfrom
kl-line-fill-sort-key
Mar 6, 2020
Merged

Chaoba merged 1 commit into
masterfrom
kl-line-fill-sort-key

Conversation

@Chaoba

@Chaoba Chaoba commented Mar 2, 2020 •

Copy link
Copy Markdown
Contributor

Resolves #212
This pr add sort key api for Filllayer and Lilllayer, part of #197

The jni binding pr: https://github.com/mapbox/mapbox-gl-native/pull/16256/files

@tobrun tobrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did you generate these files? or manually wrote them?

@Chaoba

Chaoba commented Mar 2, 2020

Copy link
Copy Markdown
Contributor Author

@tobrun I manually write them. Is there any method to generate them?

@tobrun

tobrun commented Mar 2, 2020

Copy link
Copy Markdown
Member

Our Layer and paint/layout property API is fully generated from the style-specification. The version of the style specification comes from the mapbox-gl-js submodule within mapbox-gl-native. These properties will be automatically supported once we update to the new style spec version and we have reran the command to generate new code. I created a small task for you to start looking into how this code generator works in #212. Let me know if you need more information!

@Chaoba Chaoba changed the title Add sort key api for Filllayer and Lilllayer Add sort key api for FillLayer and LilneLayer Mar 3, 2020
@Chaoba Chaoba changed the title Add sort key api for FillLayer and LilneLayer Add sort key api for FillLayer and LineLayer Mar 3, 2020
@Chaoba
Chaoba force-pushed the kl-line-fill-sort-key branch 2 times, most recently from 68fc21f to d6cf174 Compare March 4, 2020 09:17

@pengdev pengdev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM :shipit:


writeIfModified(`vendor/mapbox-gl-native/platform/android/src/style/layers/${layerFileName}_layer.hpp`, layerHpp(layer));
writeIfModified(`vendor/mapbox-gl-native/platform/android/src/style/layers/${layerFileName}_layer.cpp`, layerCpp(layer));
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 should this jni generator run on the gl-native side? I think it also make sense to add it here, because we are going to migrate the jni code to this repo. Currently we need one extra step to commit this change in the gl-native submodule.
cc @tobrun

@Chaoba
Chaoba merged commit 04722fd into master Mar 6, 2020
@Chaoba
Chaoba deleted the kl-line-fill-sort-key branch March 6, 2020 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ejs template files are not compatible with Androidx

3 participants