Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for configuring in storyboards and from a delegate #6

Merged
merged 13 commits into from
Jan 30, 2023

Conversation

Eskils
Copy link
Contributor

@Eskils Eskils commented Jan 18, 2023

Skjermbilde 2023-01-18 kl  14 39 48

  • Supports configuring in storyboards and nib-files.
  • Adds WelcomeSheetDelegate to configure pages and handle didDismiss
  • Adds a demo project for UIKit with a storyboard
  • Adds models UIWelcomeSheetPage and UIWelcomeSheetPageRow

There is a problem with referencing system images from IB. As those are UIImages, passing them to Image makes them look stretched. See image. Not sure if there’s any way to fix this.

Simulator Screen Shot - iPhone 14 Pro - 2023-01-19 at 19 34 07

In progress

  • Test didDismiss properly
  • Add documentation on how to configure in storyboards
  • Add all pages to the demo

Add `WelcomeSheetControllerConfigurator` and `WelcomeSheetControllerConfiguratorPageRow` to be able to configure WelcomeSheet entirely from Storyboards
Remove fatalError from inis from coder
Make WelcomeSheetPage conform to NSObject and have an objective-c runtime
Change UIKit example
Rollback changes to UIKit-WelcomeSheetDemo
Add model UIWelcomeSheetPage
Add model UIWelcomeSheetPageRow
Make `pages` an IBOutlet (Outlet collection)
Make WelcomeSheetController open
Make ModalWelcomeSheetUIHostingController open
Fix dismissal
Add didDismiss to demo
@Eskils Eskils marked this pull request as ready for review January 19, 2023 18:29
@MAJKFL MAJKFL merged commit d39dbb0 into MAJKFL:main Jan 30, 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.

None yet

2 participants