Skip to content

Conversation

@arkwnet
Copy link
Contributor

@arkwnet arkwnet commented Apr 5, 2025

やったこと

  • 現在のデザインがChakra UIを想定しておらず、レイアウト面でかなり無茶をしているため、SCSSで全て書き直しました
    • モーダルとトーストを出す処理にChakra UIが残っており、ライブラリ自体の削除はしていません
  • TODOを1件 (Result画面からスコア表示を削除) 対応しました
  • 権利的にクリーンなユーザアイコンを作成しました
    • しずっぴーをOSSに使うと、しずっぴーだけ別ライセンスになってしまいます (https://github.com/xr-hamaria/virtual-inf を参照)
    • また、格納場所が静大のウェブサイトからローカルに移り、全アセットをGitHub管理できるようになりました

やらないこと

見て分かるレベルのデザイン変更

できるようになること(ユーザ目線)

無し

できなくなること(ユーザ目線)

無し

動作確認

全機能を目視で確認していますが、レビューされる方はお手元の環境でダブルチェックをお願いします。

その他

恐らくToastが機能していません。動作に支障はありませんが、他のデザイン調整と併せて別プルリクで改修予定です。

@arkwnet arkwnet added Problem: Design デザインに関する問題がある Status: Confirm Needed 確認が必要 Scope: Frontend javascript Pull requests that update Javascript code labels Apr 5, 2025
@arkwnet arkwnet self-assigned this Apr 5, 2025
araaki12345
araaki12345 previously approved these changes Apr 5, 2025
Copy link
Collaborator

@h-takeyeah h-takeyeah left a comment

Choose a reason for hiding this comment

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

変更お願いします

@arkwnet arkwnet merged commit b4d3d8e into develop Apr 8, 2025
2 checks passed
@h-takeyeah h-takeyeah deleted the feat/frontend/no-chakra branch April 8, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update Javascript code Problem: Design デザインに関する問題がある Scope: Frontend Status: Confirm Needed 確認が必要

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants