Skip to content

Conversation

@stevenzeck
Copy link
Contributor

@stevenzeck stevenzeck commented Jul 1, 2022

This PR adds more screens and functionality to the TestApp using SwiftUI:

  • Removed the use of ViewModels
  • Delete an OPDS feed via swipe
  • Catalog Detail screen, with navigation links, groups, and publications
  • Publication Detail screen, download button does not work yet

Notes

  • Publication and Group are both now Identifiable
  • At this time, the links shown on Catalog Detail are not in a List format (where it has the chevron arrow). This is because LazyVGrid and LazyHGrid don't lazy load items when they're nested in a List, so the links are plain links and the entire catalog detail screen is in a ScrollView

Left to do

  • Download a publication from OPDS feed
  • Rename a feed
  • Add a book to the bookshelf via URL or selecting a file on the device
  • Open a book from the bookshelf
  • Delete a book from bookshelf
  • Localization and accessibility

@stevenzeck stevenzeck marked this pull request as draft July 1, 2022 01:52
@stevenzeck stevenzeck marked this pull request as ready for review July 9, 2022 16:50
@stevenzeck
Copy link
Contributor Author

I marked this as ready for review. Everything under "Left to do" will be a next PR. @mickael-menu @gatamar any feedback would be welcomed, especially CatalogDetail

Copy link
Member

@mickael-menu mickael-menu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, one step closer to full SwiftUI! 🔥

@mickael-menu mickael-menu merged commit f3bb753 into readium:swiftui Jul 18, 2022
@mickael-menu mickael-menu deleted the swiftui branch July 18, 2022 07:33
@mickael-menu mickael-menu restored the swiftui branch July 18, 2022 07:34
stevenzeck added a commit to stevenzeck/swift-toolkit that referenced this pull request Mar 23, 2023
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.

3 participants