From cdb67d6c3dbcffb996e0542da97b95863d2beeb5 Mon Sep 17 00:00:00 2001
From: alexdbondoc17 <alexdbondoc17@gmail.com>
Date: Wed, 9 Feb 2022 08:12:56 +0000
Subject: [PATCH] Fix `sidenav` dropdown accordion

---
 .../subjects_sidenav_sub_menu_component.html.erb            | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/components/subjects_sidenav_sub_menu_component/subjects_sidenav_sub_menu_component.html.erb b/app/components/subjects_sidenav_sub_menu_component/subjects_sidenav_sub_menu_component.html.erb
index bdc14a0..b9a99a6 100644
--- a/app/components/subjects_sidenav_sub_menu_component/subjects_sidenav_sub_menu_component.html.erb
+++ b/app/components/subjects_sidenav_sub_menu_component/subjects_sidenav_sub_menu_component.html.erb
@@ -20,7 +20,7 @@
                   <div class="accordion accordion-flush ps-20" id="secondLevelSubMenu">
                     <div class="accordion-item bg-transparent">
                       <div class="accordion-header bg-transparent">
-                        <% if sub1.children.present? %>
+                        <% if sub1.children.order(name: :asc).present? %>
                           <button class="<%= sub1.children.present? ? 'accordion-button' : '' %> collapsed bg-transparent" type="button" data-bs-toggle="collapse" data-bs-target="#<%= [opts[:third_sub_menu], sub1.id].join %>">
                             <%= link_to sub1.name, index_url(sub1.id), class: "accordion-link text-white text-decoration-none text-dark d-block sub2 clickable-link" %> 
                           </button>
@@ -42,7 +42,7 @@
                                   <% else %>
                                     <%= link_to sub2.name, index_url(sub2.id), class: "accordion-link text-white text-decoration-none text-dark d-block sub3 clickable-link" %>
                                   <% end %>
-                                
+                                </div>
 
                                 <div class="accordion-collapse collapse" id="<%= [opts[:fourth_sub_menu], sub2.id].join %>" data-bs-parent="#thirdLevelSubMenu">
                                   <div class="accordion-body pb-0 pe-0 pt-0">
@@ -57,7 +57,7 @@
                                             <% else%>
                                               <%= link_to sub3.name, index_url(sub3.id), class: "accordion-link text-white text-decoration-none text-dark d-block sub4 clickable-link" %> 
                                             <% end %>
-                                          
+                                          </div>
 
                                           <div class="accordion-collapse collapse" id="<%= [opts[:fifth_sub_menu], sub3.id].join %>" data-bs-parent="#fourthLevelSubMenu">
                                             <div class="accordion-body pb-0 pe-0 pt-0">