From aeed92d50260d31e5f84b4e366bb55b1c2afcc27 Mon Sep 17 00:00:00 2001
From: zzhaobraze <zeyuan.zhao@braze.com>
Date: Thu, 19 Dec 2024 13:14:40 -0500
Subject: [PATCH 1/2] normalize tab names better

---
 _plugins/tabs.rb | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/_plugins/tabs.rb b/_plugins/tabs.rb
index adc273de945..3022649ebcd 100644
--- a/_plugins/tabs.rb
+++ b/_plugins/tabs.rb
@@ -14,11 +14,11 @@ def render(context)
           if tabs.length > 0
             tabs.each_with_index do |tab, ind|
               # scan returns array of results, only care about first match
-              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(' ', '-')
+              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(/[^0-9a-z]/i, '-')
               if ind == 0
                 tabslist += ' active'
               end
-              tabslist += '"><a class="' + @tabclass + '" data-tab-target="' + @tabid + '" data-tab="' + tab[0].gsub(' ', '-') + '">' + tab[0] + '</a></li>' + "\n"
+              tabslist += '"><a class="' + @tabclass + '" data-tab-target="' + @tabid + '" data-tab="' + tab[0].gsub(/[^0-9a-z]/i, '-') + '">' + tab[0] + '</a></li>' + "\n"
             end
           end
           tabslist += '</ul>'  + "\n"
@@ -48,7 +48,7 @@ def render(context)
           content = indentation ? super.gsub(/^#{' |\t' * indentation}/, '') : super
           content = converter.convert(content)
           content = content.strip # Strip again to avoid "\n"
-          tabslug = @tab.gsub(' ', '-')
+          tabslug = @tab.gsub(/[^0-9a-z]/i, '-')
 
           '<div class="ab-tab-pane ' + tabslug + '_tab " data-tab="' + @tab + '">' + content + "</div>"
       end
@@ -70,11 +70,11 @@ def render(context)
           if tabs.length > 0
             tabs.each_with_index do |tab, ind|
               # scan returns array of results, only care about first match
-              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(' ', '-') + '_sub_tab'
+              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(/[^0-9a-z]/i, '-') + '_sub_tab'
               if ind == 0
                 tabslist += ' sub_active'
               end
-              tabslist += '"><a class="' + @tabclass + '" data-sub_tab-target="' + @tabid + '" data-sub_tab="' + tab[0].gsub(' ', '-') + '_sub_tab">' + tab[0] + '</a></li>' + "\n"
+              tabslist += '"><a class="' + @tabclass + '" data-sub_tab-target="' + @tabid + '" data-sub_tab="' + tab[0].gsub(/[^0-9a-z]/i, '-') + '_sub_tab">' + tab[0] + '</a></li>' + "\n"
             end
           end
           tabslist += '</ul>'  + "\n"

From 7bf8c0bed97e44fc6a5ce6184ba2725000436a1b Mon Sep 17 00:00:00 2001
From: zzhaobraze <zeyuan.zhao@braze.com>
Date: Thu, 19 Dec 2024 14:19:44 -0500
Subject: [PATCH 2/2] use md5 fallback

---
 _plugins/tabs.rb | 23 ++++++++++++++++-------
 1 file changed, 16 insertions(+), 7 deletions(-)

diff --git a/_plugins/tabs.rb b/_plugins/tabs.rb
index 3022649ebcd..cea844897f7 100644
--- a/_plugins/tabs.rb
+++ b/_plugins/tabs.rb
@@ -1,3 +1,5 @@
+require 'digest/md5'
+
 module Tags
     class TabsBlock < Liquid::Block
       def initialize(tag_name, tabonly = 'false', tokens)
@@ -13,12 +15,15 @@ def render(context)
           tabslist = '<ul class="ab-nav ab-nav-tabs ' + @tabclass + '_ul" id="' + @tabid + '_nav">' + "\n"
           if tabs.length > 0
             tabs.each_with_index do |tab, ind|
+              tabslug = tab[0].gsub(/[^0-9a-z]/i, '')
+              tabslug = Digest::MD5.hexdigest(tab[0]) if tabslug.empty?
+
               # scan returns array of results, only care about first match
-              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(/[^0-9a-z]/i, '-')
+              tabslist += '    <li tabindex="0" class="coderow ' + tabslug
               if ind == 0
                 tabslist += ' active'
               end
-              tabslist += '"><a class="' + @tabclass + '" data-tab-target="' + @tabid + '" data-tab="' + tab[0].gsub(/[^0-9a-z]/i, '-') + '">' + tab[0] + '</a></li>' + "\n"
+              tabslist += '"><a class="' + @tabclass + '" data-tab-target="' + @tabid + '" data-tab="' + tabslug + '">' + tab[0] + '</a></li>' + "\n"
             end
           end
           tabslist += '</ul>'  + "\n"
@@ -48,7 +53,8 @@ def render(context)
           content = indentation ? super.gsub(/^#{' |\t' * indentation}/, '') : super
           content = converter.convert(content)
           content = content.strip # Strip again to avoid "\n"
-          tabslug = @tab.gsub(/[^0-9a-z]/i, '-')
+          tabslug = @tab.gsub(/[^0-9a-z]/i, '')
+          tabslug = Digest::MD5.hexdigest(@tab) if tabslug.empty?
 
           '<div class="ab-tab-pane ' + tabslug + '_tab " data-tab="' + @tab + '">' + content + "</div>"
       end
@@ -66,15 +72,17 @@ def initialize(tag_name, tabonly = 'false', tokens)
       def render(context)
           tabs = super.scan(/data\-sub\_tab=\"(.*?)\"/)
           tabslist = '<ul class="ab-sub_nav ab-sub_nav-sub_tabs ' + @tabclass + '_ul" id="' + @tabid + '_nav">' + "\n"
-
           if tabs.length > 0
             tabs.each_with_index do |tab, ind|
+              tabslug = tab[0].gsub(/[^0-9a-z]/i, '')
+              tabslug = Digest::MD5.hexdigest(tab[0]) if tabslug.empty?
+
               # scan returns array of results, only care about first match
-              tabslist += '    <li tabindex="0" class="coderow ' + tab[0].gsub(/[^0-9a-z]/i, '-') + '_sub_tab'
+              tabslist += '    <li tabindex="0" class="coderow ' + tabslug + '_sub_tab'
               if ind == 0
                 tabslist += ' sub_active'
               end
-              tabslist += '"><a class="' + @tabclass + '" data-sub_tab-target="' + @tabid + '" data-sub_tab="' + tab[0].gsub(/[^0-9a-z]/i, '-') + '_sub_tab">' + tab[0] + '</a></li>' + "\n"
+              tabslist += '"><a class="' + @tabclass + '" data-sub_tab-target="' + @tabid + '" data-sub_tab="' + tabslug + '_sub_tab">' + tab[0] + '</a></li>' + "\n"
             end
           end
           tabslist += '</ul>'  + "\n"
@@ -104,7 +112,8 @@ def render(context)
           content = indentation ? super.gsub(/^#{' |\t' * indentation}/, '') : super
           content = converter.convert(content)
           content = content.strip # Strip again to avoid "\n"
-          tabslug = @tab.gsub(' ', '-')
+          tabslug = @tab.gsub(/[^0-9a-z]/i, '')
+          tabslug = Digest::MD5.hexdigest(@tab) if tabslug.empty?
 
           '<div class="ab-sub_tab-pane ' + tabslug + '_sub_tab " data-sub_tab="' + @tab + '">' + content + "</div>"
       end