You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've been seeing a crash in Bookmarks on macOS when attempting to perform a layout pass and the frame is zero. Definitely a result of a behavior change in the latest incarnation of SwiftUI. It's currently unclear to me if this is an actual SwiftUI bug, or an issue in the layout logic / custom constraints resolution in SelectableCollectionView. Either way, ideally SelectableCollectionView should be robust to this kind of issue though and we should add a test case for this and guard against it if possible.
The text was updated successfully, but these errors were encountered:
jbmorley
added a commit
to inseven/bookmarks
that referenced
this issue
Jan 7, 2025
This is a work around for a possible issue in `SelectableCollectionView` (see inseven/SelectableCollectionView#26) which was leading to a crash on first run when the default window frame hadn't been set. This change explicitly sets the minimum size of the collection view to 640x480 which hopefully isn't _too_ large for users with smaller screens.
This is a work around for a possible issue in `SelectableCollectionView`
(see inseven/SelectableCollectionView#26)
which was leading to a crash on first run when the default window frame
hadn't been set. This change explicitly sets the minimum size of the
collection view to 640x480 which hopefully isn't _too_ large for users
with smaller screens.
I've been seeing a crash in Bookmarks on macOS when attempting to perform a layout pass and the frame is zero. Definitely a result of a behavior change in the latest incarnation of SwiftUI. It's currently unclear to me if this is an actual SwiftUI bug, or an issue in the layout logic / custom constraints resolution in SelectableCollectionView. Either way, ideally SelectableCollectionView should be robust to this kind of issue though and we should add a test case for this and guard against it if possible.
The text was updated successfully, but these errors were encountered: