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

Remote bug watches

Bug watches keep track of this bug in other bug trackers.