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

Mobile UI #39

Merged
merged 27 commits into from Nov 16, 2021
Merged

Mobile UI #39

merged 27 commits into from Nov 16, 2021

Conversation

salahamassi
Copy link
Contributor

@salahamassi salahamassi commented Nov 14, 2021

Description

Working on mobile UI.

1- We have a different screen for mobile, web common widgets like key_pad.

image

2- create routes for web and mobile

image

3- check the platform and return the correct route

final String initial = kIsWeb ? WebRoutes.SPLASH : MobileRoutes.HOME;

final List<GetPage<Widget>> routes = kIsWeb ? webRoutes : mobileRoutes;

any suggestions?

Fixes #17

Type of change

  • New feature

Checklist:

  • I have performed a self-review of my own code.
  • My changes generate no new warnings on the project.

@AhmedHamdan54
Copy link
Contributor

Please resolve the conflicts so we can merge this PR:

lib/features/carts/presentation/controllers/carts_controller.dart
lib/features/carts/presentation/views/cart_view.dart
lib/features/home/presentation/views/home_view.dart
lib/features/home/presentation/widgets/header_home.dart
lib/main.dart
lib/routes/app_pages.dart
lib/routes/app_routes.dart

@salahamassi
Copy link
Contributor Author

please check now @AhmedHamdan54

@AhmedHamdan54
Copy link
Contributor

Solves #17
Thanks @salahamassi

@AhmedHamdan54 AhmedHamdan54 merged commit c409e52 into The-POS:main Nov 16, 2021
@salahamassi salahamassi deleted the mobile-ui branch December 3, 2021 23:53
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.

Home UI Mobile
2 participants