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): ウィジェットを非表示にできるPageMetaを追加 #12456

Merged
merged 5 commits into from
Dec 8, 2023

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Nov 25, 2023

What

definePageMetadatahideWidgets が設定できるようになる
true にすると、そのページではウィジェットが非表示になる(右下のボタンでドロワーとして表示できる)

image

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 Nov 25, 2023
Copy link

codecov bot commented Nov 25, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f80ae7f) 78.81% compared to head (2a06956) 78.54%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12456      +/-   ##
===========================================
- Coverage    78.81%   78.54%   -0.27%     
===========================================
  Files          958      183     -775     
  Lines       104272    24122   -80150     
  Branches      8335      497    -7838     
===========================================
- Hits         82177    18947   -63230     
+ Misses       22095     5175   -16920     

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

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

ページ側が自身が表示されることになる親UIについての情報を知っているというのは微妙だからneedWideAreaとかどうかしら

@syuilo syuilo merged commit ac4089f into misskey-dev:develop Dec 8, 2023
14 of 15 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 8, 2023

👍🏻

@kakkokari-gtyih kakkokari-gtyih deleted the enh-hide-widgets branch December 8, 2023 06:32
camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* (enhance) ウィジェットを非表示にできるPageMetaを追加

* fix lint

* rename

---------

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
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
Development

Successfully merging this pull request may close these issues.

2 participants