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

feat: 🎸 add carousel components #9

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

feat: 🎸 add carousel components #9

wants to merge 14 commits into from

Conversation

jumpjet0524
Copy link
Collaborator

跑馬燈獨立出一個component,箭頭固定大小、位置置中,還沒加型別檢查

@ek2061 ek2061 self-requested a review October 9, 2022 11:06
@ek2061
Copy link
Collaborator

ek2061 commented Oct 9, 2022

Files changed 咋那麼多? 還有 CI 沒過

@jumpjet0524
Copy link
Collaborator Author

應該過了 我也不知道為啥那麽多 點進去看起來都沒改

@ek2061
Copy link
Collaborator

ek2061 commented Oct 9, 2022

要找出原因

@jumpjet0524
Copy link
Collaborator Author

我目前查到是換行符的問題,windows是CRLF但是我是用mac改的是LF格式,顯示有被改過的文件確實大小不一樣,大概就差’行數‘個Bytes

@jumpjet0524
Copy link
Collaborator Author

搞定 原因是我當時在學校做一半想帶回家做直接src整包打包帶走(CRLF格式),一般win是CRLF格式在git 的時候會自動轉LF,但是我在推的時候不知道為什麼沒轉。

@jumpjet0524 jumpjet0524 self-assigned this Oct 27, 2022
src/pages/pageA.jsx Show resolved Hide resolved
src/components/MyCarousel.css Outdated Show resolved Hide resolved
src/components/ObjectCard.jsx Outdated Show resolved Hide resolved
src/components/ObjectCard.jsx Outdated Show resolved Hide resolved
src/components/MyCarousel.jsx Outdated Show resolved Hide resolved
MyCarousel.propTypes = {
shownums: PropTypes.number,
views: PropTypes.array,
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

我注意到有傳scroll進去但沒有加檢查

);

ObjectCard.propTypes = {
width: PropTypes.number,
Copy link
Collaborator

Choose a reason for hiding this comment

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

這個width沒用到,而且他也不是number

@ek2061
Copy link
Collaborator

ek2061 commented Nov 3, 2022

喔對了再自己檢查一下每個 propTypes,
雖然目前有做型別檢查,但有些props應該是必要值,
那就必須要加上必填isRequired或者設預設值defaultProps

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.

None yet

2 participants