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.

Sign up for GitHub

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

Jump to bottom

new dashboard #747

Merged
merged 282 commits into from Dec 8, 2023
Merged

new dashboard #747

merged 282 commits into from Dec 8, 2023

Conversation

eyemono-moe
Copy link
Contributor

@eyemono-moe eyemono-moe commented Oct 4, 2023 •
edited

preview→https://747.ns-preview.trapti.tech/

なぜやるか

新UIへの置き換え

やったこと

後で追記

やらなかったこと

後で追記

資料

@motoki317
Copy link
Member

ゴリっとできてて草

svg iconをどこから持ってきたのか、再現性的にもcopyright的にも書いたほうが良いかも

@eyemono-moe
✨ 環境変数設定画面
63ce7e0
@eyemono-moe
🐛 デプロイされてない場合の表示が誤っていた
f318012
@eyemono-moe
🩹 最新ビルドのリフェッチを中断しないように
1f3ab42 
#747 (comment)
@eyemono-moe
🐛 アイコンのサイズ指定が誤っていた
cdcf19c
@eyemono-moe
✨ ツールチップの追加
bd8cccb
@eyemono-moe
💄 スタイル崩れの修正
2c1b90c 
- show moreボタンをセンターに
- リスト内のボタンをセンターに
@eyemono-moe
🔥 remove unused imports
85c685d
@eyemono-moe
✨ アプリ情報ページに設定への遷移ボタンを追加
d7bba81
@eyemono-moe
💄 リポジトリ/アプリ名が長い場合に省略して表示するように
957b91f
@eyemono-moe
♻️ ヘッダー/ナビ用コンポーネントの共通化
7b23bb6
@eyemono-moe
💄 長いアプリ名/リポジトリ名の省略表示
638753a
@eyemono-moe
🩹 Static状態アイコンの色の修正
1358008
@eyemono-moe
🩹 Navのタイトルの幅を修正
bb593d2
@eyemono-moe
🩹 URL数が改行されないように
8710068
@eyemono-moe
💄 Selectの最小幅を修正
b475b27
@eyemono-moe
💄 add scrollbar-gutter: stable
4e4a622
@eyemono-moe
✨ add open Adminer button
7590148
@eyemono-moe
✨ ユーザーボタンの追加
1ab6431
@eyemono-moe
✨ add provider icon in repository list
312247f
@eyemono-moe
✨ add tooltip in app/build status icon
d14d666
@eyemono-moe
💄 モバイル端末でのパディングを小さく
979f16e
@eyemono-moe
✨ add builds page
a57f690
@eyemono-moe
💄 app/repository ownerの説明を追加
0b4e5b9
eyemono-moe and others added 7 commits December 8, 2023 04:46
@eyemono-moe
💄 fix queue page background color
bb12858
@eyemono-moe
🔥 build detailsからrepository nameを削除
0e6c99c
@eyemono-moe
💄 create appでのprogressのアイコンを削除/step No.を追加
2b0d3fe 
左から右に順に進むことが初見でわからなかったので数字を追加してみた
邪魔なら消す
@eyemono-moe
🐛 use dbの値が適切に更新されていなかった
3407ad4
@eyemono-moe
💄 Use Databaseが変更できない旨を表示
b05f2b2 
Figmaのデザインを元に作成
@eyemono-moe
💄 高度な設定をまとめる
86c6058 
ネストしたフォームのデザインが無かったのでもうちょっとデザイン考える必要アリ
@motoki317
fix: set apps-include-no-app default to false
d3c61b5
@motoki317
Copy link
Member

motoki317 commented Dec 8, 2023 •
edited

少し検討が必要そうなフィードバックまとめ（適宜後に回す）

  • アプリ登録時の「Use Database」は後から変更できないという点で、他の設定と大きく異なるので、それを強調したい（現在は"?"のtooltipでしかわからない）
    • どう強調するかは考える
  • 全体的に日本語で、単語を統一する
    • e.g. アプリ登録画面step 3/3で「アクセスURLの設定」と「No Websites Configured」「Add Website」と、同じ概念なのに違う単語を使っている → これは「アクセスURL」と統一？（内部で使ってはいるが「Website」は正直違和感がある）
    • e.g. アプリ設定画面で「No Port Forwarding Configured」など長い英単語列を使われると違和感を覚える → これは「ポートフォワード設定」などと統一？
    • e.g. アプリ設定 Is SPA「Yes」「No」と英語があるすぐ下に「今すぐ起動する」という日本語があったりする 日本語で統一？
  • ビルド詳細画面 「Succeeded」などのステータスが一番見たい情報なはずなので、/app/:id のDeploymentの表示のようにもっとデカく強調してよい
    • ここだけ背景色が違うのも少し違和感
  • ビルド詳細画面 時刻が3つとビルド時間1つがそのまま置いてあって、分かりづらいのでどうにかしたい
    • タイムライン的表示？

@motoki317
Copy link
Member

残りは後回しにするか

@motoki317 motoki317 marked this pull request as ready for review December 8, 2023 06:17
@motoki317 motoki317 changed the title WIP: new dashboard new dashboard Dec 8, 2023
@motoki317 motoki317 merged commit 969dc37 into main Dec 8, 2023
15 checks passed
@motoki317 motoki317 deleted the feature/new-dashboard branch December 8, 2023 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@motoki317 motoki317 motoki317 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

UI Parts 作成 Atom Component 作成
2 participants
@eyemono-moe @motoki317