Replace collapsible group and panel directives with the existing directives from angular-bootstrap

Bug #1411636 reported by Timur Sufiev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Merlin
Fix Committed
Wishlist
Unassigned

Bug Description

Angular-bootstrap package provides a number of angular directives that we've reinvented in Merlin. It is quite possible that our implementations are inferior to the angular-bootstrap solutions, so it makes sense to study these solutions and if they fit in our use-cases use them instead. This also would lessen the amount of code in Merlin's directives.

Starting point for the study is http://angular-ui.github.io/bootstrap/#/alert

Timur Sufiev (tsufiev-x)
Changed in merlin:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to merlin (master)

Reviewed: https://review.openstack.org/177809
Committed: https://git.openstack.org/cgit/stackforge/merlin/commit/?id=e5b8fb8a3acc0855ab7486536d9f9eb01a4b22b0
Submitter: Jenkins
Branch: master

commit e5b8fb8a3acc0855ab7486536d9f9eb01a4b22b0
Author: Timur Sufiev <email address hidden>
Date: Mon Apr 27 17:59:38 2015 +0300

    Rewrite <panel> and <collapsible-group> directives

    Use 'collapse' directive from angular-bootstrap inside them instead of
    hand-written bootstrap css transitions. Clicking on panel title no
    longer collapses/expands panel contents - this is done in order enable
    panel entity name in-line editing in next commit.

    Change-Id: Ifcc32cd74a5482a59b417333824522ebf48c73b5
    Closes-Bug: #1411636
    Closes-Bug: #1428719

Changed in merlin:
status: New → Fix Committed
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.