-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
refactor(frontend): popupMenuの項目作成時に三項演算子をなるべく使わないように #14554
refactor(frontend): popupMenuの項目作成時に三項演算子をなるべく使わないように #14554
Conversation
なんかパフォーマンスとかを気にしてそうしてると思ってたけど、そうじゃないならpushにするの強く賛成 オフトピ(あとdivierをなんかいい感じにMenuItem[][]とかから生成できるともっと便利そうだと思ってたりするけど色々考えると面倒なので一旦なしで良さそう) |
これはやる予定 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14554 +/- ##
===========================================
- Coverage 19.53% 19.52% -0.02%
===========================================
Files 713 713
Lines 100287 100549 +262
Branches 997 997
===========================================
+ Hits 19591 19631 +40
- Misses 80152 80374 +222
Partials 544 544 ☔ View full report in Codecov by Sentry. |
多分いけてると思うけど変換ミスがあるかもしれないので確認してから(そのためにdraftにしてある) |
text: i18n.ts._pages.viewSource, | ||
action: () => router.push(`/@${props.username}/pages/${props.pageName}/view-source`), | ||
}, { | ||
icon: 'ti ti-exclamation-circle', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここは本来こうあるべきだったので修正(自分にはページの編集画面へのリンク・他人にはソースを表示へのリンクが表示されるはずだった)
🙏🏿 |
What
書いてあるとおり
Why
読みやすさが上がる
Additional info (optional)
大規模に置き換えているのでミスがあるかも
Checklist