rdoc 6.14.2 → 7.1.0

data/lib/rdoc/generator/template/darkfish/_sidebar_search.rhtml

@@ -4,6 +4,7 @@
       <input id="search-field" role="combobox" aria-label="Search"
              aria-autocomplete="list" aria-controls="search-results"
              type="text" name="search" placeholder="Search (/) for a class, method, ..." spellcheck="false"
+             autocomplete="off"
              title="Type to search, Up and Down to navigate, Enter to load">
     </div>
 

data/lib/rdoc/generator/template/darkfish/_sidebar_sections.rhtml

@@ -3,9 +3,9 @@
   <h3>Sections</h3>
 
   <ul class="link-list" role="directory">
-    <%- klass.sort_sections.each do |section| -%>
+    <%- klass.sort_sections.each do |section| %>
       <li><a href="#<%= section.aref %>"><%= h section.title %></a></li>
-    <%- end -%>
+    <%- end %>
   </ul>
 </div>
-<%- end -%>
+<%- end %>

data/lib/rdoc/generator/template/darkfish/_sidebar_table_of_contents.rhtml

@@ -9,31 +9,31 @@
 <div class="nav-section">
   <h3>Table of Contents</h3>
 
-    <%- display_link = proc do |heading| -%>
+    <%- display_link = proc do |heading| %>
       <a href="#<%= heading.label current %>"><%= heading.plain_html %></a>
-    <%- end -%>
+    <%- end %>
 
-    <%- list_siblings = proc do -%>
-      <%- level = table.first&.level -%>
-      <%- while table.first && table.first.level >= level -%>
-        <%- heading = table.shift -%>
-        <%- if table.first.nil? || table.first.level <= heading.level -%>
-          <li><% display_link.call heading -%>
-        <%- else -%>
+    <%- list_siblings = proc do %>
+      <%- level = table.first&.level %>
+      <%- while table.first && table.first.level >= level %>
+        <%- heading = table.shift %>
+        <%- if table.first.nil? || table.first.level <= heading.level %>
+          <li><% display_link.call heading %></li>
+        <%- else %>
           <li>
             <details open>
-              <summary><%- display_link.call heading -%></summary>
+              <summary><%- display_link.call heading %></summary>
               <ul class="link-list" role="directory">
                 <% list_siblings.call %>
               </ul>
             </details>
           </li>
-        <%- end -%>
-      <%- end -%>
-    <%- end -%>
+        <%- end %>
+      <%- end %>
+    <%- end %>
 
   <ul class="link-list" role="directory">
     <% list_siblings.call %>
   </ul>
 </div>
-<%- end -%>
+<%- end %>

data/lib/rdoc/generator/template/darkfish/class.rhtml

@@ -17,10 +17,10 @@
   <%= render '_footer.rhtml' %>
 </nav>
 
-<main role="main" aria-labelledby="<%=h klass.aref %>">
+<main role="main" aria-labelledby="<%= h klass.aref %>">
   <%# If nesting level is 1, breadcrumb list is not needed %>
   <% if breadcrumb.size > 1 %>
-    <ol role="navigation" aria-label="breadcrumb" class="breadcrumb">
+    <ol role="navigation" aria-label="Breadcrumb navigation" class="breadcrumb">
       <% breadcrumb.each do |namespace| %>
       <li>
         <% if namespace[:self] %>
@@ -33,7 +33,8 @@
     </ol>
   <% end %>
 
-  <h1 id="<%=h klass.aref %>" class="anchor-link <%= klass.type %>">
+  <span id="<%= h klass.legacy_aref %>" class="legacy-anchor"></span>
+  <h1 id="<%= h klass.aref %>" class="anchor-link <%= klass.type %>">
     <%= klass.type %> <%= klass.full_name %>
   </h1>
 
@@ -41,9 +42,10 @@
     <%= klass.description %>
   </section>
 
-  <%- klass.each_section do |section, constants, attributes| -%>
+  <%- klass.each_section do |section, constants, attributes| %>
+  <span id="<%= section.legacy_aref %>" class="legacy-anchor"></span>
   <section id="<%= section.aref %>" class="documentation-section anchor-link">
-    <%- if section.title then -%>
+    <%- if section.title then %>
     <header class="documentation-section-title">
       <h2>
         <%= section.title %>
@@ -52,45 +54,46 @@
         <a href="#top">&uarr; top</a>
       </span>
     </header>
-    <%- end -%>
+    <%- end %>
 
-    <%- if section.comment then -%>
+    <%- if section.comment then %>
     <div>
       <%= section.description %>
     </div>
-    <%- end -%>
+    <%- end %>
 
-    <%- unless constants.empty? then -%>
+    <%- unless constants.empty? then %>
     <section class="constants-list">
       <header>
         <h3>Constants</h3>
       </header>
       <dl>
-      <%- constants.each do |const| -%>
-        <dt id="<%= const.name %>"><%= const.name %>
-        <%- if const.comment then -%>
+      <%- constants.each do |const| %>
+        <dt id="<%= const.name %>"><%= const.name %></dt>
+        <%- if const.comment then %>
         <dd>
-          <%- if const.mixin_from then -%>
+          <%- if const.mixin_from then %>
             <div class="mixin-from">
-              Included from <a href="<%= klass.aref_to(const.mixin_from.path)%>"><%= const.mixin_from.full_name %></a>
+              Included from <a href="<%= klass.aref_to(const.mixin_from.path) %>"><%= const.mixin_from.full_name %></a>
             </div>
-          <%- end -%>
+          <%- end %>
           <%= const.description.strip %>
-        <%- else -%>
-        <dd class="missing-docs">(Not documented)
-        <%- end -%>
-      <%- end -%>
+        </dd>
+        <%- else %>
+        <dd class="missing-docs">(Not documented)</dd>
+        <%- end %>
+      <%- end %>
       </dl>
     </section>
-    <%- end -%>
+    <%- end %>
 
-    <%- unless attributes.empty? then -%>
-    <section class="attribute-method-details" class="method-section">
+    <%- unless attributes.empty? then %>
+    <section class="attribute-method-details method-section">
       <header>
         <h3>Attributes</h3>
       </header>
 
-      <%- attributes.each do |attrib| -%>
+      <%- attributes.each do |attrib| %>
       <div id="<%= attrib.aref %>" class="method-detail anchor-link">
         <div class="method-heading attribute-method-heading">
           <a href="#<%= attrib.aref %>" title="Link to this attribute">
@@ -100,36 +103,36 @@
         </div>
 
         <div class="method-description">
-        <%- if attrib.mixin_from then -%>
+        <%- if attrib.mixin_from then %>
           <div class="mixin-from">
-            <%= attrib.singleton ? "Extended" : "Included" %> from <a href="<%= klass.aref_to(attrib.mixin_from.path)%>"><%= attrib.mixin_from.full_name %></a>
+            <%= attrib.singleton ? "Extended" : "Included" %> from <a href="<%= klass.aref_to(attrib.mixin_from.path) %>"><%= attrib.mixin_from.full_name %></a>
           </div>
-        <%- end -%>
-        <%- if attrib.comment then -%>
+        <%- end %>
+        <%- if attrib.comment then %>
         <%= attrib.description.strip %>
-        <%- else -%>
-        <p class="missing-docs">(Not documented)
-        <%- end -%>
+        <%- else %>
+        <p class="missing-docs">(Not documented)</p>
+        <%- end %>
         </div>
       </div>
-      <%- end -%>
+      <%- end %>
     </section>
-    <%- end -%>
+    <%- end %>
 
-    <%- klass.methods_by_type(section).each do |type, visibilities|
-       next if visibilities.empty?
-       visibilities.each do |visibility, methods|
-         next if methods.empty? %>
+    <%- klass.methods_by_type(section).each do |type, visibilities| %>
+    <%- next if visibilities.empty? %>
+    <%- visibilities.each do |visibility, methods| %>
+    <%- next if methods.empty? %>
      <section id="<%= visibility %>-<%= type %>-<%= section.aref %>-method-details" class="method-section anchor-link">
        <header>
          <h3><%= visibility.to_s.capitalize %> <%= type.capitalize %> Methods</h3>
        </header>
 
-    <%- methods.each do |method| -%>
+    <%- methods.each do |method| %>
       <div id="<%= method.aref %>" class="method-detail anchor-link <%= method.is_alias_for ? "method-alias" : '' %>">
         <div class="method-header">
-          <%- if (call_seq = method.call_seq) then -%>
-            <%- call_seq.strip.split("\n").each_with_index do |call_seq, i| -%>
+          <%- if (call_seq = method.call_seq) then %>
+            <%- call_seq.strip.split("\n").each_with_index do |call_seq, i| %>
               <div class="method-heading">
                 <a href="#<%= method.aref %>" title="Link to this method">
                   <span class="method-callseq">
@@ -139,49 +142,47 @@
                   </span>
                 </a>
               </div>
-            <%- end -%>
-          <%- elsif method.has_call_seq? then -%>
+            <%- end %>
+          <%- elsif method.has_call_seq? then %>
             <div class="method-heading">
               <a href="#<%= method.aref %>" title="Link to this method">
                 <span class="method-name"><%= h method.name %></span>
               </a>
             </div>
-          <%- else -%>
+          <%- else %>
             <div class="method-heading">
               <a href="#<%= method.aref %>" title="Link to this method">
                 <span class="method-name"><%= h method.name %></span>
                 <span class="method-args"><%= h method.param_seq %></span>
               </a>
             </div>
-          <%- end -%>
+          <%- end %>
         </div>
 
-        <%- if method.token_stream -%>
+        <%- if method.token_stream %>
           <div class="method-controls">
             <details class="method-source-toggle">
               <summary>Source</summary>
             </details>
           </div>
-        <%- end -%>
-
-        <%- unless method.skip_description? then -%>
-        <div class="method-description">
-          <%- if method.token_stream then -%>
           <div class="method-source-code" id="<%= method.html_name %>-source">
             <pre><%= method.markup_code %></pre>
           </div>
-          <%- end -%>
-          <%- if method.mixin_from then -%>
+        <%- end %>
+
+        <%- unless method.skip_description? then %>
+        <div class="method-description">
+          <%- if method.mixin_from then %>
             <div class="mixin-from">
-              <%= method.singleton ? "Extended" : "Included" %> from <a href="<%= klass.aref_to(method.mixin_from.path)%>"><%= method.mixin_from.full_name %></a>
+              <%= method.singleton ? "Extended" : "Included" %> from <a href="<%= klass.aref_to(method.mixin_from.path) %>"><%= method.mixin_from.full_name %></a>
             </div>
-          <%- end -%>
-          <%- if method.comment then -%>
+          <%- end %>
+          <%- if method.comment then %>
           <%= method.description.strip %>
-          <%- else -%>
-          <p class="missing-docs">(Not documented)
-          <%- end -%>
-          <%- if method.calls_super then -%>
+          <%- else %>
+          <p class="missing-docs">(Not documented)</p>
+          <%- end %>
+          <%- if method.calls_super then %>
             <div class="method-calls-super">
               Calls superclass method
               <%=
@@ -189,11 +190,11 @@
                   method.formatter.link(method.superclass_method.full_name, method.superclass_method.full_name) : nil
               %>
             </div>
-          <%- end -%>
+          <%- end %>
         </div>
-        <%- end -%>
+        <%- end %>
 
-        <%- unless method.aliases.empty? then -%>
+        <%- unless method.aliases.empty? then %>
         <div class="aliases">
           Also aliased as: <%= method.aliases.map do |aka|
             if aka.parent then # HACK lib/rexml/encodings
@@ -203,19 +204,20 @@
             end
           end.join ", " %>
         </div>
-        <%- end -%>
+        <%- end %>
 
-        <%- if method.is_alias_for then -%>
+        <%- if method.is_alias_for then %>
         <div class="aliases">
           Alias for: <a href="<%= klass.aref_to method.is_alias_for.path %>"><%= h method.is_alias_for.name %></a>
         </div>
-        <%- end -%>
+        <%- end %>
       </div>
 
-    <%- end -%>
+    <%- end %>
     </section>
-  <%- end
-     end %>
+  <%- end %>
+  <%- end %>
   </section>
-<%- end -%>
+<%- end %>
 </main>
+</body>

data/lib/rdoc/generator/template/darkfish/css/rdoc.css

@@ -92,6 +92,25 @@ main .anchor-link:target {
   scroll-margin-top: 1rem;
 }
 
+/* Legacy anchor for backward compatibility with old label- prefix links */
+.legacy-anchor {
+  display: block;
+  position: relative;
+  visibility: hidden;
+  scroll-margin-top: 1rem;
+}
+
+/* When a legacy anchor is targeted, highlight the next heading sibling */
+.legacy-anchor:target + h1,
+.legacy-anchor:target + h2,
+.legacy-anchor:target + h3,
+.legacy-anchor:target + h4,
+.legacy-anchor:target + h5,
+.legacy-anchor:target + h6 {
+  margin-left: -10px;
+  border-left: 10px solid var(--border-color);
+}
+
 /* 4. Links */
 a {
   color: var(--link-color);

data/lib/rdoc/generator/template/darkfish/index.rhtml

@@ -17,7 +17,8 @@
 <main role="main">
 <%- if @main_page %>
   <%= @main_page.description %>
-<%- else -%>
-  <p>This is the API documentation for <%= h @title %>.
-<%- end -%>
+<%- else %>
+  <p>This is the API documentation for <%= h @title %>.</p>
+<%- end %>
 </main>
+</body>

data/lib/rdoc/generator/template/darkfish/js/darkfish.js

@@ -72,10 +72,30 @@ function hookSearch() {
   }
 
   search.select = function(result) {
-    window.location.href = result.firstChild.firstChild.href;
+    var href = result.firstChild.firstChild.href;
+    var query = this.input.value;
+    if (query) {
+      var url = new URL(href, window.location.origin);
+      url.searchParams.set('q', query);
+      url.searchParams.set('nav', '0');
+      href = url.toString();
+    }
+    window.location.href = href;
   }
 
   search.scrollIntoView = search.scrollInWindow;
+
+  // Check for ?q= URL parameter and trigger search automatically
+  if (typeof URLSearchParams !== 'undefined') {
+    var urlParams = new URLSearchParams(window.location.search);
+    var queryParam = urlParams.get('q');
+    if (queryParam) {
+      var navParam = urlParams.get('nav');
+      var autoSelect = navParam !== '0';
+      input.value = queryParam;
+      search.search(queryParam, autoSelect);
+    }
+  }
 };
 
 function hookFocus() {

data/lib/rdoc/generator/template/darkfish/js/search.js

@@ -34,6 +34,8 @@ Search.prototype = Object.assign({}, Navigation, new function() {
   }
 
   this.search = function(value, selectFirstMatch) {
+    this.selectFirstMatch = selectFirstMatch;
+
     value = value.trim().toLowerCase();
     if (value) {
       this.setNavigationActive(true);
@@ -76,7 +78,15 @@ Search.prototype = Object.assign({}, Navigation, new function() {
     //TODO: ECMAScript
     //if (jQuery.browser.msie) this.$element[0].className += '';
 
-    if (isLast) this.result.setAttribute('aria-busy', 'false');
+    if (this.selectFirstMatch && this.current) {
+      this.selectFirstMatch = false;
+      this.select(this.current);
+    }
+
+    if (isLast) {
+      this.selectFirstMatch = false;
+      this.result.setAttribute('aria-busy', 'false');
+    }
   }
 
   this.move = function(isDown) {

data/lib/rdoc/generator/template/darkfish/page.rhtml

@@ -13,6 +13,7 @@
   <%= render '_footer.rhtml' %>
 </nav>
 
-<main role="main" aria-label="Page <%=h file.full_name%>">
+<main role="main" aria-label="Page <%= h file.full_name %>">
 <%= file.description %>
 </main>
+</body>

data/lib/rdoc/generator/template/darkfish/servlet_not_found.rhtml

@@ -16,5 +16,6 @@
 <main role="main">
   <h1>Not Found</h1>
 
-  <p><%= message %>
+  <p><%= message %></p>
 </main>
+</body>

data/lib/rdoc/generator/template/darkfish/servlet_root.rhtml

@@ -20,46 +20,46 @@
   <h1>Local RDoc Documentation</h1>
 
   <p>Here you can browse local documentation from the ruby standard library and
-  your installed gems.
+  your installed gems.</p>
 
-<%- extra_dirs = installed.select { |_, _, _, type,| type == :extra } -%>
-<%- unless extra_dirs.empty? -%>
+<%- extra_dirs = installed.select { |_, _, _, type,| type == :extra } %>
+<%- unless extra_dirs.empty? %>
   <h2>Extra Documentation Directories</h2>
 
   <p>The following additional documentation directories are available:</p>
 
   <ol>
-  <%- extra_dirs.each do |name, href, exists, _, path| -%>
+  <%- extra_dirs.each do |name, href, exists, _, path| %>
     <li>
-    <%- if exists -%>
+    <%- if exists %>
       <a href="<%= href %>"><%= h name %></a> (<%= h path %>)
-    <%- else -%>
+    <%- else %>
       <%= h name %> (<%= h path %>; <i>not available</i>)
-    <%- end -%>
+    <%- end %>
     </li>
-  <%- end -%>
+  <%- end %>
   </ol>
-<%- end -%>
+<%- end %>
 
-<%- gems = installed.select { |_, _, _, type,| type == :gem } -%>
-<%- missing = gems.reject { |_, _, exists,| exists } -%>
-<%- unless missing.empty? then -%>
+<%- gems = installed.select { |_, _, _, type,| type == :gem } %>
+<%- missing = gems.reject { |_, _, exists,| exists } %>
+<%- unless missing.empty? %>
   <h2>Missing Gem Documentation</h2>
 
   <p>You are missing documentation for some of your installed gems.
   You can install missing documentation for gems by running
   <kbd>gem rdoc --all</kbd>.  After installing the missing documentation you
   only need to reload this page.  The newly created documentation will
-  automatically appear.
+  automatically appear.</p>
 
   <p>You can also install documentation for a specific gem by running one of
-  the following commands.
+  the following commands.</p>
 
   <ul>
-  <%- names = missing.map { |name,| name.sub(/-([^-]*)$/, '') }.uniq -%>
-  <%- names.each do |name| -%>
-    <li><kbd>gem rdoc <%=h name %></kbd>
-  <%- end -%>
+  <%- missing.map { |name,| name.sub(/-([^-]*)$/, '') }.uniq.each do |name| %>
+    <li><kbd>gem rdoc <%= h name %></kbd></li>
+  <%- end %>
   </ul>
-<%- end -%>
+<%- end %>
 </main>
+</body>

data/lib/rdoc/generator/template/darkfish/table_of_contents.rhtml

@@ -13,11 +13,11 @@
 <main role="main">
 <h1 class="class"><%= h @title %></h1>
 
-<%- simple_files = @files.select { |f| f.text? } -%>
-<%- unless simple_files.empty? then -%>
+<%- simple_files = @files.select { |f| f.text? } %>
+<%- unless simple_files.empty? then %>
 <h2 id="pages">Pages</h2>
 <ul>
-<%- simple_files.sort.each do |file| -%>
+<%- simple_files.sort.each do |file| %>
   <li class="file">
     <a href="<%= h file.path %>"><%= h file.page_name %></a>
 <%
@@ -25,19 +25,19 @@
    table = file.parse(file.comment).table_of_contents
    unless table.empty? then %>
     <ul>
-<%- table.each do |heading| -%>
-      <li><a href="<%= h file.path %>#<%= heading.aref %>"><%= heading.plain_html %></a>
-<%-   end -%>
+    <%- table.each do |heading| %>
+      <li><a href="<%= h file.path %>#<%= heading.aref %>"><%= heading.plain_html %></a></li>
+    <%- end %>
     </ul>
-<%- end -%>
+<%- end %>
   </li>
-  <%- end -%>
+  <%- end %>
 </ul>
-<%- end -%>
+<%- end %>
 
 <h2 id="classes">Classes and Modules</h2>
 <ul>
-<%- @modsort.each do |klass| -%>
+<%- @modsort.each do |klass| %>
   <li class="<%= klass.type %>">
     <a href="<%= klass.path %>"><%= klass.full_name %></a>
 <%- table = []
@@ -46,14 +46,14 @@
 
    unless table.empty? then %>
     <ul>
-<%- table.each do |item| -%>
-<%-   label = item.respond_to?(:label) ? item.label(klass) : item.aref -%>
-      <li><a href="<%= klass.path %>#<%= label %>"><%= item.plain_html %></a>
-<%- end -%>
+<%- table.each do |item| %>
+  <%- label = item.respond_to?(:label) ? item.label(klass) : item.aref %>
+      <li><a href="<%= klass.path %>#<%= label %>"><%= item.plain_html %></a></li>
+<%- end %>
     </ul>
-<%- end -%>
+<%- end %>
   </li>
-<%- end -%>
+<%- end %>
 </ul>
 
 <h2 id="methods">Methods</h2>
@@ -65,6 +65,8 @@
     <a href="<%= method.path %>"><%= h method.pretty_name %></a>
     &mdash;
     <span class="container"><%= method.parent.full_name %></span>
-<%- end -%>
+  </li>
+<%- end %>
 </ul>
 </main>
+</body>