Skip to content

Conversation

@rfcbf
Copy link
Contributor

@rfcbf rfcbf commented Dec 18, 2025

Description

Type of Change

  • πŸŽ‰ feat: New feature
  • πŸ› fix: Bug fix
  • πŸ“ docs: Documentation update
  • πŸ’„ style: Code style/formatting changes
  • ♻️ refactor: Code refactoring
  • ⚑ perf: Performance improvement
  • βœ… test: Adding or updating tests
  • πŸ”§ chore: Maintenance tasks
  • πŸ—οΈ build: Build system changes
  • πŸ‘· ci: CI configuration changes

Related Issue

Screenshots/Videos

Testing

  • Tested on iPhone (specify model/iOS version):
  • Tested on iPad (specify model/iOS version):
  • All unit tests pass
  • No SwiftLint warnings/errors

Checklist

  • My code follows the project's code style
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have updated the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • My branch is up-to-date with the base branch

Additional Notes

Renames `FeedRootViewModel` to `FeedMainViewModel` for clarity.

Updates `FeedMainView` to use the renamed ViewModel and initializes the `items` with a fetch limit of 10.

Improves the initial data loading logic by ensuring it only runs once.
Adds a check to prevent refreshing while a refresh is already in progress.

Streamlines navigation title.

Removes context menu sheet options for favoriting, reading posts, as well as the refreshable modifier and topBarLeading toolbarItem as the button functionality has been removed
Simplifies the context menu structure for a clearer and more efficient user experience on the watchOS app.

Removes unnecessary ScrollView and VStack nesting to streamline the layout.
Refactors the `FeedMainView` to improve the layout
based on the current loading state.

This change enhances the user experience by providing
a more consistent and predictable interface during data loading.
@cassio-rossi cassio-rossi self-requested a review December 18, 2025 17:20
@cassio-rossi cassio-rossi merged commit f399303 into MacMagazine:release/v5 Dec 18, 2025
2 checks passed
@rfcbf rfcbf deleted the feature/update_watch branch December 18, 2025 17:46
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.

2 participants