Skip to:
Content

BuddyPress.org

Changeset 11973


Ignore:
Timestamp:
04/14/2018 12:53:34 PM (6 years ago)
Author:
hnla
Message:

Nouveau: Update group_creation_steps function

Commit addresses markup issues in Nouveau function.

Revert group create steps nav function to use BP core function.

This removes the problem of nested echo function & anchor links being set when they are not an active link.

Commit removes redundant Nouveau custom template tag function.

Commit adds class 'count' to spans generating numbers e.g steps count or group/member counts in navs.

Props dcavins, mercime, djpaul, r-a-y

See #7732

Location:
trunk/src/bp-templates/bp-nouveau
Files:
11 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-templates/bp-nouveau/buddypress/common/nav/directory-nav.php

    r11899 r11973  
    2323
    2424                        <?php if ( bp_nouveau_nav_has_count() ) : ?>
    25                             <span><?php bp_nouveau_nav_count(); ?></span>
     25                            <span class="count"><?php bp_nouveau_nav_count(); ?></span>
    2626                        <?php endif; ?>
    2727                    </a>

  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/create.php

    r11856 r11973  
    2121                <ul class="group-create-buttons button-tabs">
    2222
    23                     <?php bp_nouveau_group_creation_tabs(); ?>
     23                    <?php bp_group_creation_tabs(); ?>
    2424
    2525                </ul>

  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/single/parts/admin-subnav.php

    r11899 r11973  
    2323
    2424                        <?php if ( bp_nouveau_nav_has_count() ) : ?>
    25                             <span><?php bp_nouveau_nav_count(); ?></span>
     25                            <span class="count"><?php bp_nouveau_nav_count(); ?></span>
    2626                        <?php endif; ?>
    2727                    </a>

  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/single/parts/item-nav.php

    r11899 r11973  
    2323
    2424                        <?php if ( bp_nouveau_nav_has_count() ) : ?>
    25                             <span><?php bp_nouveau_nav_count(); ?></span>
     25                            <span class="count"><?php bp_nouveau_nav_count(); ?></span>
    2626                        <?php endif; ?>
    2727                    </a>

  • trunk/src/bp-templates/bp-nouveau/buddypress/members/single/parts/item-nav.php

    r11899 r11973  
    2323
    2424                        <?php if ( bp_nouveau_nav_has_count() ) : ?>
    25                             <span><?php bp_nouveau_nav_count(); ?></span>
     25                            <span class="count"><?php bp_nouveau_nav_count(); ?></span>
    2626                        <?php endif; ?>
    2727                    </a>

  • trunk/src/bp-templates/bp-nouveau/buddypress/members/single/parts/item-subnav.php

    r11899 r11973  
    1919
    2020                <?php if ( bp_nouveau_nav_has_count() ) : ?>
    21                     <span><?php bp_nouveau_nav_count(); ?></span>
     21                    <span class="count"><?php bp_nouveau_nav_count(); ?></span>
    2222                <?php endif; ?>
    2323            </a>

  • trunk/src/bp-templates/bp-nouveau/common-styles/_bp-mixins.scss

    r11906 r11973  
    374374                border-width: 1px;
    375375                margin-bottom: -1px;
     376                margin-top: 1px;
    376377                padding: 0 $pad-sml;
    377378

  • trunk/src/bp-templates/bp-nouveau/common-styles/_bp_navigation.scss

    r11919 r11973  
    3333            }
    3434
    35             span {
     35            .count {
    3636                background: $light-grey;
    3737                border: 1px solid $bp-border-dark;
    … …  
    9393            a {
    9494
    95                 span {
     95                .count {
    9696                    background-color: $grey;
    9797                }
    … …  
    118118                color: $off-white;
    119119
    120                 span {
     120                .count {
    121121                    border-color: $off-white;
    122122                }
    … …  
    311311            a {
    312312
    313                 span {
     313                .count {
    314314                    background-color: $white;
    315315                }
    … …  
    323323            a {
    324324
    325                 span {
     325                .count {
    326326                    background-color: $highlight;
    327327                    border: 0;
    … …  
    335335            a:hover {
    336336
    337                 span {
     337                .count {
    338338                    background-color: $highlight;
    339339                    border: 0;
    … …  
    347347            a {
    348348
    349                 span:empty {
     349                .count:empty {
    350350                    display: none;
    351351                }
    … …  
    360360
    361361            li:not(.current) {
    362 
     362                color: $light-text;
    363363                a {
    364364                    color: $light-text;
    … …  
    437437            a {
    438438
    439                 span {
     439                .count {
    440440                    display: inline-block;
    441441                    float: right;
    … …  
    459459            a {
    460460
    461                 span {
     461                .count {
    462462                    float: right;
    463463

  • trunk/src/bp-templates/bp-nouveau/css/buddypress-rtl.css

    r11952 r11973  
    303303}
    304304
    305 .bp-navs ul li span {
     305.bp-navs ul li .count {
    306306    background: #eaeaea;
    307307    border: 1px solid #ccc;
    … …  
    331331}
    332332
    333 .buddypress-wrap .bp-navs li.selected a span,
    334 .buddypress-wrap .bp-navs li.current a span,
    335 .buddypress-wrap .bp-navs li a:hover a span {
     333.buddypress-wrap .bp-navs li.selected a .count,
     334.buddypress-wrap .bp-navs li.current a .count,
     335.buddypress-wrap .bp-navs li a:hover a .count {
    336336    background-color: #ccc;
    337337}
    … …  
    355355}
    356356
    357 .buddypress-wrap .bp-navs li.selected a span,
    358 .buddypress-wrap .bp-navs li.selected a:focus span,
    359 .buddypress-wrap .bp-navs li.selected a:hover span,
    360 .buddypress-wrap .bp-navs li.current a span,
    361 .buddypress-wrap .bp-navs li.current a:focus span,
    362 .buddypress-wrap .bp-navs li.current a:hover span {
     357.buddypress-wrap .bp-navs li.selected a .count,
     358.buddypress-wrap .bp-navs li.selected a:focus .count,
     359.buddypress-wrap .bp-navs li.selected a:hover .count,
     360.buddypress-wrap .bp-navs li.current a .count,
     361.buddypress-wrap .bp-navs li.current a:focus .count,
     362.buddypress-wrap .bp-navs li.current a:hover .count {
    363363    border-color: #fafafa;
    364364}
    … …  
    462462}
    463463
    464 .buddypress_object_nav .bp-navs li.selected a span,
    465 .buddypress_object_nav .bp-navs li.current a span,
    466 .buddypress-wrap .bp-navs li.selected a span,
    467 .buddypress-wrap .bp-navs li.current a span {
     464.buddypress_object_nav .bp-navs li.selected a .count,
     465.buddypress_object_nav .bp-navs li.current a .count,
     466.buddypress-wrap .bp-navs li.selected a .count,
     467.buddypress-wrap .bp-navs li.current a .count {
    468468    background-color: #fff;
    469469}
    470470
    471 .buddypress_object_nav .bp-navs li.dynamic a span,
    472 .buddypress_object_nav .bp-navs li.dynamic.selected a span,
    473 .buddypress_object_nav .bp-navs li.dynamic.current a span,
    474 .buddypress-wrap .bp-navs li.dynamic a span,
    475 .buddypress-wrap .bp-navs li.dynamic.selected a span,
    476 .buddypress-wrap .bp-navs li.dynamic.current a span {
     471.buddypress_object_nav .bp-navs li.dynamic a .count,
     472.buddypress_object_nav .bp-navs li.dynamic.selected a .count,
     473.buddypress_object_nav .bp-navs li.dynamic.current a .count,
     474.buddypress-wrap .bp-navs li.dynamic a .count,
     475.buddypress-wrap .bp-navs li.dynamic.selected a .count,
     476.buddypress-wrap .bp-navs li.dynamic.current a .count {
    477477    background-color: #5087e5;
    478478    border: 0;
    … …  
    480480}
    481481
    482 .buddypress_object_nav .bp-navs li.dynamic a:hover span,
    483 .buddypress-wrap .bp-navs li.dynamic a:hover span {
     482.buddypress_object_nav .bp-navs li.dynamic a:hover .count,
     483.buddypress-wrap .bp-navs li.dynamic a:hover .count {
    484484    background-color: #5087e5;
    485485    border: 0;
    … …  
    487487}
    488488
    489 .buddypress_object_nav .bp-navs li a span:empty,
    490 .buddypress-wrap .bp-navs li a span:empty {
     489.buddypress_object_nav .bp-navs li a .count:empty,
     490.buddypress-wrap .bp-navs li a .count:empty {
    491491    display: none;
     492}
     493
     494.buddypress_object_nav .bp-navs.group-create-links ul li:not(.current),
     495.buddypress-wrap .bp-navs.group-create-links ul li:not(.current) {
     496    color: #767676;
    492497}
    493498
    … …  
    533538}
    534539
    535 .buddypress_object_nav .bp-navs li a span {
     540.buddypress_object_nav .bp-navs li a .count {
    536541    display: inline-block;
    537542    float: left;
    … …  
    542547        background: none;
    543548    }
    544     .bp-dir-vert-nav .bp-navs.dir-navs a span {
     549    .bp-dir-vert-nav .bp-navs.dir-navs a .count {
    545550        float: left;
    546551    }
    … …  
    578583        border-width: 1px;
    579584        margin-bottom: -1px;
     585        margin-top: 1px;
    580586        padding: 0 0.5em;
    581587    }

  • trunk/src/bp-templates/bp-nouveau/css/buddypress.css

    r11952 r11973  
    303303}
    304304
    305 .bp-navs ul li span {
     305.bp-navs ul li .count {
    306306    background: #eaeaea;
    307307    border: 1px solid #ccc;
    … …  
    331331}
    332332
    333 .buddypress-wrap .bp-navs li.selected a span,
    334 .buddypress-wrap .bp-navs li.current a span,
    335 .buddypress-wrap .bp-navs li a:hover a span {
     333.buddypress-wrap .bp-navs li.selected a .count,
     334.buddypress-wrap .bp-navs li.current a .count,
     335.buddypress-wrap .bp-navs li a:hover a .count {
    336336    background-color: #ccc;
    337337}
    … …  
    355355}
    356356
    357 .buddypress-wrap .bp-navs li.selected a span,
    358 .buddypress-wrap .bp-navs li.selected a:focus span,
    359 .buddypress-wrap .bp-navs li.selected a:hover span,
    360 .buddypress-wrap .bp-navs li.current a span,
    361 .buddypress-wrap .bp-navs li.current a:focus span,
    362 .buddypress-wrap .bp-navs li.current a:hover span {
     357.buddypress-wrap .bp-navs li.selected a .count,
     358.buddypress-wrap .bp-navs li.selected a:focus .count,
     359.buddypress-wrap .bp-navs li.selected a:hover .count,
     360.buddypress-wrap .bp-navs li.current a .count,
     361.buddypress-wrap .bp-navs li.current a:focus .count,
     362.buddypress-wrap .bp-navs li.current a:hover .count {
    363363    border-color: #fafafa;
    364364}
    … …  
    462462}
    463463
    464 .buddypress_object_nav .bp-navs li.selected a span,
    465 .buddypress_object_nav .bp-navs li.current a span,
    466 .buddypress-wrap .bp-navs li.selected a span,
    467 .buddypress-wrap .bp-navs li.current a span {
     464.buddypress_object_nav .bp-navs li.selected a .count,
     465.buddypress_object_nav .bp-navs li.current a .count,
     466.buddypress-wrap .bp-navs li.selected a .count,
     467.buddypress-wrap .bp-navs li.current a .count {
    468468    background-color: #fff;
    469469}
    470470
    471 .buddypress_object_nav .bp-navs li.dynamic a span,
    472 .buddypress_object_nav .bp-navs li.dynamic.selected a span,
    473 .buddypress_object_nav .bp-navs li.dynamic.current a span,
    474 .buddypress-wrap .bp-navs li.dynamic a span,
    475 .buddypress-wrap .bp-navs li.dynamic.selected a span,
    476 .buddypress-wrap .bp-navs li.dynamic.current a span {
     471.buddypress_object_nav .bp-navs li.dynamic a .count,
     472.buddypress_object_nav .bp-navs li.dynamic.selected a .count,
     473.buddypress_object_nav .bp-navs li.dynamic.current a .count,
     474.buddypress-wrap .bp-navs li.dynamic a .count,
     475.buddypress-wrap .bp-navs li.dynamic.selected a .count,
     476.buddypress-wrap .bp-navs li.dynamic.current a .count {
    477477    background-color: #5087e5;
    478478    border: 0;
    … …  
    480480}
    481481
    482 .buddypress_object_nav .bp-navs li.dynamic a:hover span,
    483 .buddypress-wrap .bp-navs li.dynamic a:hover span {
     482.buddypress_object_nav .bp-navs li.dynamic a:hover .count,
     483.buddypress-wrap .bp-navs li.dynamic a:hover .count {
    484484    background-color: #5087e5;
    485485    border: 0;
    … …  
    487487}
    488488
    489 .buddypress_object_nav .bp-navs li a span:empty,
    490 .buddypress-wrap .bp-navs li a span:empty {
     489.buddypress_object_nav .bp-navs li a .count:empty,
     490.buddypress-wrap .bp-navs li a .count:empty {
    491491    display: none;
     492}
     493
     494.buddypress_object_nav .bp-navs.group-create-links ul li:not(.current),
     495.buddypress-wrap .bp-navs.group-create-links ul li:not(.current) {
     496    color: #767676;
    492497}
    493498
    … …  
    533538}
    534539
    535 .buddypress_object_nav .bp-navs li a span {
     540.buddypress_object_nav .bp-navs li a .count {
    536541    display: inline-block;
    537542    float: right;
    … …  
    542547        background: none;
    543548    }
    544     .bp-dir-vert-nav .bp-navs.dir-navs a span {
     549    .bp-dir-vert-nav .bp-navs.dir-navs a .count {
    545550        float: right;
    546551    }
    … …  
    578583        border-width: 1px;
    579584        margin-bottom: -1px;
     585        margin-top: 1px;
    580586        padding: 0 0.5em;
    581587    }

  • trunk/src/bp-templates/bp-nouveau/includes/groups/template-tags.php

    r11900 r11973  
    229229                </a>
    230230            <?php else : ?>
    231                 <a disabled="disabled"><?php echo (int) $counter; ?>. <?php echo esc_html( $step['name'] ); ?></a>
     231                <?php echo (int) $counter; ?>. <?php echo esc_html( $step['name'] ); ?>
    232232            <?php endif ?>
    233233        </li>
Note: See TracChangeset for help on using the changeset viewer.