Multiline comments support
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- react, typescript
- Domain
- frontend
Research direction
Start by reviewing the existing shift-key line-selection implementation referenced in the issue and the discussion in issue #180. Compare the requested mouse-drag behavior with GitHub's line selection and inspect the relevant diff component code. Done means the intended drag interaction is implemented and verified alongside the existing selection behavior.
Written by the indexing model from the issue text.
Description
hey @otakustay, First of all Thanks for going ahead and making the changes also, really appreciate it.
and sorry, for being inactive for a while I got busy in some other things.
I checked your implementation and selecting lines with shift key does look good but I wanted to know If we can have something more similar to what Github has which is selecting lines by clicking and dragging the mouse?
Let me know your thoughts on this and meanwhile I'm also going through the codebase to check the possibilities.
Originally posted by @msk4862 in https://github.com/otakustay/react-diff-view/issues/180#issuecomment-1387481871
- Dominant language
- TypeScript
- Stars
- 1k
- Forks
- 88
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from otakustay/react-diff-view
-
pinned
Difficulty 1/5 Under an hour Newbie friendliness 62/100
otakustay/react-diff-view#101 · 5 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
otakustay/react-diff-view#238 · 1 comment ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
otakustay/react-diff-view#237 ·
-
跨行注释未闭合时的代码被高亮为注释了Open
Difficulty 3/5 1-2 days Newbie friendliness 45/100
otakustay/react-diff-view#233 · 2 comments ·
-
Accessibility: Add screenreader-only text to table cells for status indicatorsPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 3/5 1-2 days Newbie friendliness 42/100
otakustay/react-diff-view#232 · 1 comment ·
All issues in otakustay/react-diff-view
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Comfy-Org/ComfyUI_frontend#20346 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
decentralized-identity/didwebvh-ts#203 ·
Maintainers usually reply within 1 day
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
lingdojo/kana-dojo#31791 · 1 comment · 5 reactions ·
Maintainers usually reply within 1 day
-
Telegram webhook: line breaks lost since switch to rich messagesPossibly taken @Kshot3000 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day
-
github_actions security
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day