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

Touch-event substitutes for click-event #50

Merged
merged 10 commits into from
Jan 21, 2021

Conversation

jyuunnii
Copy link
Collaborator

@jyuunnii jyuunnii commented Jan 18, 2021

터치 슬라이드 기능 대체

  • 검색 결과: 카페 리스트(대표 이미지) 슬라이드
  • 카페 정보: 카페 이미지 슬라이드
  • 이 외 사용중인 context 변수명을 변경했습니다. searchValueContext => searchValueCtx
  • 이 외 디자인 수정이 추가 되었습니다. (아이콘, 배경색, 헤더 크기 등 )

사용 라이브러리: react-swipeable

@jyuunnii
Copy link
Collaborator Author

jyuunnii commented Jan 18, 2021

카페 정보를 나타내는 CafeDetail 컴포넌트를 현재 cafe !== null 이면 보여주는 식으로 컨트롤 하고 있는데,
이 부분이 개인적으로는 조금 불안정하게 느껴서

  1. /cafes/:cafeId 식으로 라우터를 추가해서 페이지로 분리해야할지
  2. modal을 이용할지
  3. 그대로 유지할지
    의견을 물어봅니다.

- Responsive height
- Theme color
app/package.json Show resolved Hide resolved
app/src/components/others/CarouselCol/index.tsx Outdated Show resolved Hide resolved
app/src/components/others/CarouselCol/index.tsx Outdated Show resolved Hide resolved
app/src/components/others/CafeDetail/index.tsx Outdated Show resolved Hide resolved
app/src/utils/styled.tsx Outdated Show resolved Hide resolved
- Add bottom dot : represent current index & total number of images
- Delete prev, next sign on image
inhibitor1217
inhibitor1217 previously approved these changes Jan 19, 2021
@jyuunnii
Copy link
Collaborator Author

jyuunnii commented Jan 19, 2021

문제되는 컴포넌트
CarouselHorizontal 컴포넌트가 렌더링 되는 CafeDetail > CafeDetailImageCarousel > CarouselDetailImage >... 이 부분입니다.

에러가 발생하는 경우카페 무아르 처럼 이미지가 2개인 경우에 이미지를 swipe 한 이후 좌측상단의 < (닫기) 버틑을 누르면 발생하고 있습니다. (현재까지 제가 파악하기로는 ㅜ)
Screen Shot 2021-01-19 at 11 33 54 PM

추가로 발견된 부분은 이미지가 3개 이상인 경우 swipe 액션이 계속 일어나도 이미지의 순서가 바뀌지 않는데, 2개인 경우에만 swipe이 일어나면 이미지 순서가 랜덤하게 변경됩니다. (이건 일단 이슈에 추가해놨습니다)

@inhibitor1217
Copy link
Owner

카페 정보를 나타내는 CafeDetail 컴포넌트를 현재 cafe !== null 이면 보여주는 식으로 컨트롤 하고 있는데,
이 부분이 개인적으로는 조금 불안정하게 느껴서

  1. /cafes/:cafeId 식으로 라우터를 추가해서 페이지로 분리해야할지
  2. modal을 이용할지
  3. 그대로 유지할지
    의견을 물어봅니다.

이것의 경우에는, 나중에 페이지 외부에서 카페 detail로 들어가는 링크 같은 걸 쏴주려면 라우터를 추가하는 것이 맞습니다.

@inhibitor1217
Copy link
Owner

문제되는 컴포넌트
CarouselHorizontal 컴포넌트가 렌더링 되는 CafeDetail > CafeDetailImageCarousel > CarouselDetailImage >... 이 부분입니다.

에러가 발생하는 경우카페 무아르 처럼 이미지가 2개인 경우에 이미지를 swipe 한 이후 좌측상단의 < (닫기) 버틑을 누르면 발생하고 있습니다. (현재까지 제가 파악하기로는 ㅜ)
Screen Shot 2021-01-19 at 11 33 54 PM

추가로 발견된 부분은 이미지가 3개 이상인 경우 swipe 액션이 계속 일어나도 이미지의 순서가 바뀌지 않는데, 2개인 경우에만 swipe이 일어나면 이미지 순서가 랜덤하게 변경됩니다. (이건 일단 이슈에 추가해놨습니다)

아마 어제 말한대로 settimeout으로 들어간 dispatch가 컴포넌트 unmount 이후에도 실행되어서 그런 것 같은데요,

inhibitor1217
inhibitor1217 previously approved these changes Jan 21, 2021
@jyuunnii jyuunnii merged commit aa95da5 into inhibitor1217:master Jan 21, 2021
@jyuunnii jyuunnii deleted the feature/app/cafe-slide branch April 3, 2021 03:45
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