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

Feature v1 auth UI #702

Merged
merged 26 commits into from
Oct 29, 2023
Merged

Feature v1 auth UI #702

merged 26 commits into from
Oct 29, 2023

Conversation

studiokaiji
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Oct 10, 2023

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

Name Status Preview Comments Updated (UTC)
nostter ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 29, 2023 0:02am

@@ -35,7 +35,7 @@
}

async function loginWithDemo() {
await goto('/home');
location.pathname = '/home';
Copy link
Owner

Choose a reason for hiding this comment

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

ここ変更したのって何か意図がありますか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

gotoだと遷移前の要素のCSSが適応されてしまいglobalでhidden化してるheader等が、遷移後の画面でもhiddenされてしまうのでこうしています

Copy link
Owner

Choose a reason for hiding this comment

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

了解です。
goto を使わないとサイトの初期化処理が再度走ってしまうのでそのあたり回避できないか調べてみます~

Copy link
Owner

@SnowCait SnowCait Oct 17, 2023

Choose a reason for hiding this comment

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

+layout.svelte の中に if 文を書いて CSS を :global()!improtant を使わずに書いた方が素直な気がしてます。
書く方法は模索中(URL で判定?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

SvelteのLayoutの仕様を見るとスコープ範囲?のようなものがあるようなので(ちゃんと理解できていない)、内部でURLごとの切り分けを書くのはコードが複雑になるかなあと思って!importで指定してました。
CSS in JSで書いちゃえば一応それっぽくは作れると思うんですが、できればトップページとそれ以外のページでlayoutは変えられるのが理想だよなーと思っています…

Comment on lines 70 to 72
if ($authorProfile) {
gotoHome();
}
Copy link
Owner

Choose a reason for hiding this comment

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

ログインしてる間はランディングページにアクセスさせない方が都合いいですかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LP、認証画面ともにできるだけちらつきをなくすためただの背景色を表示するだけが望ましいと思ってます

Copy link
Owner

Choose a reason for hiding this comment

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

"login": "ログイン"
"login": "ログイン",
"hero-message": "みんなのための、自由なSNS。",
"hero-message-sub": "突然投稿ができなくなることも、アカウントが消えることもない。必要なのは「鍵」ひとつだけ。"
Copy link
Owner

Choose a reason for hiding this comment

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

突然投稿ができなくなることも、アカウントが消えることもない。

やや意味が被ってそうなのでまとめてしまいたいかもです。

必要なのは「鍵」ひとつだけ。

個人的には言い回し好きなんですが、鍵というワードを表に出すと全体的にもう少し npub/nsec を表に出さないと整合性取れなさそうなので出来ればメールアドレス不要のように言い換えたいです。

どういう文言にするのが良さそうか検討中。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

まとめてもいいんですけど、キャッチコピーとしては短くなるので、どうしようかなという感じです。
文言周りはまたどこかで決めましょう。

Copy link
Owner

Choose a reason for hiding this comment

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

ネガティブ否定系(凍結がない、メアド不要)だけじゃなくてポジティブ系も入れたいかも。
絵文字が使えるとか使い慣れたデザインであるとか。

キャッチコピーとしてそれっぽくできるかは分からないけど…

Copy link
Owner

Choose a reason for hiding this comment

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

かっこよさは減るけど「凍結のないオープンな環境で絵文字キッチンを楽しもう。」とかでもいいのかなぁ。

@@ -15,13 +15,13 @@
"recommended": "推奨",
"key": "鍵でログイン",
"create_account": "アカウントを作成",
"create": "作成",
"name": "名前",
"try_demo": "デモを試す",
Copy link
Owner

Choose a reason for hiding this comment

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

ここの文言変更したいなと思ってるんですがどうでしょうか?(分かりやすさとデザイン面)

Suggested change
"try_demo": "デモを試す",
"try_demo": "登録せずに見る",

英語だと Try without register とかかな。
アカウント作成ダイアログ内のボタンも register, 登録 に。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

試してみるとか、ログインせずに見るとかいろいろ考えられますよねー…

Copy link
Owner

@SnowCait SnowCait Oct 12, 2023

Choose a reason for hiding this comment

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

そうそう。「デモ」っていうとなんか少し堅苦しい(?)感じがするからもうちょっと気軽に中を覗けるよーって感じの文言にした方がいいかなと思って。

web/src/routes/Login.svelte Outdated Show resolved Hide resolved
@SnowCait SnowCait merged commit fb9814d into SnowCait:main Oct 29, 2023
4 checks passed
@SnowCait SnowCait mentioned this pull request Oct 29, 2023
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