W kodzie i tak masz użytego JSa, w tagu <script>. Ogólnie rzecz biorąc masz tam strasznie namotane w kodzie i są lepsze wyjścia. Po co używać onclick w HTML jak można dodać addEventListener w JS?
Już sama struktura HTML niepotrzebnie się powtarza, bo w każdym divie miasta masz osobne strzałki (nawigację), a nawigacja powinna być jedna do całości.
P.S.
Widzę, że używasz tam klas z Bootstrapa, to czemu nie wykorzystasz tabów (komponent), które tam już są?