Skip to content

enhance(frontend): 検索時URLをクエリパラメータ付きのもので置き換えるように#16278

Open
k0range wants to merge 4 commits intomisskey-dev:developfrom
k0range:15238-search-query-params
Open

enhance(frontend): 検索時URLをクエリパラメータ付きのもので置き換えるように#16278
k0range wants to merge 4 commits intomisskey-dev:developfrom
k0range:15238-search-query-params

Conversation

@k0range
Copy link
Copy Markdown
Contributor

@k0range k0range commented Jul 13, 2025

What

  • 検索時に、URLをクエリパラメータ付きのもので置き換えるようになりました
    • 検索ページを全画面で開いている場合は、アドレスバーのURLが変わります。
    • 検索ページをウィンドウで開いている場合は、右クリックメニューの「リンクをコピー」からURLを取得できます。
  • ページ転移を行わずにURLを置き換える仕組みがなさそうでしたので、nirax.ts にURLを置き換える関数 replaceUrl() を追加しました。

Why

Resolves #15238
クエリパラメータがあることの認知度向上や、検索結果の共有がやりやすくなるなどの利点があります。

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

k0range added 3 commits July 13, 2025 16:32
ページ転移を行わずにアドレスバー上や、ウィンドウからコピーできるページのURLを変更することができます。
ページを再描画せずにクエリパラメータを追加したい場合などで利用できます。
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 13, 2025

Codecov Report

❌ Patch coverage is 3.57143% with 27 lines in your changes missing coverage. Please review.
✅ Project coverage is 17.68%. Comparing base (e98252a) to head (4ef08fa).
⚠️ Report is 1338 commits behind head on develop.

Files with missing lines Patch % Lines
packages/frontend/src/pages/search.note.vue 0.00% 11 Missing ⚠️
packages/frontend/src/pages/search.user.vue 0.00% 10 Missing ⚠️
packages/frontend/src/lib/nirax.ts 14.28% 6 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #16278       +/-   ##
============================================
- Coverage    42.56%   17.68%   -24.89%     
============================================
  Files         1685      860      -825     
  Lines       170650    78785    -91865     
  Branches      4223     1335     -2888     
============================================
- Hits         72643    13930    -58713     
+ Misses       97546    64849    -32697     
+ Partials       461        6      -455     

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

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

@k0range
Copy link
Copy Markdown
Contributor Author

k0range commented Jul 13, 2025

4ef08fa
lintがパスしなかったのを直しました🙏

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.

/searchで検索した際、URLがクエリパラメータつきに書き換わると使いやすくなる

1 participant