Now actionsbar.js needs to be included

Bug #1427729 reported by Daniel Holbach
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu HTML5 UI SDK
Fix Released
Medium
Adnane Belmadiaf
ubuntu-html5-theme (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

In RTM the following section in a page will work:

                    <footer id="footer1" data-role="footer" class="revealed">
                        <nav>
                            <ul>
                                <li>
                                    <a href="#" id="addfeed">
                                        <img src="/usr/share/ubuntu-html5-ui-toolkit/0.1//ambiance/img/actions/add.svg" alt="Add feed" />
                                        <span>Add feed</span>
                                    </a>
                                </li>
                            </ul>
                        </nav>
                    </footer>

For vivid, I have to add

    <script src="/usr/share/ubuntu-html5-ui-toolkit/0.1/ambiance/js/actionsbar.js"></script>

Example code: lp:~dholbach/+junk/new-header-toolbar-breakage-testcase

This breaks our HTML5 training materials, so it'd be nice to fix this soon if possible.

Related branches

David Barth (dbarth)
Changed in ubuntu-html5-theme:
assignee: nobody → Alexandre Abreu (abreu-alexandre)
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

This was a mistake in the way the new header branch is laid out. This is an API breakage and IMO actionbars.js should not exist. Assigning it to daker

Changed in ubuntu-html5-theme:
assignee: Alexandre Abreu (abreu-alexandre) → Adnane Belmadiaf (daker)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-html5-theme - 0.1.2+15.04.20150304-0ubuntu1

---------------
ubuntu-html5-theme (0.1.2+15.04.20150304-0ubuntu1) vivid; urgency=medium

  [ Alexandre Abreu ]
  * revert revision 196: push to trunk instead of usual landing

  [ CI Train Bot ]
  * New rebuild forced.

  [ David Barth ]
  * Fixes Uncaught ReferenceError: ActionBar is not defined

  [ daker ]
  * Removed ActionsBar deps Fixed the back button in PageStack Fixed
    PageStack actions (LP: #1427729, #1427909)
 -- CI Train Bot <email address hidden> Wed, 04 Mar 2015 21:32:05 +0000

Changed in ubuntu-html5-theme (Ubuntu):
status: New → Fix Released
David Barth (dbarth)
Changed in ubuntu-html5-theme:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers