-
Notifications
You must be signed in to change notification settings - Fork 8
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
df/FloatingPanelEnhancements #108
Conversation
When dragging the handle up or down in Apple Maps, the speed at which you drag helps dictate whether the floating panel moves to the next detent. Move a little bit slowly does not; moving a little bit quickly, does. You could base the decision to go to the next detent on the position AND velocity of the drag. I'm OK with creating another issue for this and tackling it separately. |
It would be nice if we had a custom "summary" detent height (or a "minimumHeight), to allow at least a minimum height needed to display a view's "summary" information. Again, this can be a future issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's looking good! I like the new additions.
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanel.swift
Outdated
Show resolved
Hide resolved
Sources/ArcGISToolkit/Components/FloatingPanel/FloatingPanelDetent.swift
Outdated
Show resolved
Hide resolved
I've increased the height of the handle area.
Opened #132
Opened #133 |
See #14
To see these changes in action, try the Utility Network Sample in the FloatingPanelDemo branch.