Angular ModalFormService should follow new style for help

Bug #1781151 reported by Shu Muto
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Shu Muto

Bug Description

Angular ModalFormService should use new style for help like WizardModalService

ModalFormService (horizon.framework.widgets.form.ModalFormService, used in create key pair dialog or create user dialog) and WizardModalService (horizon.framework.widgets.modal.wizard-modal.service, used in create instance dialog or create image dialog) are used in dialogs for create/edit/etc in Angularized panel. But their help descriptions are shown in different style.

Angularization proposes new look and feel, so ModalFormService should follow new style for help description like WizardModalService.

Discussed at followings:
Comments in https://review.openstack.org/#/c/537336/
L181 in https://etherpad.openstack.org/p/horizon-ptg-rocky

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

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

Changed in horizon:
assignee: nobody → Shu Muto (shu-mutou)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Akihiro Motoki (amotoki)
tags: added: angularjs
Changed in horizon:
importance: Undecided → Wishlist
milestone: none → stein-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/581618
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=f5666eed009be13d710f3420954a9cfbb4378c88
Submitter: Zuul
Branch: master

commit f5666eed009be13d710f3420954a9cfbb4378c88
Author: Shu Muto <email address hidden>
Date: Wed Jul 11 16:56:52 2018 +0900

    Use new style for help in Angular ModalFormService

    ModalFormService (horizon.framework.widgets.form.ModalFormService,
    used in create key pair dialog or create user dialog) and WizardModalService
    (horizon.framework.widgets.modal.wizard-modal.service, used in create instance
    dialog or create image dialog) are used in dialogs for create/edit/etc in Angularized
    panel. But their help descriptions are shown in different style.

    Angularization proposes new look and feel, so ModalFormService should follow
    new style for help description like WizardModalService.

    This patch proposes new style for help in dialogs that uses ModalFormService.

    And "?" button for opening help panel is overlaped on input field on the first row,
    so this patch moves it up little bit.

    In this new style, if the help contents is too simple, it is not needed to
    make description space in the right harf of dialog for looks consistency,
    it is prefer not to add help panel, i.e. simply do not show "?" button.

    Change-Id: Ia720180c7aaef1f3d9c5e24dbebe3fe2d796d6c7
    Closes-Bug: #1781151

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 15.0.0.0b1

This issue was fixed in the openstack/horizon 15.0.0.0b1 development milestone.

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.