Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: HTML and CSS, Libraries
    • Labels:
    • Testing Instructions:
      Hide

      Having the totara elementlibrary installed helps as it has a page which excercises the tab render in a variety of ways.

      If not then visit anything with tabs, Wiki, Lesson, Quiz editing, Glossary.

      Check

      a) the HTML produced is in line with that documented by Bootstrap:

      http://twitter.github.io/bootstrap/components.html#navs

      b) the visual appearance is consistent with above, and doesn't clash with surroundings in Moodle Bootstrapbase theme

      c) tabs with a second level should render as one row above the other

      d) the tabs should collapse into what Bootstrap calls stackable tabs at screen widths less than 480px

      Show
      Having the totara elementlibrary installed helps as it has a page which excercises the tab render in a variety of ways. If not then visit anything with tabs, Wiki, Lesson, Quiz editing, Glossary. Check a) the HTML produced is in line with that documented by Bootstrap: http://twitter.github.io/bootstrap/components.html#navs b) the visual appearance is consistent with above, and doesn't clash with surroundings in Moodle Bootstrapbase theme c) tabs with a second level should render as one row above the other d) the tabs should collapse into what Bootstrap calls stackable tabs at screen widths less than 480px
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-39388-master

      Description

      The code for generating tabs got moved to a renderer. This is a renderer for the Bootstrapbase theme that generates the standard Bootstrap HTML for tabs as documented here:

      http://twitter.github.io/bootstrap/components.html#navs

      As a result we can also delete some CSS that was being generated to make Bootstrap's standard tab HTML look superficially like Bootstrap.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            marina Marina Glancy added a comment -

            Looks good to me!
            Maybe needs adding phpdocs to the render_ functions (even if it is copy-paste from the parent class)

            Show
            marina Marina Glancy added a comment - Looks good to me! Maybe needs adding phpdocs to the render_ functions (even if it is copy-paste from the parent class)
            Hide
            bawjaws David Scotson added a comment -

            I copied in the docs from the parent class.

            I'm happy for this to go to integration if you're okay with it. I don't have the button to do it myself.

            cheers,

            dave

            Show
            bawjaws David Scotson added a comment - I copied in the docs from the parent class. I'm happy for this to go to integration if you're okay with it. I don't have the button to do it myself. cheers, dave
            Hide
            marina Marina Glancy added a comment -

            Thank you David. Submitting for integration

            Show
            marina Marina Glancy added a comment - Thank you David. Submitting for integration
            Hide
            damyon Damyon Wiese added a comment -

            I just checked and David M says this change wont break the behat tests because:

            a) we are just looking for the links in the tabs and
            b) we are not running the behat tests under bootstrap/clean atm.

            Show
            damyon Damyon Wiese added a comment - I just checked and David M says this change wont break the behat tests because: a) we are just looking for the links in the tabs and b) we are not running the behat tests under bootstrap/clean atm.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks David,

            This needs some work though - it is incorrectly escaping url parameters for the tabs which breaks the links. To reproduce the error - just add a feedback module and click on "Show responses".

            Show
            damyon Damyon Wiese added a comment - Thanks David, This needs some work though - it is incorrectly escaping url parameters for the tabs which breaks the links. To reproduce the error - just add a feedback module and click on "Show responses".
            Hide
            marina Marina Glancy added a comment -

            TO INTEGRATORS: please pick up last commit in
            git://github.com/marinaglancy/moodle.git
            branch
            wip-MDL-39388-master

            Show
            marina Marina Glancy added a comment - TO INTEGRATORS: please pick up last commit in git://github.com/marinaglancy/moodle.git branch wip- MDL-39388 -master
            Hide
            damyon Damyon Wiese added a comment -

            Thanks David and Marina (especially for the quick fix!).

            This has been integrated to master now.

            Show
            damyon Damyon Wiese added a comment - Thanks David and Marina (especially for the quick fix!). This has been integrated to master now.
            Hide
            markn Mark Nelson added a comment -

            One thing I noticed was that in the bootstrap examples the href is set to "#" for active links. I don't think this really matters though and I imagine the anchor tag is only placed around active text for CSS purposes. So,looks good to me. Thanks for your contribution. Passing.

            Show
            markn Mark Nelson added a comment - One thing I noticed was that in the bootstrap examples the href is set to "#" for active links. I don't think this really matters though and I imagine the anchor tag is only placed around active text for CSS purposes. So,looks good to me. Thanks for your contribution. Passing.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Did you think this day was not going to arrive ever?

            Your patience has been rewarded, yay, sent upstream, thanks!

            Closing...ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13