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 includes significant refactoring to the
AppService
class in theinternal/services
package, focusing on removing theLayoutService
and integrating its functionality directly into theAppService
. The changes improve the structure and readability of the code by consolidating layout management.Key changes include:
Integration of Layout Management:
LayoutService
and integrated its functionalities directly intoAppService
. (internal/services/app.go
,internal/services/io.go
) [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]Simplification of Notification and Modal Handling:
LayoutService
notification methods with direct calls tolayout.GetNotifier()
. (internal/services/app.go
,internal/services/io.go
) [1] [2]showModal
method inAppService
for handling modal dialogs. (internal/services/io.go
)Enhancement of Search and Table Handling:
LayoutService
search field and result table methods with direct calls tolayout.GetSearch()
andlayout.GetTable()
. (internal/services/app.go
,internal/services/io.go
) [1] [2] [3] [4] [5]Removal of
LayoutService
:layout.go
file and all associated methods and interfaces forLayoutService
. (internal/services/layout.go
)These changes streamline the code and reduce redundancy by consolidating layout-related functionalities into the
AppService
class, enhancing maintainability and readability.