サイトのデザインと構成を変更 #123

Myxogastria0808 · 2023-10-25T10:45:37Z

vercel · 2023-10-25T10:45:43Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
qr-frontend	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 26, 2023 5:45pm

puripuri2100

リファクタリングとデザインのモダン化の作業お疲れさまでした
かなり良く、ありがとうございます
一部あった修正してほしい箇所をコメントしました
対応よろしくお願いします @Myxogastria0808

puripuri2100 · 2023-10-25T17:18:40Z

next.config.js

@@ -2,6 +2,10 @@
 const nextConfig = {
  reactStrictMode: true,
  swcMinify: true,
+  //以下の記述は、安定してstyleのstyleを充てるのに必要なもの


styleのstyle

「styled-componentsのstyle」の間違いかな？

書き間違えました...
書き直しておきます。

puripuri2100 · 2023-10-25T17:19:41Z

src/components/Header/index.tsx

@@ -24,12 +26,19 @@ const DrawerHeader = styled('div')(({ theme }) => ({

 const Header = () => {
  const [menuOpen, setMenuOpen] = useState(false)
+  //!変更したよ~~~~~~~~


gitでログとdiffが見られるのでコメントは削除して大丈夫

コメント消し忘れてました...
消しておきます。

puripuri2100 · 2023-10-25T17:19:55Z

src/components/Header/index.tsx

@@ -24,12 +26,19 @@ const DrawerHeader = styled('div')(({ theme }) => ({

 const Header = () => {
  const [menuOpen, setMenuOpen] = useState(false)
+  //!変更したよ~~~~~~~~
+  const router = useRouter()
+  //!変更したよ~~~~~~~~


コメント削除

コメント消し忘れてました...
消しておきます。

puripuri2100 · 2023-10-25T17:33:38Z

src/components/SpotList/index.tsx

+          //! 変更 (Myxogastria0808)
+          window.location.href = '/spot'
+          //! 変更 (Myxogastria0808)
        } catch (err) {
          toast.error('削除に失敗')
+          //! 変更 (Myxogastria0808)
+          window.location.href = '/spot'
+          //! 変更 (Myxogastria0808)


コメントは削除して大丈夫
その代わりリダイレクトをかけないといけない理由が書いてあると良い

router.replace(`/spot`)

ではなく

window.location.href = '/spot'

になっているのは何か理由がある…？揃えられるなら他と揃えておくと良いと思います

同一パスにリダイレクトするとき(上記のパターンでは /spot から /spotにリダイレクト)、router.replace()は、リダイレクトがかからないのですが、window.location.hrefだとリダイレクトがかけれるのでwindow.location.hrefを採用しています。
リダイレクトをかけている理由は、削除のボタンを押したときに、リダイレクトをしないと更新されないからです。
コメントは削除しておきます。

puripuri2100 · 2023-10-25T17:36:08Z

src/components/SystemButton/index.tsx

SystemButton/Button.stories.tsxファイルについて、ファイル名をSystemButton/SystemButton.stories.tsxに変更し、中身も合わせて書き替えるようにしてください

storybooksを使ったことがないので、ファイル内の変更箇所がわかりません...
申し訳ございません。
ファイル名だけ変更しました。

了解です
自分で巻き取っておきます

puripuri2100 · 2023-10-25T17:46:14Z

src/pages/items/[qr_id].tsx

+                      //! 変更 (Myxogastria0808)
+                      router.replace('/')
+                      //! 変更 (Myxogastria0808)


コメントは削除して大丈夫
トップページにリダイレクトをかける理由（「物品検索ページに誘導するため」など）を書いておくと良いと思う

アドバイスありがとうございます。
コメント削除しておきます。

puripuri2100 · 2023-10-25T17:52:03Z

src/pages/register-item.tsx

+    <>
+      <Header />
+      <Top />
+      <PrintingPage />


ページはregister-itemだけれどもここが<PrintingPage />になっているのはやや混乱があるので、どちらかに統一した方が良さそうです
個人的には、QRコード印刷よりも物品登録の方が主なので<RegisterItemPage />の方が良いと思います

<RegisterItemPage />に変更しておきました。

puripuri2100 · 2023-10-25T17:52:57Z

src/pages/spot/info_edit.tsx

+          //! 変更 (Myxogastria0808)
+          router.replace(`/spot`)
+          //! 変更 (Myxogastria0808)
          return result
        } catch (err) {
          toast.error('地点情報の編集に失敗')
-          router.replace(`/spot/list`)
+          //! 変更 (Myxogastria0808)
+          router.replace(`/spot`)
+          //! 変更 (Myxogastria0808)


コメントは削除して大丈夫

コメント削除しておきます。

puripuri2100 · 2023-10-25T17:54:36Z

src/pages/spot/register.tsx

@@ -36,7 +36,7 @@ const StyledMain = styled.main.withConfig({
 /**
 * 物品を登録できる
 */
-const FixturesRegister = () => {
+const SpotFixturesRegister = () => {


SpotRegisterで良いと思う

SpotRegisterに変更しました。

puripuri2100 · 2023-10-25T17:54:57Z

src/pages/spot/register.tsx

+          //! 変更 (Myxogastria0808)
+          window.location.href = '/spot'
+          //! 変更 (Myxogastria0808)
          return result
        } catch (err) {
          toast.error('地点の登録に失敗')
+          //! 変更 (Myxogastria0808)
+          window.location.href = '/spot'
+          //! 変更 (Myxogastria0808)


コメントは削除して大丈夫

router.replace(`/spot`)

に揃えた方が良いと思う

登録情報を地点情報一覧に反映するために必要なので、リダイレクトかけています。
同一パスにリダイレクトするとき(上記のパターンでは /spot から /spotにリダイレクト)、router.replace()は、リダイレクトがかからないのですが、window.location.hrefだとリダイレクトがかけれるのでwindow.location.hrefを採用しています。

Myxogastria0808 · 2023-10-26T09:45:35Z

一部、デザインの修正が残っていた箇所がありましたので、その修正が完了次第、pushします。

puripuri2100 · 2023-10-26T17:46:48Z

mainをmergeして良さそうなのでmergeします
深夜なので影響無さそうだし

Myxogastria0808 added 8 commits October 21, 2023 21:10

first commit

19f2048

second commit

6e58627

third commit

f102d3b

firth commit

fa154b9

fifth commit

f40f8f8

sohosai/qr-backend#119 に追従の差分追加

6875d7e

fix routing bug

96889d3

fix design

772b936

Myxogastria0808 requested a review from momeemt as a code owner October 25, 2023 10:45

puripuri2100 self-requested a review October 25, 2023 16:12

puripuri2100 changed the title ~~fix/#109-edit-site-design~~ サイトのデザインと構成を変更 Oct 25, 2023

puripuri2100 reviewed Oct 25, 2023

View reviewed changes

fix pull request

efb513e

vercel bot deployed to Preview October 26, 2023 15:19 View deployment

fix pull request 2

e618fe9

vercel bot deployed to Preview October 26, 2023 17:07 View deployment

puripuri2100 added 2 commits October 27, 2023 02:42

merge main

1305e06

yarn format

ec6056b

vercel bot deployed to Preview October 26, 2023 17:45 View deployment

puripuri2100 merged commit c12f8fd into main Oct 26, 2023

This was referenced Oct 26, 2023

デザインを綺麗にする #109

Closed

リファクタリング #110

Closed

CHANGELOGをアップデートする #132

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

サイトのデザインと構成を変更 #123

サイトのデザインと構成を変更 #123

Myxogastria0808 commented Oct 25, 2023 •

edited by puripuri2100

Loading

vercel bot commented Oct 25, 2023 •

edited

Loading

puripuri2100 left a comment •

edited

Loading

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023 •

edited

Loading

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

puripuri2100 Oct 25, 2023

Myxogastria0808 Oct 26, 2023

Myxogastria0808 commented Oct 26, 2023

puripuri2100 commented Oct 26, 2023

サイトのデザインと構成を変更 #123

サイトのデザインと構成を変更 #123

Conversation

Myxogastria0808 commented Oct 25, 2023 • edited by puripuri2100 Loading

vercel bot commented Oct 25, 2023 • edited Loading

puripuri2100 left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Myxogastria0808 Oct 26, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Myxogastria0808 commented Oct 26, 2023

puripuri2100 commented Oct 26, 2023

Myxogastria0808 commented Oct 25, 2023 •

edited by puripuri2100

Loading

vercel bot commented Oct 25, 2023 •

edited

Loading

puripuri2100 left a comment •

edited

Loading

Myxogastria0808 Oct 26, 2023 •

edited

Loading