Skip to content
This repository has been archived by the owner on Jan 8, 2025. It is now read-only.

Youtubeclone #68

Merged
merged 8 commits into from
Feb 12, 2021
Merged

Conversation

Ambikadulal
Copy link
Contributor

youtubeclone

child: ListTile(
leading: CircleAvatar(
backgroundImage: NetworkImage(
'https://cdn.logo.com/hotlink-ok/logo-social.png'),
Copy link
Owner

Choose a reason for hiding this comment

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

You can use links variables from assets.dart instead of hardcoding.
Use other images not logos..

README.md Outdated
@@ -138,7 +139,7 @@ The code for Springy Widget is taken and refactored from (https://github.com/mat
- Stateless & Stateful Widgets
- Custom widgets for list items, category items etc.

<img height="480px" src="screenshots/grocery_home.png"> <img height="480px" src="screenshots/grocery_details.png"> <img height="480px" src="screenshots/grocery_cart.png"> <img height="480px" src="screenshots/grocery_wishlist.png"> <img height="480px" src="screenshots/grocery_options.png">
<img height="480px" src="screenshots/youtube.jpg"> <img height="480px" src="screenshots/grocery_home.png"> <img height="480px" src="screenshots/grocery_details.png"> <img height="480px" src="screenshots/grocery_cart.png"> <img height="480px" src="screenshots/grocery_wishlist.png"> <img height="480px" src="screenshots/grocery_options.png">
Copy link
Owner

Choose a reason for hiding this comment

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

Wrong section keep this under miscellaneous, this is Grocery App UI kit

@@ -76,6 +76,7 @@ import 'package:flutter_ui_challenges/src/pages/login/login7.dart';
import 'package:flutter_ui_challenges/src/pages/login/login6.dart';
import 'package:flutter_ui_challenges/src/pages/misc/loaders.dart';
import 'package:flutter_ui_challenges/src/pages/grocery/ghome.dart';
import 'package:flutter_ui_challenges/src/youtube_clone/HomePage.dart';
Copy link
Owner

Choose a reason for hiding this comment

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

You have created a new folder youtube_clone, this will break the code view, you will not be able to view the code in the app, as youtube_clone is not available in the pubspec, just put youtube_clone.dart inside misc folder instead of creating new folder.

Also HomePage.dart is not proper naming it should have been home_page.dart

@@ -332,6 +333,7 @@ final List<dynamic> pages = [
MenuItem(
title: "UI Kits (Clones)",
items: [
SubMenuItem('Youtube Clone App', HomePage(), path: HomePage.path),
Copy link
Owner

Choose a reason for hiding this comment

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

Name should be something like YoutubeClonePage() instead of HomePage(), its confusing.

Copy link
Owner

Choose a reason for hiding this comment

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

Keep it under misc section. UI kits clones are the sections for such UIs that have implemented multiple pages of an existing app or UI kit

@lohanidamodar lohanidamodar merged commit 3c851fd into lohanidamodar:master Feb 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants