Skip to content

Commit d476925

Browse files
authored
fix: Re-add agenda day navigation targets to right-hand nav (ietf-tools#3723)
Fixes ietf-tools#3698
1 parent fd2adc8 commit d476925

2 files changed

Lines changed: 9 additions & 8 deletions

File tree

ietf/static/js/ietf.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -110,15 +110,15 @@ $(function () {
110110
const contentElement = $('#content.ietf-auto-nav');
111111
if (contentElement.length > 0) {
112112
const headings = contentElement
113-
.find("h1:visible, h2:visible, h3:visible, h4:visible, h5:visible, h6:visible")
113+
.find("h1:visible, h2:visible, h3:visible, h4:visible, h5:visible, h6:visible, .nav-heading:visible")
114114
.not(".navskip");
115115

116116
const contents = (headings.length > 0) &&
117-
($(headings)
118-
.html()
119-
.split("<")
120-
.shift()
121-
.trim());
117+
($(headings)
118+
.html()
119+
.split("<")
120+
.shift()
121+
.trim());
122122

123123
if (
124124
contents &&
@@ -143,7 +143,7 @@ $(function () {
143143
</nav>
144144
</div>
145145
`))
146-
.find("h1:visible, h2:visible, h3:visible, h4:visible, h5:visible, h6:visible")
146+
.find("h1:visible, h2:visible, h3:visible, h4:visible, h5:visible, h6:visible, .nav-heading:visible")
147147
.not(".navskip")
148148
.each(function () {
149149
// Some headings have complex HTML in them - only use first part in that case.

ietf/templates/meeting/agenda.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,8 @@ <h2 class="mt-3">
110110
{% ifchanged item.timeslot.time|date:"Y-m-d" %}
111111
<tr class="table-primary show-with-children">
112112
<th colspan="{% if personalize %}6{% else %}5{% endif %}"
113-
id="slot-{{ item.timeslot.time|slugify }}">
113+
id="slot-{{ item.timeslot.time|slugify }}"
114+
class="nav-heading">
114115
{{ item.timeslot.time|date:"l, F j, Y" }}
115116
</th>
116117
</tr>

0 commit comments

Comments
 (0)