Inline-block may cause Chrome's built-in search to fail to match the corresponding content
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 62/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Stale
- Tech stack
- css
- Domain
- frontend
Research direction
Start at src/styles/index.css around line 77 and inspect the .diff-code-edit display rule. Reproduce the provided rendered markup in Chrome and use Ctrl+F to search for "onUpdate". Done means the corresponding split inline code is found by the browser's built-in search without disrupting the diff rendering.
Written by the indexing model from the issue text.
Description
Assuming that the rendered code is like this
<td class="diff-code diff-code-insert" style="user-select: auto;">
<span class="diff-code-edit">
<span class="token function">onU</span>
</span>
<span class="token function">pdat</span>
<span class="diff-code-edit">
<span class="token function">e</span>
</span>
</td>
When I use ctrl+f to search for "onUpdate" on the webpage, I cannot find the above one because the display of .diff-code-edit is inline-block.
I think display: inline-block should be changed to display: inline.
- 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
-
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 ·
-
pinned
Difficulty 4/5 3-5 days Newbie friendliness 35/100
otakustay/react-diff-view#185 · 15 comments ·
All issues in otakustay/react-diff-view
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
clawsweeper:needs-maintainer-review clawsweeper:needs-product-decision clawsweeper:no-new-fix-pr impact:session-state issue-rating: 🌊 off-meta tidepool P2
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
openclaw/openclaw#165245 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
-
agent-canvas bug priority:low ready-for-dev
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
🐛 bug
Difficulty 2/5 1-3 hours Newbie friendliness 61/100
spicetify/marketplace#1262 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 Half a day Newbie friendliness 68/100
Maintainers usually reply within 1 day