Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Forms Library
    • Labels:

      Description

      Open all the fieldsets by default

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Adrian Greeve added a comment -

            [Y] Syntax
            [Y] Output
            [Y] Whitespace
            [-] Language
            [-] Databases
            [*] Testing
            [-] Security
            [-] Documentation
            [Y] Git
            [Y] Sanity check

            Hello Fred,

            The code looks good and works as described.

            I personally am not a huge fan of the test instructions. Maybe you could remove your course name as this had me confused when I saw it. When I was moving through the navigation tree I didn't realise I was on the correct page until I noticed that the breadcrumbs at the top matched point one of your testing instructions.

            Anyway, nothing serious at all.

            Thanks.

            Show
            Adrian Greeve added a comment - [Y] Syntax [Y] Output [Y] Whitespace [-] Language [-] Databases [*] Testing [-] Security [-] Documentation [Y] Git [Y] Sanity check Hello Fred, The code looks good and works as described. I personally am not a huge fan of the test instructions. Maybe you could remove your course name as this had me confused when I saw it. When I was moving through the navigation tree I didn't realise I was on the correct page until I noticed that the breadcrumbs at the top matched point one of your testing instructions. Anyway, nothing serious at all. Thanks.
            Hide
            Frédéric Massart added a comment -

            Cheers Adrian!

            Show
            Frédéric Massart added a comment - Cheers Adrian!
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated (master only), thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated (master only), thanks!
            Hide
            Andrew Davis added a comment -

            Passing.

            Show
            Andrew Davis added a comment - Passing.
            Hide
            Damyon Wiese added a comment -

            This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).

            Thanks for your contributions!

            Show
            Damyon Wiese added a comment - This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads). Thanks for your contributions!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: