From fc19799afdf8ec4166d15320fa528212058ad3eb Mon Sep 17 00:00:00 2001 From: David Stone Date: Sat, 24 Oct 2026 22:58:21 -0600 Subject: [PATCH 1/2] fix: clarify read-only DNS lookup and show nameservers --- DESIGN.md | 17 +++++++++++++++++ assets/js/dns-table.js | 11 +++++++++++ views/domain/admin-dns-management.php | 14 +++++++++++--- 3 files changed, 39 insertions(+), 3 deletions(-) diff --git a/DESIGN.md b/DESIGN.md index 775b62954..22f450b81 100644 --- a/DESIGN.md +++ b/DESIGN.md @@ -8,3 +8,20 @@ ring, while the selected control retains the blue card border. Contact-us cards use a non-submitting button so they remain keyboard operable without becoming a checkout selection. After a plan refresh, focus returns to the selected radio only when the visitor has not moved it elsewhere during the request. + +## Domain DNS lookup + +DNS lookup without a configured integration is a normal read-only state, not a +warning. Use a neutral gray information strip with a secondary configuration +link; reserve warning/error styling for actual failures. + +Show returned NS records in a separate nameserver summary, using escaped text +and a neutral explanation when none are returned. Nameservers may help identify +the DNS host, but do not infer a provider name or imply an integration is enabled. + +### Responsive behaviour + +Keep the existing stacked widget layout on desktop and mobile. List nameservers +on separate lines and allow long hostnames to wrap without truncation. Hide the +summary during loading or lookup errors; preserve the existing record table and +provider-managed editing controls. diff --git a/assets/js/dns-table.js b/assets/js/dns-table.js index aa5aa3458..da3ae730e 100644 --- a/assets/js/dns-table.js +++ b/assets/js/dns-table.js @@ -8,6 +8,17 @@ results: {}, loading: true, }, + computed: { + nameservers() { + const records = (this.results.entries || []).concat(this.results.auth || []); + const nameservers = records + .filter((record) => record.type === 'NS' && typeof record.data === 'string') + .map((record) => record.data.replace(/\.$/, '').toLowerCase()) + .filter((nameserver) => nameserver); + + return [ ...new Set(nameservers) ].sort(); + }, + }, updated() { this.$nextTick(function() { diff --git a/views/domain/admin-dns-management.php b/views/domain/admin-dns-management.php index 21959d666..08c9acfdf 100644 --- a/views/domain/admin-dns-management.php +++ b/views/domain/admin-dns-management.php @@ -144,10 +144,10 @@ class="wubox wu-text-red-600 hover:wu-text-red-800" -
- +
+ - + @@ -156,6 +156,14 @@ class="wubox wu-text-red-600 hover:wu-text-red-800"
+
+ +
    +
  • {{ nameserver }}
    • +
+

+

+
From 653e8b4e7ad9529fcb74c010031ebaa3795cc7ec Mon Sep 17 00:00:00 2001 From: David Stone Date: Sun, 25 Oct 2026 09:28:55 -0600 Subject: [PATCH 2/2] fix: hide nameserver summary when no records are found --- DESIGN.md | 2 +- views/domain/admin-dns-management.php | 7 +++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/DESIGN.md b/DESIGN.md index 22f450b81..df364c719 100644 --- a/DESIGN.md +++ b/DESIGN.md @@ -16,7 +16,7 @@ warning. Use a neutral gray information strip with a secondary configuration link; reserve warning/error styling for actual failures. Show returned NS records in a separate nameserver summary, using escaped text -and a neutral explanation when none are returned. Nameservers may help identify +and hiding the entire summary when none are returned. Nameservers may help identify the DNS host, but do not infer a provider name or imply an integration is enabled. ### Responsive behaviour diff --git a/views/domain/admin-dns-management.php b/views/domain/admin-dns-management.php index 08c9acfdf..0a6cf441f 100644 --- a/views/domain/admin-dns-management.php +++ b/views/domain/admin-dns-management.php @@ -156,13 +156,12 @@ class="wubox wu-text-red-600 hover:wu-text-red-800"
-
+
-
    +
    • {{ nameserver }}
    -

    -

    +