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

feat(frontend/MkUrlPreview): support expanding ActivityPub notes #11478

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

saschanaz
Copy link
Member

@saschanaz saschanaz commented Aug 6, 2023

What

APをサポートしているウェブリンクをノートとして展開できるように

Why

クリックしてページ開かなくてもいいようにしたいので

Additional info (optional)

Checklist

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Aug 6, 2023
@codecov
Copy link

codecov bot commented Aug 6, 2023

Codecov Report

Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (30594dd) 79.98% compared to head (d8434a2) 79.92%.

Files Patch % Lines
packages/frontend/src/components/MkUrlPreview.vue 76.47% 12 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11478      +/-   ##
===========================================
- Coverage    79.98%   79.92%   -0.06%     
===========================================
  Files          954      956       +2     
  Lines       108806   109037     +231     
  Branches      8446     8452       +6     
===========================================
+ Hits         87024    87147     +123     
- Misses       21782    21890     +108     

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

@saschanaz saschanaz marked this pull request as ready for review August 8, 2023 21:45
@github-actions github-actions bot requested review from syuilo and tamaina August 8, 2023 21:46
@@ -80,12 +81,24 @@ const showContent = $ref(false);
padding: 0;
}

.quoted {
margin: 8px 0;
Copy link
Member

Choose a reason for hiding this comment

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

コンポーネントのルート要素がmarginを持つのはデザイン上トラブルの元になるからやめたいかも

Copy link
Member Author

Choose a reason for hiding this comment

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

ちょっといじってみたらmarginなくても良くなりました(postExpandedがtemplate使っていてMkNoteからのmarginが無視されている現象でした)

@saschanaz saschanaz requested a review from syuilo December 30, 2023 16:17
@samunohito
Copy link
Member

@syuilo この件、メンテされたら取り込む意思はありますか?

@syuilo
Copy link
Member

syuilo commented Jan 17, 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
Development

Successfully merging this pull request may close these issues.

3 participants