Order of fields in Update Default Quotas should match Default Quotas page

Bug #1275892 reported by Guilherme Lazzari
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Tapan Sharma

Bug Description

The fields listed in the Update Default Quotas dialog are in a different order than the fields listed on the Default Quotas page itself. For a better UX, update dialog should list in the same order as the Default Quotas page.

Changed in horizon:
assignee: nobody → Jeffrey J. Walls (jeff-walls)
Changed in horizon:
assignee: Jeffrey J. Walls (jeff-walls) → Tapan Sharma (tapan-sharma)
Changed in horizon:
status: New → Confirmed
Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
Tapan Sharma (tapan-sharma) wrote :

This defect fix may not land.

Thai Tran commented on code review for this defect:

"""
This is great. With the fix introduce in bug#1231894, this will no longer apply. We are populating the fields dynamically from a dictionary, and due to the hashing nature of the dictionary, the order of the fields are somewhat random.
As it stands, we are currently debating whether new volume types should even be in the defaults tab. With that said, if we do decide that volume type defaults do belong here, I think the next logical step is use a sorted dictionary that will address this issue.
"""

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/71084
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=8d7ea95f8498d52289f301969e4f07409d5275ea
Submitter: Jenkins
Branch: master

commit 8d7ea95f8498d52289f301969e4f07409d5275ea
Author: Tapan Sharma <email address hidden>
Date: Tue Feb 4 12:07:55 2014 -0800

    The order of fields have changed for update default quota form.

    Changed the ordering of the the fields for update default quota form
    so that it now matches the default quota table page.

    Change-Id: I76c5b75f30d9db39c83b395bffbf11340b2e0bde
    Closes-Bug: #1275892

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → icehouse-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-3 → 2014.1
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.