Skip to:
Content

BuddyPress.org

Changeset 11973


Ignore:
Timestamp:
04/14/2018 12:53:34 PM (8 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.