misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.95k stars 1.35k forks source link

feat: アイコンデコレーションをアイコンの背後に表示できるように #14708

Open kakkokari-gtyih opened 1 week ago

kakkokari-gtyih commented 1 week ago

What

Why

Fix #14707

Additional info (optional)

Checklist

syuilo commented 1 week ago

below というより behind な気がするわね

kakkokari-gtyih commented 1 week ago

below というより behind な気がするわね

たしかに

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 37.83784% with 23 lines in your changes missing coverage. Please review.

Project coverage is 41.29%. Comparing base (043fef9) to head (58856b2).

Files with missing lines Patch % Lines
...nd/src/pages/settings/avatar-decoration.dialog.vue 0.00% 13 Missing :warning:
...ckages/frontend/src/components/global/MkAvatar.vue 50.00% 3 Missing :warning:
.../frontend/src/pages/settings/avatar-decoration.vue 0.00% 3 Missing :warning:
...rc/pages/settings/avatar-decoration.decoration.vue 0.00% 2 Missing :warning:
...ges/backend/src/core/entities/UserEntityService.ts 0.00% 1 Missing :warning:
...kages/backend/src/server/api/endpoints/i/update.ts 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #14708 +/- ## =========================================== + Coverage 39.68% 41.29% +1.61% =========================================== Files 1549 1553 +4 Lines 194593 200358 +5765 Branches 3603 3624 +21 =========================================== + Hits 77223 82744 +5521 - Misses 116771 117013 +242 - Partials 599 601 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 1 week ago

このPRによるapi.jsonの差分

差分はこちら ```diff --- base +++ head @@ -49834,6 +49834,12 @@ ], "maximum": 0.25, "minimum": -0.25 + }, + "showBehind": { + "type": [ + "boolean", + "null" + ] } }, "required": [ @@ -76725,6 +76731,9 @@ }, "offsetY": { "type": "number" + }, + "showBehind": { + "type": "boolean" } }, "required": [ ``` [Get diff files from Workflow Page](https://github.com/misskey-dev/misskey/actions/runs/11191143600)
kakkokari-gtyih commented 1 week ago

アイコンの重ね順を自由に設定できるようになった場合、このアプローチだとまずいかもしれない(今は重ね順=配列順になっているけど、このフラグの有無でその順番が崩れてややこしくなる可能性がある)

kakkokari-gtyih commented 1 week ago

理想はアイコンも含めた重ね順を指定できるようになることかも(アイコンを0として正負で指定するとか)

syuilo commented 1 week ago

このフラグの有無でその順番が崩れるというのがよくわからんわね

syuilo commented 1 week ago

配列をフラグの有無でグルーピングした後の並び順で良いんじゃないかしら

syuilo commented 1 week ago

実態はひとつの配列だけどフラグがオンのデコレーションの配列とフラグがオフのデコレーションの配列の二つであると解釈する

kakkokari-gtyih commented 1 week ago

実態はひとつの配列だけどフラグがオンのデコレーションの配列とフラグがオフのデコレーションの配列の二つであると解釈する

これをするのが面倒そうなので、orderとかのプロパティを生やして実際のアイコン画像を0とした正負の値を設定して明示的に順番を指定したほうがいいんじゃないかと考えた

syuilo commented 1 week ago

設定UI上でその情報を表現するのが難しそう

syuilo commented 1 week ago

実態はひとつの配列だけどフラグがオンのデコレーションの配列とフラグがオフのデコレーションの配列の二つであると解釈する

これをするのが面倒そう

filterすれば良いだけじゃないかしら

CenTdemeern1 commented 18 hours ago

I was originally planning on upstreaming this myself, but I'm glad to see you're already doing it ^^ let me know if you need any help