Skip to content

Prevent IPv6 DNS records from being visually truncated#308

Merged
lissy93 merged 1 commit into
lissy93:masterfrom
dicnunz:codex/fix-dns-ipv6-wrap
May 21, 2026
Merged

Prevent IPv6 DNS records from being visually truncated#308
lissy93 merged 1 commit into
lissy93:masterfrom
dicnunz:codex/fix-dns-ipv6-wrap

Conversation

@dicnunz
Copy link
Copy Markdown
Contributor

@dicnunz dicnunz commented May 20, 2026

Summary

  • allow long DNS record values to wrap inside the DNS Records card
  • fixes full IPv6 AAAA values being visually cut off while keeping existing card scrolling behavior
  • no data-fetching behavior changed

Closes #300.

Verification

  • YARN_CACHE_FOLDER=/tmp/web-check-yarn-cache-$$ yarn install --frozen-lockfile --ignore-scripts
  • yarn eslint --config .config/eslint.config.js src/client/components/Results/DnsRecords.tsx
  • yarn prettier --check src/client/components/Results/DnsRecords.tsx
  • git diff --check

Note: the first install attempt hit a corrupt local Yarn cache; rerunning with an isolated cache completed successfully.

Paid-fix note

If this closes the IPv6 visibility bug and saves maintainer time, I am asking $10 here: https://nicdunz.gumroad.com/l/slop-kill-pass?wanted=true

No issue if this project only handles volunteer contributions; the patch stands either way.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 20, 2026

Deploy Preview for web-check ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a502afa
🔍 Latest deploy log https://app.netlify.com/projects/web-check/deploys/6a0df30912fc830008362c11
😎 Deploy Preview https://deploy-preview-308--web-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a UI-only issue in the DNS Records results card where long record values (notably full IPv6 AAAA addresses) were being visually cut off. It does so by allowing long text to wrap within the scrollable content area, without changing any DNS fetching/processing behavior.

Changes:

  • Enable wrapping of long DNS record values inside the DNS Records card by adding overflow-wrap: anywhere to the card’s content container.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lissy93 lissy93 merged commit 273a6bf into lissy93:master May 21, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] "DNS Records" card truncates IPv6 addresses (visibility only)

3 participants