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
Migrate comment to React #424
Conversation
Super nice. |
Conflicts: client/app.js client/app.tsx client/crowi.js client/crowi.ts lib/views/widget/page_side_content.html
60976b7
to
ad34b4b
Compare
ad34b4b
to
5cdeb5f
Compare
Codecov Report
@@ Coverage Diff @@
## master #424 +/- ##
==========================================
+ Coverage 35.17% 35.28% +0.11%
==========================================
Files 112 112
Lines 4543 4557 +14
Branches 700 702 +2
==========================================
+ Hits 1598 1608 +10
- Misses 2319 2321 +2
- Partials 626 628 +2
Continue to review full report at Codecov.
|
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.
@lightnet328 Only 1 point should be fixed.
<> | ||
<CommentList className="collapse" id="page-comments-list-newer" crowi={crowi} comments={comments} revisionId={revisionId} /> | ||
<a className="text-center" data-toggle="collapse" href="#page-comments-list-newer"> | ||
<i className="fa fa-angle-double-up" /> Comments for Newer Revision <i className="fa fa-angle-double-up" /> |
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.
❌ fa
is no longer valid.
Use Common/Icon
instead.
b3466dd
to
7f3197e
Compare
Overview
There is no major change in appearance or feature, and the comment feature was migrated to React.
Changes
Depends on
#420