[PATCH conductor] Require a back button/link to return to provider realms.

Jirka Tomasek jtomasek at redhat.com
Mon Feb 13 12:35:00 UTC 2012


On 02/13/2012 12:28 PM, Tomáš Hrčka wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=786010
>
> added back button to Provider realm show view
> ---
>   src/app/views/provider_realms/show.html.haml |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/app/views/provider_realms/show.html.haml b/src/app/views/provider_realms/show.html.haml
> index afa895a..c221551 100644
> --- a/src/app/views/provider_realms/show.html.haml
> +++ b/src/app/views/provider_realms/show.html.haml
> @@ -1,6 +1,9 @@
>   = render :partial =>  'layouts/admin_nav'
>   %header.admin-page-header
>     %h1{:class =>  controller.controller_name}= @realm.name
> +  #obj_actions.button_container
> +    .button-group
> +      =link_to t('back'), edit_provider_path(@realm.provider, :details_tab =>  'realms', :only_tab =>  true), :class =>  'button primary', :id =>  'new_catalog_button'
>
>   - unless @realm.available
>     %section.admin-content-section
I think it would be better to use

#obj_actions.button_container
   .button-group
     = t('return_to')
     = link_to t('provider_realms_translation'), edit_provider_path(@realm.provider, :details_tab =>  'realms', :only_tab =>  true), :class =>  'rounded-link', :id =>  'back_to_provider_realms_button'


to make it consistent with other back buttons. The translation should be for staging and separate patch could be made for product.

Jirka




More information about the aeolus-devel mailing list