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

24-25-App-Assignment-04 이민용 #4

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

myllli
Copy link

@myllli myllli commented Nov 14, 2024

Description

  • 어쩌구 저쩌구
  • MyProfile UI 만들기

Important content

  • 어쩌구 저쩌구
  • 이미지 파일 제대로 만들어진 건가요?

Question

  • 어쩌구 저쩌구
  • test 파일이 빨간색으로 표시되는 이유가 궁금해요

Reference

Copy link
Collaborator

@hyeminililo hyeminililo left a comment

Choose a reason for hiding this comment

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

과제 잘 수행하셨습니다 😊 이미지 잘 나와요 !� 일단 test 파일은 main.dart에서 적은 로직들을 test 할 수 있는 파일이에요 :) 그래서 현재 test 파일에는 초기 코드였던 앱을 test할 수 있는 MyApp() 위젯으로 설정되어있어요. 그래서 없는 위젯을 가리키고 있어서, 오류 상태입니다 ! 과제 완료입니다 수고하셨습니다 😄

leading: Icon(Icons.phone),
title: Text('+82 010-4312-6243'),
),
ListTile(
Copy link
Collaborator

Choose a reason for hiding this comment

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

ListTile을 이용해서, 깔끔하게 표현 잘 하신 것 같아요 :)

child: CircleAvatar(
radius: 60,

backgroundImage: AssetImage('assets/images/KakaoTalk_20241022_235349121.png'),
Copy link
Collaborator

Choose a reason for hiding this comment

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

나중에 image가 많아지면 image이름이 헷갈릴 수 있어서, 되도록 파일의 이름을 용도에 맞게 수정하면 좋을 것 같아요 !

),


Text(
Copy link
Collaborator

Choose a reason for hiding this comment

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

전체 코드 정렬 기능을 이용하시면 조금 더 가독성있게 코드를 볼 수 있어요 👍

),
SizedBox(height: 10),

Text(
Copy link
Collaborator

Choose a reason for hiding this comment

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

github에 대한 정보를 적은 코드라는 공통점이 있기 때문에, 이런 경우, 하나의 Container로 감싸는게 좋아요 !

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.

2 participants