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

React-swipeable-viewsからSwiperへのリプレイス #128

Merged
merged 15 commits into from
Mar 2, 2023

Conversation

Nikudanngo
Copy link
Collaborator

背景

react-swipeable-viewsがReact18で非推奨になっており、npm iのたびに-fオプションをつけないとまともに動かない状況だった。
そのため、代わりにSwiperに乗り換えて、ついでにreact-swipeable-viewsをnpmから削除してその他の依存関係をアップデートしたい

変更

  • react-swipeable-viewsの削除
  • いくつかの脆弱性のあった依存ライブラリをアップデート
  • TailwindCSSのプラグインの導入
  • npm run lint,npm run fixコマンドを追加 : 警告と修正するコマンド
  • スワイプをSwiperで実装
  • useRef,useMemoでパフォーマンス向上?(あんまり変わらん)

@github-actions
Copy link

github-actions bot commented Feb 28, 2023

Visit the preview URL for this PR (updated for commit 8e2c5e3):

https://mobile-order-4d383--pr128-replace-swiper-j6jkwt9v.web.app

(expires Tue, 07 Mar 2023 07:37:06 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d05948d1c21d3195428590dd6f13a9369eb73441

Copy link
Owner

@charokoukuu charokoukuu left a comment

Choose a reason for hiding this comment

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

実装ありがとう!LGTMです👍

src/component/SwipeTabs.tsx Show resolved Hide resolved
src/component/SwipeTabs.tsx Show resolved Hide resolved
@Nikudanngo Nikudanngo merged commit eae4bdc into main Mar 2, 2023
Nikudanngo added a commit that referenced this pull request Mar 2, 2023
* delete react-swipeable-views

* いくつかの依存関係のバージョンアップ

* prettier-plugin-tailwindcssの追加

* npm i eslint-plugin-tailwindcss

* tailwindフォーマットプラグイン導入

* npm コマンド追加

- npm run lint : eslintの警告を表示
- npm run fix : 自動で色々直してくれる

* prettierによる自動整形とtailwindのwarning解消

* デグレ修正

* npm i swiper

* react-swipeable-viewsからSwiperへのリプレイス

- 処理のメモ化

* git hub actionsの-fオプション外した

* Menuにmax-wを追加

* delete comment

* FoodCartを左寄せ

* FoodCardのSOLD OUTのカラー修正
Nikudanngo added a commit that referenced this pull request Mar 2, 2023
* delete react-swipeable-views

* いくつかの依存関係のバージョンアップ

* prettier-plugin-tailwindcssの追加

* npm i eslint-plugin-tailwindcss

* tailwindフォーマットプラグイン導入

* npm コマンド追加

- npm run lint : eslintの警告を表示
- npm run fix : 自動で色々直してくれる

* prettierによる自動整形とtailwindのwarning解消

* デグレ修正

* npm i swiper

* react-swipeable-viewsからSwiperへのリプレイス

- 処理のメモ化

* git hub actionsの-fオプション外した

* Menuにmax-wを追加

* delete comment

* FoodCartを左寄せ

* FoodCardのSOLD OUTのカラー修正
Nikudanngo added a commit that referenced this pull request Mar 2, 2023
* delete react-swipeable-views

* いくつかの依存関係のバージョンアップ

* prettier-plugin-tailwindcssの追加

* npm i eslint-plugin-tailwindcss

* tailwindフォーマットプラグイン導入

* npm コマンド追加

- npm run lint : eslintの警告を表示
- npm run fix : 自動で色々直してくれる

* prettierによる自動整形とtailwindのwarning解消

* デグレ修正

* npm i swiper

* react-swipeable-viewsからSwiperへのリプレイス

- 処理のメモ化

* git hub actionsの-fオプション外した

* Menuにmax-wを追加

* delete comment

* FoodCartを左寄せ

* FoodCardのSOLD OUTのカラー修正
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