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

enhance(frontend): WidgetInstanceInfo.vue と WidgetProfile.vue のスタイル調整 #14028

Merged
merged 5 commits into from
Jun 22, 2024

Conversation

nullnyat
Copy link
Contributor

@nullnyat nullnyat commented Jun 16, 2024

What

ウィジェットのプロフィールとサーバー情報のドロップシャドウが見切れているのが改善される

Why

見た目があまり良くない

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jun 16, 2024
@syuilo
Copy link
Member

syuilo commented Jun 16, 2024

afterのスクショがあると助かるかも

Copy link

codecov bot commented Jun 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.84%. Comparing base (379ce01) to head (0758aa9).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #14028      +/-   ##
===========================================
+ Coverage    77.79%   77.84%   +0.05%     
===========================================
  Files          184      185       +1     
  Lines        25536    25601      +65     
  Branches       487      487              
===========================================
+ Hits         19865    19930      +65     
  Misses        5664     5664              
  Partials         7        7              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nullnyat
Copy link
Contributor Author

びふぉー

image

あふたー

image

です。

@syuilo
Copy link
Member

syuilo commented Jun 16, 2024

overflow無くすと名前が長い時にはみ出すようにならないかしら?

@nullnyat
Copy link
Contributor Author

なんかそこは文字ごとはみ出るからいいかなって気持ちになってるかも、、?

 2024-06-17 8 28 20  2024-06-17 8 30 50

@nullnyat
Copy link
Contributor Author

枠からシャドーがはみ出るってことはないかも

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jun 17, 2024

overflow: clip; はそのままにしといて、

.body {
	text-overflow: ellipsis;
	overflow: clip;
	margin-left: -5px;
	padding-left: 5px;
	width: calc(100% + 5px);
}

とかにするといいかも

@nullnyat
Copy link
Contributor Author

確かに、そっちの方が良さそう

@kakkokari-gtyih kakkokari-gtyih changed the title 🎨 WidgetInstanceInfo.vue and WidgetProfile.vue enhance(frontend): WidgetInstanceInfo.vue と WidgetProfile.vue のスタイル調整 Jun 17, 2024
@nullnyat
Copy link
Contributor Author

 2024-06-17 20 21 21

よくなってきた☺️

@nullnyat
Copy link
Contributor Author

色、いい案が出なかったのでそのままになってます。

 2024-06-18 18 09 13  2024-06-18 18 08 05

どうでしょう。。。

@syuilo
Copy link
Member

syuilo commented Jun 18, 2024

drop shadowではなくtext shadowに変更した理由ってあるかしら

@syuilo
Copy link
Member

syuilo commented Jun 18, 2024

text-shadowだと文字通りテキストにしか適用されないからカスタム絵文字が浮きそう

@nullnyat
Copy link
Contributor Author

drop shadowはデザイン的にビミョいって思っただけなので、特に特別な理由があってというわけではないですね。

カスタム絵文字のこと考えてませんでした。。。

ここが薄すぎるのがなぁと思っていただけなので、drop shadowに変更します🙏

 2024-06-18 18 19 05

@kakkokari-gtyih
Copy link
Contributor

drop shadowを重ね掛けすれば薄さ改善できるはず

@nullnyat
Copy link
Contributor Author

nullnyat commented Jun 18, 2024

ふむ。。。

アクセントカラーをつけて見た。どうだろう。。。(色がうるさすぎる可能性がありそうって思ってる)

 2024-06-19 1 51 24  2024-06-19 1 51 13  2024-06-19 1 50 59

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jun 19, 2024

普通に黒シャドウを複数回かけるのでよさそう
たとえば

filter: drop-shadow(0 0 3px #000) drop-shadow(0px 0px 6px rgba(0, 0, 0, .5));

image

@nullnyat
Copy link
Contributor Author

nullnyat commented Jun 19, 2024

こんな感じになりました。

 2024-06-19 21 35 47

@nullnyat
Copy link
Contributor Author

ユーザーページも見切れてしまっているのに気づいてしまったのでひっそり直しました。

ビフォーアフターです。

 2024-06-19 21 31 09  2024-06-19 21 52 21

@syuilo syuilo merged commit ef205fb into misskey-dev:develop Jun 22, 2024
17 checks passed
@syuilo
Copy link
Member

syuilo commented Jun 22, 2024

👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
3 participants