Merge pull request #28 from lexintegritastech/improve-main-ui

Refix `sidenav` dropdown accordion

app/components/subjects_sidenav_sub_menu_component/subjects_sidenav_sub_menu_component.html.erb

@@ -13,14 +13,14 @@
             <% end %>
           </div>
 
-          <% if root.children.order(name: :asc).present? %>
+          <% if root.children.present? %>
             <div class="accordion-collapse collapse" id="<%= [opts[:second_sub_menu], root.id].join %>" data-bs-parent="#firstLevelSubMenu">
               <div class="accordion-body pb-0 pe-0 pt-0">
-                <% root.children.each do |sub1| %>
+                <% root.children.order(name: :asc).each do |sub1| %>
                   <div class="accordion accordion-flush ps-20" id="secondLevelSubMenu">
                     <div class="accordion-item bg-transparent">
                       <div class="accordion-header bg-transparent">
-                        <% if sub1.children.order(name: :asc).present? %>
+                        <% if sub1.children.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>