misskey icon indicating copy to clipboard operation
misskey copied to clipboard

enhance(frontend): ノートの公開範囲に応じて色分けできるように

Open kakkokari-gtyih opened this issue 1 year ago • 9 comments

What

MkNoteのみ色分けするように (MkNoteDetailedは、本来のノートを見たい際にMFMと干渉して不便な可能性があるのであえてそのままにした)

色はテーマファイル内で指定可能

image

Why

Fix #14784

Additional info (optional)

ノートに背景色を指定したらMkDateSeparatedListのボーダーが表示されない箇所が発生した(ブラウザ側でズームすると表示される、謎)

image

Checklist

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

kakkokari-gtyih avatar Oct 20 '24 05:10 kakkokari-gtyih

Codecov Report

Attention: Patch coverage is 50.00000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 17.51%. Comparing base (a5fa9a2) to head (80cc233).

Files with missing lines Patch % Lines
...ckages/frontend/src/pages/settings/preferences.vue 0.00% 10 Missing :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14800       +/-   ##
============================================
- Coverage    48.22%   17.51%   -30.72%     
============================================
  Files         1700      817      -883     
  Lines       178497    73654   -104843     
  Branches      5376     1334     -4042     
============================================
- Hits         86089    12901    -73188     
+ Misses       91950    60314    -31636     
+ Partials       458      439       -19     

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Oct 20 '24 05:10 codecov[bot]

ノートに背景色を指定したらMkDateSeparatedListのボーダーが表示されない箇所が発生した(ブラウザ側でズームすると表示される、謎)

太さが.5pxになってるからだと思われる

tamaina avatar Oct 20 '24 09:10 tamaina

ノートに背景色を指定したらMkDateSeparatedListのボーダーが表示されない箇所が発生した(ブラウザ側でズームすると表示される、謎)

太さが.5pxになってるからだと思われる

1pxにしてもだめだった(なんか要素がお互いにめり込んでる気がする)

kakkokari-gtyih avatar Oct 20 '24 09:10 kakkokari-gtyih

box-sizingあたりの話かも?

tamaina avatar Oct 20 '24 09:10 tamaina

box-sizing自信ないからなしで

→ 関係ないってかっこかりさんがMisskeyで言ってた

tamaina avatar Oct 20 '24 09:10 tamaina

どのブラウザでも共通して発生するのかしら

tamaina avatar Oct 20 '24 09:10 tamaina

ブラウザ側のズームをすると表示されるのでおま環 or レンダリングバグかも Google Chrome 129.0.6668.101 (Official Build) (64 ビット)/ Windows 11 23H2

kakkokari-gtyih avatar Oct 20 '24 09:10 kakkokari-gtyih

Windowsのディスプレイスケーリングの悪さの可能性はある

lynlynlynx avatar Oct 20 '24 09:10 lynlynlynx

レンダリングバグを疑うならFirefoxで違う結果になるかだけでも確認したらいいと思うけど

tamaina avatar Oct 20 '24 09:10 tamaina