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): ノートの公開範囲に応じて色分けできるように #14800

Draft
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Oct 20, 2024

What

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

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

image

Why

Fix #14784

Additional info (optional)

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

image

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 Oct 20, 2024
Copy link

codecov bot commented Oct 20, 2024

Codecov Report

Attention: Patch coverage is 82.60870% with 12 lines in your changes missing coverage. Please review.

Project coverage is 18.86%. Comparing base (9230ee5) to head (5a113a1).

Files with missing lines Patch % Lines
packages/frontend/src/pages/settings/general.vue 0.00% 6 Missing ⚠️
packages/frontend/src/store.ts 0.00% 4 Missing ⚠️
...es/frontend/src/components/MkDateSeparatedList.vue 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14800       +/-   ##
============================================
- Coverage    38.68%   18.86%   -19.83%     
============================================
  Files         1603      764      -839     
  Lines       204302   111111    -93191     
  Branches      4042     1233     -2809     
============================================
- Hits         79040    20961    -58079     
+ Misses      124622    89563    -35059     
+ Partials       640      587       -53     

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

@tamaina
Copy link
Contributor

tamaina commented Oct 20, 2024

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

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

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Oct 20, 2024

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

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

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

@tamaina

This comment was marked as off-topic.

@tamaina

This comment was marked as off-topic.

@tamaina
Copy link
Contributor

tamaina commented Oct 20, 2024

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

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Oct 20, 2024

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

@lynlynlynx
Copy link
Contributor

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

@tamaina
Copy link
Contributor

tamaina commented Oct 20, 2024

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

@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft October 20, 2024 11:14
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
3 participants