OpenStack Dashboard (Horizon)

Launch instance flavor details table weirdness

Bug #1080513 reported by Kieran Spear on 2012-11-19

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	OpenStack Dashboard (Horizon)	Fix Released	Medium	Kieran Spear	OpenStack Dashboard (Horizon) 2013.1 "grizzly"

Bug Description

1. Bring up launch instance dialog
2. Click on another tab
3. Click back to the details tab

The last row of the flavor details table will say "RAM Displaying 7 items MB" (see screenshot).
Looks like the table footer js is being run where it shouldn't.

Revision history for this message

Kieran Spear (kspear) wrote on 2012-11-19:

flavor_details.png Edit (58.3 KiB, image/png)

Revision history for this message

Gabriel Hurley (gabriel-hurley) wrote on 2012-11-20:

Ha! Great catch. That's pretty hilarious.

Changed in horizon:
importance:	Undecided → Medium
milestone:	none → grizzly-2
status:	New → Confirmed

Gabriel Hurley (gabriel-hurley) on 2013-01-06

Changed in horizon:
milestone:	grizzly-2 → grizzly-3

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-01-14: Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/19603

Changed in horizon:
assignee:	nobody → Kieran Spear (kspear)
status:	Confirmed → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-01-15: Fix merged to horizon (master)

Reviewed: https://review.openstack.org/19603
Committed: http://github.com/openstack/horizon/commit/4632aa58ae9d1874f559031f592e2873c0fb4f64
Submitter: Jenkins
Branch: master

commit 4632aa58ae9d1874f559031f592e2873c0fb4f64
Author: Kieran Spear <email address hidden>
Date: Mon Jan 14 14:35:18 2013 +1100

Don't update count on tables without a summary row

    The horizon.tabs javascript was updating the last row of any <table>
    in the tab when the focus was changed to a different tab, stomping
    over the last row of the table.

    This commit limits updates to tables with the .datatable class,
    and adds a class to the updated <span> in affected templates
    so it can be selected a little more unambiguously.

Fixes bug #1080513.

Change-Id: I1b778bdc77887dcf540c96c2ab8be239bc13258c