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
#7027: add code comment brick #7028
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #7028 +/- ##
==========================================
+ Coverage 71.05% 71.09% +0.03%
==========================================
Files 1207 1210 +3
Lines 37335 37399 +64
Branches 6997 7004 +7
==========================================
+ Hits 26528 26588 +60
- Misses 10807 10811 +4 ☔ View full report in Codecov by Sentry. |
Rather than having the |
Looks good to me! |
I'll chat with @BrandonPxBx today. We don't have much screen real-estate in the outline view |
I know, but I'm not sure I'm convinced the current implementation is useful. I think I'd rather have to write shorter comments than have to click into a brick to read it. |
@grahamlangford @BrandonPxBx @mnholtz what do you think of including a snippet of the comment preview in the outline? We'd likely want a way to show it's truncated or not Right now it ignores new lines. We could consider having it show the first paragraph (but still truncate). That would support a summary + details style of commenting |
Definitely on board with this @twschiller |
|
Updated the UI to include line-clamping and the [Experimental] tag: @twschiller @grahamlangford @BrandonPxBx does this satisfy the remaining UI work? |
Looks great to me. |
When the PR is merged, the first loom link found on this PR will be posted to |
What does this PR do?
Remaining Work
Discussion
Demo
Future Work
Team Coordination
fas fa-book
or a comment bubble: @brittanyjoiner15Checklist
src/tsconfig.strictNullChecks.json
(if possible)