sphinx-filter-tabs 0.9.2__py3-none-any.whl → 1.2.2__py3-none-any.whl

filter_tabs/extension.py

@@ -1,380 +1,393 @@
-#
-# extension.py: The core logic for the sphinx-filter-tabs Sphinx extension.
-#
+# filter_tabs/extension.py
+"""
+Core extension module for sphinx-filter-tabs.
+Consolidated version containing directives, data models, nodes, and Sphinx integration.
+"""
 
-# --- Imports ---
 from __future__ import annotations
-
-import re
-import uuid
 import copy
 import shutil
 from pathlib import Path
 from docutils import nodes
-from docutils.parsers.rst import Directive
+from docutils.parsers.rst import Directive, directives
 from sphinx.application import Sphinx
 from sphinx.util import logging
 from sphinx.writers.html import HTML5Translator
 
-from typing import TYPE_CHECKING, Any, Dict, List
+from typing import TYPE_CHECKING, Any, Dict, List, Optional
+from dataclasses import dataclass, field
 from . import __version__
 
 if TYPE_CHECKING:
-    from sphinx.config import Config
     from sphinx.environment import BuildEnvironment
 
-# --- Constants ---
-_CSS_NAMESPACE = uuid.UUID('d1b1b3e8-5e7c-48d6-a235-9a4c14c9b139')
-
-SFT_CONTAINER = "sft-container"
-SFT_FIELDSET = "sft-fieldset"
-SFT_LEGEND = "sft-legend"
-SFT_TAB_BAR = "sft-tab-bar"
-SFT_CONTENT = "sft-content"
-SFT_PANEL = "sft-panel"
-SFT_TEMP_PANEL = "sft-temp-panel"
-COLLAPSIBLE_SECTION = "collapsible-section"
-COLLAPSIBLE_CONTENT = "collapsible-content"
-CUSTOM_ARROW = "custom-arrow"
+# =============================================================================
+# Custom Docutils Nodes
+# =============================================================================
 
-logger = logging.getLogger(__name__)
+class ContainerNode(nodes.General, nodes.Element): pass
+class FieldsetNode(nodes.General, nodes.Element): pass
+class LegendNode(nodes.General, nodes.Element): pass
+class RadioInputNode(nodes.General, nodes.Element): pass
+class LabelNode(nodes.General, nodes.Element): pass
+class PanelNode(nodes.General, nodes.Element): pass
+class DetailsNode(nodes.General, nodes.Element): pass
+class SummaryNode(nodes.General, nodes.Element): pass
 
-# --- Custom Nodes ---
-class ContainerNode(nodes.General, nodes.Element):
-    pass
 
-class FieldsetNode(nodes.General, nodes.Element): 
-    pass
+# =============================================================================
+# Data Models and Configuration
+# =============================================================================
 
-class LegendNode(nodes.General, nodes.Element): 
-    pass
+@dataclass
+class TabData:
+    """
+    Represents a single tab's data within a filter-tabs directive.
+    
+    Attributes:
+        name: Display name of the tab
+        is_default: Whether this tab should be selected by default
+        aria_label: Optional ARIA label for accessibility
+        content: List of docutils nodes containing the tab's content
+    """
+    name: str
+    is_default: bool = False
+    aria_label: Optional[str] = None
+    content: List[nodes.Node] = field(default_factory=list)
+    
+    def __post_init__(self):
+        """Validate tab data after initialization."""
+        if not self.name:
+            raise ValueError("Tab name cannot be empty")
+        
+        # Ensure content is a list
+        if self.content is None:
+            self.content = []
 
-class RadioInputNode(nodes.General, nodes.Element): 
-    pass
 
-class LabelNode(nodes.General, nodes.Element): 
-    pass
+@dataclass
+class FilterTabsConfig:
+    """
+    Simplified configuration settings for filter-tabs rendering.
+    Reduced from 9 options to 2 essential ones.
+    """
+    # Essential theming - only the highlight color is commonly customized
+    highlight_color: str = '#007bff'
+    
+    # Development option
+    debug_mode: bool = False
+    
+    @classmethod
+    def from_sphinx_config(cls, app_config) -> 'FilterTabsConfig':
+        """Create a FilterTabsConfig from Sphinx app.config."""
+        return cls(
+            highlight_color=getattr(
+                app_config, 'filter_tabs_highlight_color', cls.highlight_color
+            ),
+            debug_mode=getattr(
+                app_config, 'filter_tabs_debug_mode', cls.debug_mode
+            ),
+        )
+    
+    def to_css_properties(self) -> str:
+        """Convert config to CSS custom properties string."""
+        # Generate CSS variables - the highlight color drives all other colors
+        return f"--sft-highlight-color: {self.highlight_color};"
 
-class PanelNode(nodes.General, nodes.Element): 
-    pass
 
-class DetailsNode(nodes.General, nodes.Element): 
-    pass
+@dataclass
+class IDGenerator:
+    """
+    Centralized ID generation for consistent element identification.
+    
+    This ensures all IDs follow a consistent pattern and are unique
+    within their filter-tabs group.
+    """
+    group_id: str
+    
+    def radio_id(self, index: int) -> str:
+        """Generate ID for a radio button."""
+        return f"{self.group_id}-radio-{index}"
+    
+    def panel_id(self, index: int) -> str:
+        """Generate ID for a content panel."""
+        return f"{self.group_id}-panel-{index}"
+    
+    def desc_id(self, index: int) -> str:
+        """Generate ID for a screen reader description."""
+        return f"{self.group_id}-desc-{index}"
+    
+    def legend_id(self) -> str:
+        """Generate ID for the fieldset legend."""
+        return f"{self.group_id}-legend"
+    
+    def label_id(self, index: int) -> str:
+        """Generate ID for a label (if needed)."""
+        return f"{self.group_id}-label-{index}"
 
-class SummaryNode(nodes.General, nodes.Element): 
-    pass
 
+# =============================================================================
+# Directive Classes
+# =============================================================================
 
-# --- Renderer Class ---
-class FilterTabsRenderer:
-    """
-    Handles the primary logic of converting the parsed directive content
-    into a final node structure for both HTML and fallback formats (like LaTeX).
-    """
-    def __init__(self, directive: Directive, tab_names: list[str], default_tab: str, temp_blocks: list[nodes.Node]):
-        """Initializes the renderer with all necessary context from the directive."""
-        self.directive: Directive = directive
-        self.env: BuildEnvironment = directive.state.document.settings.env
-        self.tab_names: list[str] = tab_names
-        self.default_tab: str = default_tab
-        self.temp_blocks: list[nodes.Node] = temp_blocks
-
-    def render_html(self) -> list[nodes.Node]:
-        """Constructs the complete, W3C valid, and ARIA-compliant docutils node tree."""
-        # Ensure a unique ID for each filter-tabs instance on a page.
-        if not hasattr(self.env, 'filter_tabs_counter'):
-            self.env.filter_tabs_counter = 0
-        self.env.filter_tabs_counter += 1
-        group_id = f"filter-group-{self.env.filter_tabs_counter}"
-
-        config = self.env.app.config
-
-        # Create a dictionary of CSS Custom Properties from conf.py settings.
-        style_vars = {
-            "--sft-border-radius": str(config.filter_tabs_border_radius),
-            "--sft-tab-background": str(config.filter_tabs_tab_background_color),
-            "--sft-tab-font-size": str(config.filter_tabs_tab_font_size),
-            "--sft-tab-highlight-color": str(config.filter_tabs_tab_highlight_color),
-            "--sft-collapsible-accent-color": str(config.filter_tabs_collapsible_accent_color),
-        }
-        style_string = "; ".join([f"{key}: {value}" for key, value in style_vars.items()])
-
-        # If debug mode is on, log the generated ID and styles.
-        if config.filter_tabs_debug_mode:
-            logger.info(f"[sphinx-filter-tabs] ID: {group_id}, Styles: '{style_string}'")
-
-        # Create the main container node with the inline style for theming.
-        container = ContainerNode(classes=[SFT_CONTAINER], style=style_string)
-
-        # Build the semantic structure using fieldset and a hidden legend.
-        fieldset = FieldsetNode()
-        legend = LegendNode()
-        legend += nodes.Text(f"Filter by: {', '.join(self.tab_names)}")
-        fieldset += legend
-        
-        # --- CSS Generation ---
-        css_rules = []
-        for i, tab_name in enumerate(self.tab_names):
-            # FIX 1: Use position-based IDs instead of hash-based
-            radio_id = f"{group_id}-tab-{i}"
-            panel_id = f"{radio_id}-panel"
-            css_rules.append(
-                f".{SFT_TAB_BAR}:has(#{radio_id}:checked) ~ .sft-content > #{panel_id} {{ display: block; }}"
-            )
-        
-        # Write the dynamic CSS to a temporary file and add it to the build.
-        css_content = ''.join(css_rules)
-        static_dir = Path(self.env.app.outdir) / '_static'
-        static_dir.mkdir(parents=True, exist_ok=True)
-        css_filename = f"dynamic-filter-tabs-{group_id}.css"
-        (static_dir / css_filename).write_text(css_content, encoding='utf-8')
-        self.env.app.add_css_file(css_filename)
-
-        # The tab bar container - NO ARIA attributes here
-        tab_bar = ContainerNode(classes=[SFT_TAB_BAR])
-        fieldset += tab_bar
-
-        # The content area holds all the panels
-        content_area = ContainerNode(classes=[SFT_CONTENT])
-        fieldset += content_area
-
-        # Map tab names to their content blocks for easy lookup.
-        content_map = {block['filter-name']: block.children for block in self.temp_blocks}
-        
-        # 1. Create all radio buttons and labels - NO ARIA on labels
-        for i, tab_name in enumerate(self.tab_names):
-            # FIX 1: Use position-based IDs
-            radio_id = f"{group_id}-tab-{i}"
-            panel_id = f"{radio_id}-panel"
-            
-            # The radio button is for state management.
-            radio = RadioInputNode(type='radio', name=group_id, ids=[radio_id])
-            
-            is_default = (self.default_tab == tab_name) or (i == 0 and not self.default_tab)
-            if is_default:
-                radio['checked'] = 'checked'
-            tab_bar += radio
-
-            # FIX 2: Remove ALL ARIA attributes from labels - just use for_id
-            # FIX 3: Don't add IDs to labels - they don't need them
-            label = LabelNode(for_id=radio_id)
-            label += nodes.Text(tab_name)
-            tab_bar += label
-
-        # 2. Create all tab panels - panels can have ARIA attributes
-        all_tab_names = ["General"] + self.tab_names
-        for i, tab_name in enumerate(all_tab_names):
-            if tab_name == "General":
-                # General panel doesn't correspond to a specific tab control
-                panel = PanelNode(
-                    classes=[SFT_PANEL], 
-                    **{'data-filter': tab_name}
-                )
-            else:
-                # FIX 1: Use position-based IDs for panels too
-                tab_index = self.tab_names.index(tab_name)
-                radio_id = f"{group_id}-tab-{tab_index}"
-                panel_id = f"{radio_id}-panel"
-                
-                # Panels can have ARIA attributes for screen readers
-                panel_attrs = {
-                    'classes': [SFT_PANEL],
-                    'ids': [panel_id],
-                    'role': 'tabpanel',
-                    'aria-labelledby': radio_id,
-                    'tabindex': '0'
-                }
-                panel = PanelNode(**panel_attrs)
-
-            if tab_name in content_map:
-                panel.extend(copy.deepcopy(content_map[tab_name]))
-            content_area += panel
-
-        container.children = [fieldset]
-        return [container]
+# --- Constants ---
+COLLAPSIBLE_SECTION = "collapsible-section"
+COLLAPSIBLE_CONTENT = "collapsible-content"
+CUSTOM_ARROW = "custom-arrow"
 
-    def render_fallback(self) -> list[nodes.Node]:
-        """Renders content as a series of simple admonitions for non-HTML builders (e.g., LaTeX/PDF)."""
-        output_nodes: list[nodes.Node] = []
-        content_map = {block['filter-name']: block.children for block in self.temp_blocks}
-        # "General" content is rendered first, without a title.
-        if "General" in content_map:
-            output_nodes.extend(copy.deepcopy(content_map["General"]))
-        # Each specific tab's content is placed inside a titled admonition block.
-        for tab_name in self.tab_names:
-            if tab_name in content_map:
-                admonition = nodes.admonition()
-                admonition += nodes.title(text=tab_name)
-                admonition.extend(copy.deepcopy(content_map[tab_name]))
-                output_nodes.append(admonition)
-        return output_nodes
-
-    @staticmethod
-    def _css_escape(name: str) -> str:
-        """
-        Generates a deterministic, CSS-safe identifier from any given tab name string.
-        """
-        return str(uuid.uuid5(_CSS_NAMESPACE, name.strip().lower()))
+logger = logging.getLogger(__name__)
 
 
 class TabDirective(Directive):
-    """Handles the `.. tab::` directive, capturing its content."""
+    """Handles the `.. tab::` directive, capturing its content and options."""
     has_content = True
     required_arguments = 1
     final_argument_whitespace = True
+    option_spec = {'aria-label': directives.unchanged}
 
     def run(self) -> list[nodes.Node]:
-        """
-        Parses the content of a tab and stores it in a temporary container.
-        """
+        """Process the tab directive and return container node."""
         env = self.state.document.settings.env
-        # Ensure `tab` is only used inside `filter-tabs`.
+        
+        # Validate context
         if not hasattr(env, 'sft_context') or not env.sft_context:
-            raise self.error("`tab` can only be used inside a `filter-tabs` directive.")
-        # Store the tab name and parsed content in a temporary node.
-        container = nodes.container(classes=[SFT_TEMP_PANEL])
-        container['filter-name'] = self.arguments[0].strip()
+           raise self.error("`tab` can only be used inside a `filter-tabs` directive.")
+        
+        # Parse tab argument - moved from parsers.py
+        try:
+            tab_name, is_default = self._parse_tab_argument(self.arguments[0])
+        except ValueError as e:
+            raise self.error(f"Invalid tab argument: {e}")
+        
+        # Create container with parsed data
+        container = nodes.container(classes=["sft-temp-panel"])
+        container['filter_name'] = tab_name
+        container['is_default'] = is_default
+        container['aria_label'] = self.options.get('aria-label', None)
+        
+        # Parse the content
         self.state.nested_parse(self.content, self.content_offset, container)
+        
         return [container]
+    
+    def _parse_tab_argument(self, argument: str) -> tuple[str, bool]:
+        """Parse a tab argument string to extract name and default status."""
+        import re
+        
+        logger.debug(f"Parsing tab argument: '{argument}'")
+        if not argument:
+            raise ValueError("Tab argument cannot be empty")
+        
+        lines = argument.strip().split('\n')
+        first_line = lines[0].strip()
+        
+        if len(lines) > 1:
+            logger.debug(
+                f"Tab argument contains multiple lines, using only first: '{first_line}'"
+            )
+        
+        DEFAULT_PATTERN = re.compile(r"^(.*?)\s*\(\s*default\s*\)$", re.IGNORECASE)
+        match = DEFAULT_PATTERN.match(first_line)
+        if match:
+            tab_name = match.group(1).strip()
+            if not tab_name:
+                raise ValueError("Tab name cannot be empty")
+            return tab_name, True
+        
+        return first_line, False
 
 
 class FilterTabsDirective(Directive):
     """Handles the main `.. filter-tabs::` directive."""
     has_content = True
-    required_arguments = 1
-    final_argument_whitespace = True
+    required_arguments = 0
+    optional_arguments = 0
+    option_spec = {'legend': directives.unchanged}
 
     def run(self) -> list[nodes.Node]:
-        """
-        Parses the list of tabs, manages the parsing context for its content,
-        and delegates the final rendering to the FilterTabsRenderer.
-        """
+        """Process the filter-tabs directive."""
         env = self.state.document.settings.env
         
-        # Set a context flag to indicate that we are inside a filter-tabs block.
+        # Set up context
         if not hasattr(env, 'sft_context'):
             env.sft_context = []
         env.sft_context.append(True)
 
-        # Parse the content of the directive to find all `.. tab::` blocks.
+        # Parse content
         temp_container = nodes.container()
         self.state.nested_parse(self.content, self.content_offset, temp_container)
+        
         env.sft_context.pop()
+        
+        # Get the custom legend option
+        custom_legend = self.options.get('legend')
 
-        # Find all the temporary panel nodes created by the TabDirective.
-        temp_blocks = temp_container.findall(lambda n: isinstance(n, nodes.Element) and SFT_TEMP_PANEL in n.get('classes', []))
-        if not temp_blocks:
-            self.error("No `.. tab::` directives found inside `filter-tabs`. Content will not be rendered.")
-            return []
-
-        # Parse the tab names from the directive's arguments.
-        tabs_raw = [t.strip() for t in self.arguments[0].split(',')]
-        tab_names_only = [re.sub(r'\s*\(\s*default\s*\)$', '', t, re.IGNORECASE).strip() for t in tabs_raw]
-
-        if len(set(tab_names_only)) != len(tab_names_only):
-            raise self.error(f"Duplicate tab names found: {tab_names_only}")
-
-        # Identify the default tab.
-        default_tab, tab_names = "", []
-        for tab in tabs_raw:
-            match = re.match(r"^(.*?)\s*\(\s*default\s*\)$", tab, re.IGNORECASE)
-            tab_name = match.group(1).strip() if match else tab
-            if match and not default_tab:
-                default_tab = tab_name
-            tab_names.append(tab_name)
-
-        # If no default is specified, the first tab becomes the default.
-        if not default_tab and tab_names:
-            default_tab = tab_names[0]
-
-        # Instantiate the renderer and call the appropriate render method based on the builder.
-        renderer = FilterTabsRenderer(self, tab_names, default_tab, temp_blocks)
+        # Separate general content from tabs using TabData
+        general_content = []
+        tab_data_list: List[TabData] = []
+        
+        for node in temp_container.children:
+            if isinstance(node, nodes.Element) and "sft-temp-panel" in node.get('classes', []):
+                # Create TabData object instead of dictionary
+                tab_data = TabData(
+                    name=node.get('filter_name', 'Unknown'),
+                    is_default=node.get('is_default', False),
+                    aria_label=node.get('aria_label', None),
+                    content=list(node.children)
+                )
+                tab_data_list.append(tab_data)
+            else:
+                general_content.append(node)
+        
+        # Validate tabs - moved from parsers.py
+        if not tab_data_list:
+            error_message = (
+                "No `.. tab::` directives found inside `.. filter-tabs::`. "
+                "You must include at least one tab."
+            )
+            if general_content:
+                error_message += (
+                    " Some content was found, but it was not part of a `.. tab::` block."
+                )
+            raise self.error(error_message)
+
+        try:
+            self._validate_tabs(tab_data_list, skip_empty_check=True)
+        except ValueError as e:
+            raise self.error(str(e))
+        
+        # Set first tab as default if none specified
+        if not any(tab.is_default for tab in tab_data_list):
+            tab_data_list[0].is_default = True
+        
+        # Import renderer here to avoid circular imports
+        from .renderer import FilterTabsRenderer
+        renderer = FilterTabsRenderer(self, tab_data_list, general_content, custom_legend=custom_legend)
+        
+        # Render based on builder
         if env.app.builder.name == 'html':
             return renderer.render_html()
         else:
             return renderer.render_fallback()
+    
+    def _validate_tabs(self, tab_data_list: List[TabData], skip_empty_check: bool = False) -> None:
+        """Validate tab data list - moved from parsers.py"""
+        if not tab_data_list and not skip_empty_check:
+            raise ValueError(
+                "No tab directives found inside filter-tabs. "
+                "Add at least one .. tab:: directive."
+            )
+        names = []
+        for tab in tab_data_list:
+            name = tab.name
+            if name in names:
+                raise ValueError(
+                    f"Duplicate tab name '{name}'. Each tab must have a unique name."
+                )
+            names.append(name)
+            if not tab.content:
+                logger.warning(f"Tab '{name}' has no content.")    
+        
+        default_count = 0
+        default_names = []
+        for tab in tab_data_list:
+            if tab.is_default:
+                default_count += 1
+                default_names.append(tab.name)
+        
+        if default_count > 1:
+            logger.warning(
+                f"Multiple tabs marked as default: {', '.join(default_names)}. "
+                f"Using first default: '{default_names[0]}'"
+            )
+
 
+# =============================================================================
+# Collapsible Admonitions Support
+# =============================================================================
 
 def setup_collapsible_admonitions(app: Sphinx, doctree: nodes.document, docname: str):
-    """
-    Finds any admonition with the `:class: collapsible` option and transforms it
-    into an HTML `<details>`/`<summary>` element for a native collapsible effect.
-    """
-    if not app.config.filter_tabs_collapsible_enabled or app.builder.name != 'html':
+    """Convert admonitions with 'collapsible' class to details/summary elements."""
+    # Simplified: always enabled for HTML builds, no config needed
+    if app.builder.name != 'html':
         return
-
+        
     for node in list(doctree.findall(nodes.admonition)):
         if 'collapsible' not in node.get('classes', []):
             continue
-
+            
         is_expanded = 'expanded' in node.get('classes', [])
         title_node = next(iter(node.findall(nodes.title)), None)
         summary_text = title_node.astext() if title_node else "Details"
+        
         if title_node:
             title_node.parent.remove(title_node)
 
-        # Create the new <details> node.
         details_node = DetailsNode(classes=[COLLAPSIBLE_SECTION])
         if is_expanded:
             details_node['open'] = 'open'
-
-        # Create the new <summary> node with a custom arrow.
+        
         summary_node = SummaryNode()
         arrow_span = nodes.inline(classes=[CUSTOM_ARROW])
         arrow_span += nodes.Text("▶")
         summary_node += arrow_span
         summary_node += nodes.Text(summary_text)
         details_node += summary_node
-
-        # Move the original content of the admonition into a new container.
+        
         content_node = nodes.container(classes=[COLLAPSIBLE_CONTENT])
         content_node.extend(copy.deepcopy(node.children))
         details_node += content_node
-        # Replace the original admonition node with the new details node.
+        
         node.replace_self(details_node)
 
+
+# =============================================================================
+# HTML Visitor Functions
+# =============================================================================
+
 def _get_html_attrs(node: nodes.Element) -> Dict[str, Any]:
-    """Helper to get a clean dictionary of HTML attributes from a docutils node."""
+    """Extract HTML attributes from a docutils node, excluding internal attributes."""
     attrs = node.attributes.copy()
-    # Remove docutils-internal attributes to avoid rendering them in the HTML.
     for key in ('ids', 'backrefs', 'dupnames', 'names', 'classes', 'id', 'for_id'):
         attrs.pop(key, None)
     return attrs
 
-# --- HTML Visitor Functions ---
+
 def visit_container_node(self: HTML5Translator, node: ContainerNode) -> None:
     self.body.append(self.starttag(node, 'div', **_get_html_attrs(node)))
+
 def depart_container_node(self: HTML5Translator, node: ContainerNode) -> None:
     self.body.append('</div>')
 
 def visit_fieldset_node(self: HTML5Translator, node: FieldsetNode) -> None:
-    self.body.append(self.starttag(node, 'fieldset', CLASS=SFT_FIELDSET))
+    attrs = _get_html_attrs(node)
+    if 'role' in node.attributes:
+        attrs['role'] = node['role']
+    self.body.append(self.starttag(node, 'fieldset', CLASS="sft-fieldset", **attrs))
+
 def depart_fieldset_node(self: HTML5Translator, node: FieldsetNode) -> None:
     self.body.append('</fieldset>')
 
 def visit_legend_node(self: HTML5Translator, node: LegendNode) -> None:
-    self.body.append(self.starttag(node, 'legend', CLASS=SFT_LEGEND))
+    self.body.append(self.starttag(node, 'legend', CLASS="sft-legend"))
+
 def depart_legend_node(self: HTML5Translator, node: LegendNode) -> None:
     self.body.append('</legend>')
 
 def visit_radio_input_node(self: HTML5Translator, node: RadioInputNode) -> None:
     attrs = _get_html_attrs(node)
-    # Don't manually add 'id' - starttag handles 'ids' automatically
-    # Include other important attributes
-    for key in ['type', 'name', 'checked']:
+    for key in ['type', 'name', 'checked', 'aria-label']:
         if key in node.attributes:
             attrs[key] = node[key]
     self.body.append(self.starttag(node, 'input', **attrs))
 
 def depart_radio_input_node(self: HTML5Translator, node: RadioInputNode) -> None:
-    pass
+    pass  # Self-closing tag
 
 def visit_label_node(self: HTML5Translator, node: LabelNode) -> None:
     attrs = _get_html_attrs(node)
-    # Ensure the 'for' attribute is set correctly
     if 'for_id' in node.attributes:
         attrs['for'] = node['for_id']
-    # FIX: Don't add any ARIA attributes or IDs to labels
     self.body.append(self.starttag(node, 'label', **attrs))
 
 def depart_label_node(self: HTML5Translator, node: LabelNode) -> None:
@@ -382,73 +395,70 @@ def depart_label_node(self: HTML5Translator, node: LabelNode) -> None:
 
 def visit_panel_node(self: HTML5Translator, node: PanelNode) -> None:
     attrs = _get_html_attrs(node)
-    # Panels can have ARIA attributes
     for key in ['role', 'aria-labelledby', 'tabindex']:
         if key in node.attributes:
             attrs[key] = node[key]
-    # Handle data attributes
-    if 'data-filter' in node.attributes:
-        attrs['data-filter'] = node['data-filter']
-    self.body.append(self.starttag(node, 'div', CLASS=SFT_PANEL, **attrs))
+    self.body.append(self.starttag(node, 'div', CLASS="sft-panel", **attrs))
 
 def depart_panel_node(self: HTML5Translator, node: PanelNode) -> None:
     self.body.append('</div>')
 
 def visit_details_node(self: HTML5Translator, node: DetailsNode) -> None:
-    attrs = {}
+    attrs = _get_html_attrs(node)
     if 'open' in node.attributes:
-        attrs['open'] = node['open']
+        attrs['open'] = 'open'
     self.body.append(self.starttag(node, 'details', **attrs))
 
 def depart_details_node(self: HTML5Translator, node: DetailsNode) -> None:
     self.body.append('</details>')
 
 def visit_summary_node(self: HTML5Translator, node: SummaryNode) -> None:
-    self.body.append(self.starttag(node, 'summary'))
+    self.body.append(self.starttag(node, 'summary', **_get_html_attrs(node)))
 
 def depart_summary_node(self: HTML5Translator, node: SummaryNode) -> None:
     self.body.append('</summary>')
 
 
+# =============================================================================
+# Static File Handling
+# =============================================================================
+
 def copy_static_files(app: Sphinx):
-    """
-    Copies the extension's static CSS and JS files to the build output directory.
-    """
+    """Copy CSS and JS files to the build directory."""
     if app.builder.name != 'html':
         return
-    
+        
     static_source_dir = Path(__file__).parent / "static"
     dest_dir = Path(app.outdir) / "_static"
     dest_dir.mkdir(parents=True, exist_ok=True)
     
-    # Copy both CSS and JS files
-    for file_pattern in ["*.css", "*.js"]:
-        for file_path in static_source_dir.glob(file_pattern):
-            shutil.copy(file_path, dest_dir)
+    # Only copy the consolidated CSS file
+    css_file = static_source_dir / "filter_tabs.css"
+    if css_file.exists():
+        shutil.copy(css_file, dest_dir)
+    
+    # Copy JS file if it exists
+    js_file = static_source_dir / "filter_tabs.js"
+    if js_file.exists():
+        shutil.copy(js_file, dest_dir)
+
 
+# =============================================================================
+# Extension Setup
+# =============================================================================
 
 def setup(app: Sphinx) -> Dict[str, Any]:
-    """
-    The main entry point for the Sphinx extension.
-    """
-    # Register custom configuration values
-    app.add_config_value('filter_tabs_tab_highlight_color', '#007bff', 'html', [str])
-    app.add_config_value('filter_tabs_tab_background_color', '#f0f0f0', 'html', [str])
-    app.add_config_value('filter_tabs_tab_font_size', '1em', 'html', [str])
-    app.add_config_value('filter_tabs_border_radius', '8px', 'html', [str])
+    """Setup the Sphinx extension with minimal configuration."""
+    
+    # ONLY essential configuration options (down from 9 to 2)
+    app.add_config_value('filter_tabs_highlight_color', '#007bff', 'html', [str])
     app.add_config_value('filter_tabs_debug_mode', False, 'html', [bool])
-    app.add_config_value('filter_tabs_collapsible_enabled', True, 'html', [bool])
-    app.add_config_value('filter_tabs_collapsible_accent_color', '#17a2b8', 'html', [str])
     
-    # NEW: Add accessibility configuration options
-    app.add_config_value('filter_tabs_keyboard_navigation', True, 'html', [bool])
-    app.add_config_value('filter_tabs_announce_changes', True, 'html', [bool])
-
-    # Add the main stylesheet and JavaScript to the HTML output.
+    # Add static files
     app.add_css_file('filter_tabs.css')
     app.add_js_file('filter_tabs.js')
-
-    # Register all custom nodes and their HTML visitor/depart functions.
+    
+    # Register custom nodes (keep existing node registration code)
     app.add_node(ContainerNode, html=(visit_container_node, depart_container_node))
     app.add_node(FieldsetNode, html=(visit_fieldset_node, depart_fieldset_node))
     app.add_node(LegendNode, html=(visit_legend_node, depart_legend_node))
@@ -457,15 +467,15 @@ def setup(app: Sphinx) -> Dict[str, Any]:
     app.add_node(PanelNode, html=(visit_panel_node, depart_panel_node))
     app.add_node(DetailsNode, html=(visit_details_node, depart_details_node))
     app.add_node(SummaryNode, html=(visit_summary_node, depart_summary_node))
-
-    # Register the RST directives.
+    
+    # Register directives
     app.add_directive('filter-tabs', FilterTabsDirective)
     app.add_directive('tab', TabDirective)
-
-    # Connect to Sphinx events
-    app.connect('doctree-resolved', setup_collapsible_admonitions)
+    
+    # Connect event handlers
     app.connect('builder-inited', copy_static_files)
-
+    app.connect('doctree-resolved', setup_collapsible_admonitions)
+    
     return {
         'version': __version__,
         'parallel_read_safe': True,