Skip to content

taimaa-aldossary/Task-2T2#2

Open
taima-aldossary wants to merge 4 commits intoGDSC-IAU:masterfrom
taima-aldossary:master
Open

taimaa-aldossary/Task-2T2#2
taima-aldossary wants to merge 4 commits intoGDSC-IAU:masterfrom
taima-aldossary:master

Conversation

@taima-aldossary
Copy link

I've developed a social media application featuring four main pages: Home, Search, Profile, and Notifications.
In the Home page, users can browse through posts and stories shared by others, and like the posts.
Notifications keep users updated on activities related to their posts, such as receiving likes.
The Profile section provides users with an overview of their own posts and personal information.
Screenshot 2024-02-27 203415
Screenshot 2024-02-27 203435

Screenshot 2024-02-27 203509 Screenshot 2024-02-27 203543 Screenshot 2024-02-27 203612

import 'package:flutter/material.dart';
import 'package:google_fonts/google_fonts.dart';
import 'package:social_media/screens/postcard.dart';
import 'package:social_media/screens/profile.dart';

Choose a reason for hiding this comment

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

no need for this import

mainAxisAlignment: MainAxisAlignment.center,
children: [
Container(
width: MediaQuery.of(context).size.width / 1.1,

Choose a reason for hiding this comment

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

The width of the container is set to 1.1, which might not be responsive on all devices. Consider using MediaQuery.of(context).size.width * 0.9 or a similar fraction to ensure responsiveness across various screen sizes.

setState(() {
posts[index].Likedpost = !posts[index].Likedpost;
});ScaffoldMessenger.of(context).showSnackBar(
SnackBar(
Copy link

@arwaalkhathlan arwaalkhathlan Mar 15, 2024

Choose a reason for hiding this comment

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

the snack bar that will show doesn't have a set deration(timer) it's better to give it a set deration in the code

Copy link

@arwaalkhathlan arwaalkhathlan left a comment

Choose a reason for hiding this comment

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

great project i was so impressed by it.
your code was clean but for future projects utilize comments to make it more readable.
also u can use ctrl+s to hot reload and it will organize your code.
but overall you have great skills i loved your project!!

String profilepic = 'assets/post9.jpg';
@override
Widget build(BuildContext context) {
return Scaffold(backgroundColor: Colors.black,appBar: AppBar( iconTheme: IconThemeData(color: Colors.green),

Choose a reason for hiding this comment

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

You've set the icon theme for the AppBar using IconThemeData(color: Colors.green). However, you're not actually using any icons in the AppBar. If you intend to use icons in the AppBar, make sure to add them. If not, this line is useless you can remove it.

body: Column(
crossAxisAlignment: CrossAxisAlignment.center,
children: [
const SizedBox(height: 10,),

Choose a reason for hiding this comment

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

there no point of this sized box you can remove it for a cleaner code.

children: [
const SizedBox(height: 10,),
Padding(
padding: const EdgeInsets.only(left: 20.0),

Choose a reason for hiding this comment

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

try replacing it with this instead it will look better
padding: const EdgeInsets.only(
top: 50, bottom: 20, left: 20, right: 20),
try replacing it with this instead it will look better

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