Add errorSheet that scrollable error text #112
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new error handling mechanism by replacing
errorAlert
witherrorSheet
across the codebase, alongside the addition of a newErrorTextSheet
component. It also updates the project configuration to include the newErrorTextSheet.swift
file.Error Handling Updates:
errorAlert
witherrorSheet
in various views, includingContentView
,BranchesView
,CommitCreateView
,FolderView
, and more. This change enables the use of a sheet-based error display instead of alerts for a more consistent user experience. [1] [2] [3] [4] and others)ErrorTextSheet
component to display error messages in a sheet with support for text selection and a dismiss button.View
extension to include the newerrorSheet
method, which usesErrorTextSheet
for error handling.Project Configuration Updates:
ErrorTextSheet.swift
to the project files inGitClient.xcodeproj/project.pbxproj
to ensure it is included in the build process. [1] [2] [3] [4]