Skip to content

Conversation

@zkline101
Copy link
Contributor

Description

This PR implements Filter building scene layer in Layers category.
URL to README:

(There is a known issue where the basemap doesn't render sometimes)

How To Test

  • Play around in the settings to filter floors and toggle the visibility of sublayers
  • Tap on a feature to show the attributes in a popup

Screenshots

Simulator Screenshot - iPad Pro 13-inch (M5) - 2025-11-25 at 15 51 27 Simulator Screenshot - iPad Pro 13-inch (M5) - 2025-11-25 at 15 53 31

@zkline101 zkline101 requested a review from yo1995 November 26, 2025 00:18
@zkline101 zkline101 self-assigned this Nov 26, 2025
@zkline101 zkline101 requested review from a team and philium and removed request for a team November 26, 2025 00:18
import SwiftUI

extension FilterBuildingSceneLayerView {
/// The building group sublayer toggle which has a toggle for the visbility of the group
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
/// The building group sublayer toggle which has a toggle for the visbility of the group
/// The building group sublayer toggle which has a toggle for the visibility of the group

/// - Parameter groupSublayer: The group sublayer which is used to build this view.
init(groupSublayer: BuildingGroupSublayer) {
// Sets the initial value of the toggle to the
// current visbility of the group sublayer.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// current visbility of the group sublayer.
// current visibility of the group sublayer.

/// - Parameter sublayer: The sublayer to help build this view.
init(sublayer: BuildingSublayer) {
// Sets the initial value of the toggle to the
// current visbility of the sublayer.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// current visbility of the sublayer.
// current visibility of the sublayer.

Button("Settings", systemImage: "gear") {
settingsAreVisible = true
}
.popover(isPresented: $settingsAreVisible) { [settings] in
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does the settings need to be captured?

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