Skip to content

Conversation

@chohongm
Copy link
Contributor

@chohongm chohongm commented Apr 1, 2024

Screenshot 2024-04-01 at 1 35 26 PM

@chohongm chohongm self-assigned this Apr 1, 2024
@bang9
Copy link
Contributor

bang9 commented Apr 1, 2024

timestamp 영역에 맞춘 absolute empty container 도 있던거같은데, 고거는 캐러셀에서 안나오나요?

@chohongm
Copy link
Contributor Author

chohongm commented Apr 1, 2024

@bang9 빈 container 도 카루셀에 있습니다. timestamp랑 별개로 둘다 rendering 됩니다

Screenshot 2024-04-01 at 1 50 13 PM

Screenshot 2024-04-01 at 1 50 07 PM

@bang9
Copy link
Contributor

bang9 commented Apr 1, 2024

@bang9 빈 container 도 카루셀에 있습니다. timestamp랑 별개로 둘다 rendering 됩니다

저건 왼쪽정렬 따로 안해도 되나요?

@chohongm
Copy link
Contributor Author

chohongm commented Apr 1, 2024

@bang9 넵 빈 container 라서 정렬이 필요없어요.

@bang9
Copy link
Contributor

bang9 commented Apr 1, 2024

@bang9 넵 빈 container 라서 정렬이 필요없어요.

빈 container 이기는 한데, 타임스탬프 위치를 맞춰주려고 일부러 추가한거로 이해를 해서요
너비가 가득 차서 정렬이 필요 없으면 패스해도 될 것 같습니다.

@chohongm
Copy link
Contributor Author

chohongm commented Apr 1, 2024

@bang9 밑에 붙는 메세지랑 16px 만큼 거리를 두기 위한 empty container 라서 width 가 100%로 잡혀있어서 정렬이 필요없습니다~

@bang9
Copy link
Contributor

bang9 commented Apr 1, 2024

@bang9 밑에 붙는 메세지랑 16px 만큼 거리를 두기 위한 empty container 라서 width 가 100%로 잡혀있어서 정렬이 필요없습니다~

넵! 좋습니다~

@chohongm chohongm merged commit 42b5884 into v3.14.0-beta-1 Apr 1, 2024
@chohongm chohongm deleted the feat/v3/AC-1230-carousel-timestamp-location-change branch April 1, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants