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

Design, Refactor: 탭바, 팝업, 로그인 화면 디자인 오류 수정 및 마이페이지, UserDefault 리팩토링 #131

Merged
merged 9 commits into from
Mar 2, 2024

Conversation

syss220211
Copy link
Member

@syss220211 syss220211 commented Mar 1, 2024

✨ Design, Refactor: 탭바 및 로그인 화면 디자인 오류 수정 및 마이페이지 리팩토링 #89 #117 #113 #124

Pasted Graphic 5
UserDefault들 키값 보기 힘들어서 정리해서 반영해 놓았습니다.
// DesignSystem > CustomAlert > PopupContent.swift -> 팝업 문구 정리
struct PopupContent {
let title: String let desc: String
//일반 팝업
static let logout = PopupContent(title: "로그아웃 하시겠어요?", desc: "로그아웃 후에는 서비스 이용이 불가해요. ")
static let remove = PopupContent(title: “정말 탈퇴 하시겠어요?", desc: " 사용하신 계정 정보는 회원 탈퇴 후 n모두 삭제되며, 복구가 불가합니다. ")
static let stoppiary = PopupContent(title: "잠깐! 일기 작성을 중단하시나요?", desc: "여기서 그만두면 지금까지 작성한 n글이 모두 사라집니다! ")
//가이드 팝업
static let thinking = PopupContent (title: "[생각] 쓰기 꿀팁!", desc: "긍정, 부정적인 생각 모두 있는 그대로 적으며 n~하다고 생각했다, 왜~일까?와 같은 표현을 활용하면 더욱 명확히 표현할 수 있어요! ")
static let issue = PopupContent (title: "[사건] 쓰기 꿀팁!", desc: "언제, 어디서, 누가 관련되었는지 기록하며 n사건을 요약하는 제목을 붙여보세요! n고민하지 말고 생각나는대로 적어봐요:) ")
static let behavior = PopupContent(title: "[행동] 쓰기 꿀팁!", desc: "어떻게 행동했는지, 그래서 결과는 어땠는지 n돌이켜보며 해당 행동이 나에게 어떤 영향을 n 미쳤는지 기록해보세요!")


// 사용시
.popup (isShowing: $recordViewModel.isShowingOutPopUp,
type: .doubleButton(1eftTitle: "그만두기", rightTitle: "이어쓰기"),
title: PopupContent. stop.title
desc: PopupContent.stop.desc)

가이드 팝업까지 적용시켜 놓고자 했으나 enum화가 되어 있어서.. 바꾸지 않았습니다!
기본 팝업과 그만두기/이어쓰기 만 적용해 놓았어요.


회원가입 뷰에 있는 움직이는 애니메이션 효과 Modifier로 만들어 놓았습니다.
원하시는 곳에 가져다 사용하셔도 될것 같아요.

// LoginIntroView.swift
extension View {
func animationSimi (yOffset: CGFloat) -> some View {
    self.modifier (AnimationSimi (yOffset: yOffset))
        }
}
struct AnimationSimi: ViewModifier {
@State var yOffset: CGFloat
    func body (content: Content) -> some View {
    content
        .modifier(CloudFloatingAnimation(offset: yOffset))
        .onAppear {
            withAnimation (Animation.easeInOut (duration: 0.7) .repeatForever ()) {
            self.yOffset = 10
            }
        }
    }
}

// 사용 방법
@State private var yoffset: CGFloat = 0

Image("SimiSmile")
  .animationSimi (yOffset: yOffset)
  • 추가로 마이페이지가 리팩토링 및 디자인 안 맞는 부분이 많아서 모두 수정하여 반영해 놓았음
  • 커스텀 팝업 수정된 사항도 모두 반영 완료
  • TextEditor에서 200자 넘어도 추가되던 오류 수정완료 (이제 200자 넘게 입력하면 입력 안 됨!!)

🙋🏻 PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📷 Key Changes

✍🏻 To Reviewers

@syss220211 syss220211 added 🪄Feat 새로운 기능 추가 😎Refactor 코드 리팩토링 🐶서연 서연 🎨Design 디자인 수정 (CSS, UI 디자인) labels Mar 1, 2024
Copy link
Contributor

@yyomzzi yyomzzi left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Contributor

@LutherCho LutherCho left a comment

Choose a reason for hiding this comment

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

UserDefaults 키 값들 한군데 모아놔야 될거 같다고 생각했는데 👍 입니다 !!

Copy link
Contributor

@a-jb97 a-jb97 left a comment

Choose a reason for hiding this comment

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

오.. 정리 감사합니다! 고생하셨습니다~

@syss220211 syss220211 merged commit ac16418 into dev Mar 2, 2024
LutherCho pushed a commit that referenced this pull request Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨Design 디자인 수정 (CSS, UI 디자인) 🐶서연 서연 🪄Feat 새로운 기능 추가 😎Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants