Skip to content

Conversation

totocalcio
Copy link
Contributor

@totocalcio totocalcio commented Jul 21, 2024

issue

https://github.com/vuejs-jp/vuefes-2024-backside/issues/290

Design

https://www.figma.com/design/g0oSNTSYpa0byRxVv00Ugm/Vue-Fes-Japan-2024?node-id=4671-7411&t=09hXo4SApUtKEB6Y-4

Storybook

http://localhost:6006/?path=/story/namecard-namecard24--default&globals=viewport:responsive

Details of Changes

  • 基本的なI/Fは2023と同じ
    • 名前の文字色を判断するisPlaceholderを追加している
    • isPreviewなのかisViewNameなのか、適切なプロパティ名について検討の余地はある

Screenshots

image
image

Copy link

netlify bot commented Jul 21, 2024

Deploy Preview for vuefes-2024 ready!

Name Link
🔨 Latest commit 387f30e
🔍 Latest deploy log https://app.netlify.com/sites/vuefes-2024/deploys/669d366357202b000879a81b
😎 Deploy Preview https://deploy-preview-213--vuefes-2024.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@jiyuujin jiyuujin left a comment

Choose a reason for hiding this comment

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

あれ、packages/ui 配下では作成されなっかったんですね
OGP については Namecard24 と同じ場所に作成いただければ(ベース未作成ですが

https://github.com/vuejs-jp/vuefes-2024/blob/main/packages/ui/components/namecard/Namecard24.vue

@totocalcio
Copy link
Contributor Author

totocalcio commented Jul 21, 2024

slackでapps/web 範囲外でも良いですがと返信頂いたので、もともとの想定はapps/webだったのかなと思ったことと、レスポンシブ対応でapps/web/app/assets/media.cssを使用したかったので、apps/webに作成致しました。
一旦packages/uiに移植します。

@jiyuujin
Copy link
Collaborator

jiyuujin commented Jul 21, 2024

すみません、私の言葉足らずでした mm

基本 VFNamecard23 から VFNamecard24 に変更するだけではあるので
実際、作業するにあたっては apps/web への組込は範囲外でも良いですよという意図がありました

以下の実装は範囲外(apps/web)
Namecardの閲覧画面、編集画面、完了画面、シェア画面の実装

=====

コンポーネント作成自体は apps/web で作成しようという意図はありませんので急ぎ

@jiyuujin
Copy link
Collaborator

jiyuujin commented Jul 21, 2024

@totocalcio cc: @KannoStanfoot

先ころ main ブランチにマージさせていただいたんで、apps/web への組込も対応いただけます mm
( PR とは別で進めていただいても OK ですので

https://github.com/vuejs-jp/vuefes-2024/pull/212/files#r1685705024

Copy link

cloudflare-workers-and-pages bot commented Jul 21, 2024

Deploying vuefes-2024-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7447072
Status: ✅  Deploy successful!
Preview URL: https://162e9ac7.vuefes-2024-docs.pages.dev
Branch Preview URL: https://feat-namecard.vuefes-2024-docs.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Jul 21, 2024

Deploying vuefes-2024-designsystem with  Cloudflare Pages  Cloudflare Pages

Latest commit: 387f30e
Status: ✅  Deploy successful!
Preview URL: https://ce89f28d.vuefes-2024-designsystem.pages.dev
Branch Preview URL: https://feat-namecard.vuefes-2024-designsystem.pages.dev

View logs

@totocalcio totocalcio requested a review from jiyuujin July 21, 2024 13:21
<div class="avatar">
<span class="avatar-hook" aria-hidden="true" />
<div class="vuefes-logo-wrapper">
<img src="../../assets/namecard/vuefes_logo.svg" alt="vuefes logo" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

NITS

SVG アイコンの箇所、表示されていなさそうな気がします

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Storybook上では問題ありませんでしたが、apps/web側に埋め込んだら表示されないことを確認しました。
調査・対応します。

@jiyuujin jiyuujin merged commit cdf0603 into main Jul 21, 2024
9 of 10 checks passed
@jiyuujin jiyuujin deleted the feat/namecard branch July 21, 2024 22:55
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