Skip to content

Commit d39c3b1

Browse files
committed
Reduce code change within navs
1 parent 9944bda commit d39c3b1

File tree

1 file changed

+13
-34
lines changed

1 file changed

+13
-34
lines changed

shiny/ui/_navs.py

Lines changed: 13 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,8 @@ def __init__(
395395
footer: TagChild = None,
396396
) -> None:
397397
id_resolved = resolve_id_or_none(id)
398-
selected = restore_input(id_resolved, selected) if id_resolved else selected
398+
selected = restore_input(id_resolved, selected)
399+
399400
self.args = args
400401
self.ul_class = ul_class
401402
self.id = id_resolved
@@ -467,12 +468,10 @@ def navset_tab(
467468
-------
468469
See :func:`~shiny.ui.nav_panel`
469470
"""
470-
id_resolved = resolve_id_or_none(id)
471-
selected = restore_input(id_resolved, selected) if id_resolved else selected
472471
return NavSet(
473472
*args,
474473
ul_class="nav nav-tabs",
475-
id=id_resolved,
474+
id=id,
476475
selected=selected,
477476
header=header,
478477
footer=footer,
@@ -524,12 +523,10 @@ def navset_pill(
524523
-------
525524
See :func:`~shiny.ui.nav_panel`
526525
"""
527-
id_resolved = resolve_id_or_none(id)
528-
selected = restore_input(id_resolved, selected) if id_resolved else selected
529526
return NavSet(
530527
*args,
531528
ul_class="nav nav-pills",
532-
id=id_resolved,
529+
id=id,
533530
selected=selected,
534531
header=header,
535532
footer=footer,
@@ -581,12 +578,10 @@ def navset_underline(
581578
-------
582579
See :func:`~shiny.ui.nav_panel`
583580
"""
584-
id_resolved = resolve_id_or_none(id)
585-
selected = restore_input(id_resolved, selected) if id_resolved else selected
586581
return NavSet(
587582
*args,
588583
ul_class="nav nav-underline",
589-
id=id_resolved,
584+
id=id,
590585
selected=selected,
591586
header=header,
592587
footer=footer,
@@ -634,12 +629,10 @@ def navset_hidden(
634629
* :func:`~shiny.ui.navset_card_underline`
635630
* :func:`~shiny.ui.navset_pill_list`
636631
"""
637-
id_resolved = resolve_id_or_none(id)
638-
selected = restore_input(id_resolved, selected) if id_resolved else selected
639632
return NavSet(
640633
*args,
641634
ul_class="nav nav-hidden",
642-
id=id_resolved,
635+
id=id,
643636
selected=selected,
644637
header=header,
645638
footer=footer,
@@ -663,12 +656,10 @@ def __init__(
663656
footer: TagChild = None,
664657
placement: Literal["above", "below"] = "above",
665658
) -> None:
666-
id_resolved = resolve_id_or_none(id)
667-
selected = restore_input(id_resolved, selected) if id_resolved else selected
668659
super().__init__(
669660
*args,
670661
ul_class=ul_class,
671-
id=id_resolved,
662+
id=id,
672663
selected=selected,
673664
header=header,
674665
footer=footer,
@@ -757,12 +748,10 @@ def navset_card_tab(
757748
-------
758749
See :func:`~shiny.ui.nav_panel`
759750
"""
760-
id_resolved = resolve_id_or_none(id)
761-
selected = restore_input(id_resolved, selected) if id_resolved else selected
762751
return NavSetCard(
763752
*args,
764753
ul_class="nav nav-tabs card-header-tabs",
765-
id=id_resolved,
754+
id=id,
766755
selected=selected,
767756
title=title,
768757
sidebar=sidebar,
@@ -824,12 +813,10 @@ def navset_card_pill(
824813
-------
825814
See :func:`~shiny.ui.nav_panel`
826815
"""
827-
id_resolved = resolve_id_or_none(id)
828-
selected = restore_input(id_resolved, selected) if id_resolved else selected
829816
return NavSetCard(
830817
*args,
831818
ul_class="nav nav-pills card-header-pills",
832-
id=id_resolved,
819+
id=id,
833820
selected=selected,
834821
title=title,
835822
sidebar=sidebar,
@@ -891,12 +878,10 @@ def navset_card_underline(
891878
-------
892879
See :func:`~shiny.ui.nav_panel`
893880
"""
894-
id_resolved = resolve_id_or_none(id)
895-
selected = restore_input(id_resolved, selected) if id_resolved else selected
896881
return NavSetCard(
897882
*args,
898883
ul_class="nav nav-underline",
899-
id=id_resolved,
884+
id=id,
900885
selected=selected,
901886
title=title,
902887
sidebar=sidebar,
@@ -921,12 +906,10 @@ def __init__(
921906
well: bool = True,
922907
widths: tuple[int, int] = (4, 8),
923908
) -> None:
924-
id_resolved = resolve_id_or_none(id)
925-
selected = restore_input(id_resolved, selected) if id_resolved else selected
926909
super().__init__(
927910
*args,
928911
ul_class=ul_class,
929-
id=id_resolved,
912+
id=id,
930913
selected=selected,
931914
header=header,
932915
footer=footer,
@@ -993,12 +976,10 @@ def navset_pill_list(
993976
-------
994977
See :func:`~shiny.ui.nav_panel`
995978
"""
996-
id_resolved = resolve_id_or_none(id)
997-
selected = restore_input(id_resolved, selected) if id_resolved else selected
998979
return NavSetPillList(
999980
*args,
1000981
ul_class="nav nav-pills nav-stacked",
1001-
id=id_resolved,
982+
id=id,
1002983
selected=selected,
1003984
header=header,
1004985
footer=footer,
@@ -1588,12 +1569,10 @@ def navset_bar(
15881569
ul_class = "nav navbar-nav"
15891570
if navbar_opts.underline:
15901571
ul_class += " nav-underline"
1591-
id_resolved = resolve_id_or_none(id)
1592-
selected = restore_input(id_resolved, selected) if id_resolved else selected
15931572
return NavSetBar(
15941573
*new_args,
15951574
ul_class=ul_class,
1596-
id=id_resolved,
1575+
id=id,
15971576
selected=selected,
15981577
sidebar=sidebar,
15991578
fillable=fillable,

0 commit comments

Comments
 (0)