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

feat(portal) Wks add button to zoom on selected feature #796

Merged
merged 8 commits into from
Jun 15, 2022

Conversation

josee666
Copy link

@josee666 josee666 commented May 31, 2022

#751

What is the new behavior?
add button in wks to zoom on selected feature

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x ] No

Copy link
Contributor

@PhilippeLafreniere18 PhilippeLafreniere18 left a comment

Choose a reason for hiding this comment

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

Works fine for me ! I think it miss the "padding top: 5px" to make sure the button is aligned with the others.

@PhilippeLafreniere18 PhilippeLafreniere18 marked this pull request as ready for review June 8, 2022 16:19
olFeaturesSelected.push(localOlFeature);
}
if (this.map.viewController.padding[2] === 0) {
this.map.viewController.setPadding({bottom: 280});
Copy link
Member

Choose a reason for hiding this comment

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

1-C,est risqué pas mal.. faudrait trouver un moyen moins hardcodé
2-Faudrait trouver un moyen de remettre la valeur d'origine après la déselection....

@PhilippeLafreniere18 PhilippeLafreniere18 merged commit 7757488 into next Jun 15, 2022
@pelord pelord deleted the wks_zoomOnSelect branch October 24, 2022 15:00
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.

3 participants