diff --git a/app/views/admin/_menu.html.erb b/app/views/admin/_menu.html.erb index fecd29419..0b9929f46 100644 --- a/app/views/admin/_menu.html.erb +++ b/app/views/admin/_menu.html.erb @@ -129,20 +129,6 @@ -
  • > - <%= link_to admin_geozones_path do %> - <%= t('admin.menu.geozones') %> - <% end %> -
  • - - <% if feature?(:signature_sheets) %> -
  • > - <%= link_to admin_signature_sheets_path do %> - <%= t("admin.menu.signature_sheets") %> - <% end %> -
  • - <% end %> -
  • @@ -162,5 +148,19 @@
  • + +
  • > + <%= link_to admin_geozones_path do %> + <%= t('admin.menu.geozones') %> + <% end %> +
  • + + <% if feature?(:signature_sheets) %> +
  • > + <%= link_to admin_signature_sheets_path do %> + <%= t("admin.menu.signature_sheets") %> + <% end %> +
  • + <% end %> diff --git a/app/views/admin/site_customization/content_blocks/_form.html.erb b/app/views/admin/site_customization/content_blocks/_form.html.erb index b5a692d87..ae9d684d4 100644 --- a/app/views/admin/site_customization/content_blocks/_form.html.erb +++ b/app/views/admin/site_customization/content_blocks/_form.html.erb @@ -14,34 +14,35 @@ <% end %> +
    +
    + <%= f.label :name %> + <%= f.select :name, SiteCustomization::ContentBlock::VALID_BLOCKS, label: false %> +
    +
    + <%= f.label :locale %> + <%= f.select :locale, I18n.available_locales, label: false %> +
    -
    - <%= f.label :name %> - <%= f.select :name, SiteCustomization::ContentBlock::VALID_BLOCKS, label: false %> -
    -
    - <%= f.label :locale %> - <%= f.select :locale, I18n.available_locales, label: false %> -
    - -
    - <%= f.label :body %> - <%= f.text_area :body, label: false, rows: 10 %> - <%= f.submit class: "button success" %> +
    + <%= f.label :body %> + <%= f.text_area :body, label: false, rows: 10 %> +
    + <%= f.submit class: "button success expanded" %> +
    +
    <% end %> -

    <%= t('.content_blocks_information') %>

    +

    <%= t('.content_blocks_information') %>

    <%= t('.content_block_about') %>

    <%= t('.content_block_top_links_html') %>

    -
    -<li><a href="http://site1.com">Site 1</a></li>
    -<li><a href="http://site2.com">Site 2</a></li>
    -<li><a href="http://site3.com">Site 3</a></li>
    -
    -
    -

    <%= t('.content_block_footer_html') %>

    +<li><a href="http://site1.com">Site 1</a></li>
    +<li><a href="http://site2.com">Site 2</a></li>
    +<li><a href="http://site3.com">Site 3</a></li>
    + +

    <%= t('.content_block_footer_html') %>

    diff --git a/app/views/admin/site_customization/content_blocks/edit.html.erb b/app/views/admin/site_customization/content_blocks/edit.html.erb index c96f9a810..9c2694449 100644 --- a/app/views/admin/site_customization/content_blocks/edit.html.erb +++ b/app/views/admin/site_customization/content_blocks/edit.html.erb @@ -2,14 +2,13 @@ Admin - <%= t("admin.menu.site_customization.content_blocks") %> - <%= @content_block.name %> (<%= @content_block.locale %>) <% end %> -<%= link_to admin_site_customization_content_blocks_path, class: "back" do %> - - <%= t("admin.site_customization.content_blocks.edit.back") %> -<% end %> +<%= back_link_to admin_site_customization_content_blocks_path %> -<%= button_to t("admin.site_customization.content_blocks.index.delete"), admin_site_customization_content_block_path(@content_block), method: :delete, class: "button hollow alert float-right margin-right" %> +<%= button_to t("admin.site_customization.content_blocks.index.delete"), admin_site_customization_content_block_path(@content_block), method: :delete, class: "button hollow alert float-right small" %> -
    -

    <%= t("admin.site_customization.content_blocks.edit.title") %>

    - <%= render 'form' %> +
    +
    +

    <%= t("admin.site_customization.content_blocks.edit.title") %>

    + <%= render 'form' %> +
    diff --git a/app/views/admin/site_customization/content_blocks/index.html.erb b/app/views/admin/site_customization/content_blocks/index.html.erb index 653e3e731..e9fc664d1 100644 --- a/app/views/admin/site_customization/content_blocks/index.html.erb +++ b/app/views/admin/site_customization/content_blocks/index.html.erb @@ -2,28 +2,29 @@ Admin - <%= t("admin.menu.site_customization.content_blocks") %> <% end %> -<%= link_to t("admin.site_customization.content_blocks.index.create"), new_admin_site_customization_content_block_path, class: "button float-right margin-right" %> +<%= link_to t("admin.site_customization.content_blocks.index.create"), new_admin_site_customization_content_block_path, class: "button float-right" %> +

    <%= t("admin.site_customization.content_blocks.index.title") %>

    - - - - - - - - - - <% @content_blocks.each do |content_block| %> - - - - - - <% end %> - -
    <%= t("admin.site_customization.content_blocks.content_block.name") %><%= t("admin.site_customization.content_blocks.content_block.body") %>
    <%= link_to "#{content_block.name} (#{content_block.locale})", edit_admin_site_customization_content_block_path(content_block) %><%= content_block.body %> - <%= button_to t("admin.site_customization.content_blocks.index.delete"), - admin_site_customization_content_block_path(content_block), - method: :delete, class: "button hollow alert" %> -
    + + + + + + + + + + <% @content_blocks.each do |content_block| %> + + + + + + <% end %> + +
    <%= t("admin.site_customization.content_blocks.content_block.name") %><%= t("admin.site_customization.content_blocks.content_block.body") %><%= t("admin.actions.actions") %>
    <%= link_to "#{content_block.name} (#{content_block.locale})", edit_admin_site_customization_content_block_path(content_block) %><%= content_block.body.html_safe %> + <%= button_to t("admin.site_customization.content_blocks.index.delete"), + admin_site_customization_content_block_path(content_block), + method: :delete, class: "button hollow alert" %> +
    diff --git a/app/views/admin/site_customization/content_blocks/new.html.erb b/app/views/admin/site_customization/content_blocks/new.html.erb index 75ff5f389..6a5a1e0b3 100644 --- a/app/views/admin/site_customization/content_blocks/new.html.erb +++ b/app/views/admin/site_customization/content_blocks/new.html.erb @@ -2,13 +2,11 @@ Admin - <%= t("admin.menu.site_customization.content_blocks") %> - <%= t("admin.site_customization.content_blocks.new.title") %> <% end %> -<%= link_to admin_site_customization_content_blocks_path, class: "back" do %> - - <%= t("admin.site_customization.content_blocks.new.back") %> -<% end %> +<%= back_link_to admin_site_customization_content_blocks_path %> -
    -

    <%= t("admin.site_customization.content_blocks.new.title") %>

    - <%= render 'form' %> +
    +
    +

    <%= t("admin.site_customization.content_blocks.new.title") %>

    + <%= render 'form' %> +
    - diff --git a/app/views/admin/site_customization/images/index.html.erb b/app/views/admin/site_customization/images/index.html.erb index b9de152d1..95b9e18a7 100644 --- a/app/views/admin/site_customization/images/index.html.erb +++ b/app/views/admin/site_customization/images/index.html.erb @@ -1,6 +1,12 @@

    <%= t("admin.site_customization.images.index.title") %>

    + + + + + + <% @images.each do |image| %> diff --git a/app/views/admin/site_customization/pages/_form.html.erb b/app/views/admin/site_customization/pages/_form.html.erb index 197520631..2c0d38ba6 100644 --- a/app/views/admin/site_customization/pages/_form.html.erb +++ b/app/views/admin/site_customization/pages/_form.html.erb @@ -1,21 +1,44 @@ <%= form_for [:admin, @page], html: {class: "edit_page", data: {watch_changes: true}} do |f| %> <% if @page.errors.any? %> +
    + -
    - - - - <%= @page.errors.count %> - <%= t("admin.site_customization.pages.errors.form.error", count: @page.errors.count) %> - -
    - + + <%= @page.errors.count %> + <%= t("admin.site_customization.pages.errors.form.error", count: @page.errors.count) %> + +
    <% end %> -
    +
    +
    +
    +
    +
    +

    <%= t("admin.site_customization.pages.form.options") %>

    + <%= f.check_box :more_info_flag, class: "small" %> + <%= f.check_box :print_content_flag %> +
    +
    + <%= f.label :status %> + <% ::SiteCustomization::Page::VALID_STATUSES.each do |status| %> + <%= f.radio_button :status, status, label: false %> + <%= f.label "status_#{status}", t("admin.site_customization.pages.page.status_#{status}") %> +
    + <% end %> +
    +
    + <%= f.label :slug %> + <%= f.text_field :slug, label: false, size: 80, maxlength: 80 %> +
    +
    + +
    +
    +
    <%= f.label :title %> <%= f.text_field :title, label: false %> @@ -23,30 +46,14 @@ <%= f.label :subtitle %> <%= f.text_field :subtitle, label: false, size: 80, maxlength: 80 %> - <%= f.label :slug %> - <%= f.text_field :slug, label: false, size: 80, maxlength: 80 %> -
    <%= f.label :content %> <%= f.cktext_area :content, label: false, cols: 80, rows: 10, ckeditor: { language: I18n.locale } %>
    -
    -
    -
    -

    <%= t("admin.site_customization.pages.form.options") %>

    - <%= f.check_box :more_info_flag %> - <%= f.check_box :print_content_flag %> -
    -

    <%= f.label :status %>

    - <% ::SiteCustomization::Page::VALID_STATUSES.each do |status| %> - <%= f.radio_button :status, status, label: false %> - <%= f.label "status_#{status}", t("admin.site_customization.pages.page.status_#{status}") %> -
    - <% end %> - - <%= f.submit class: "button success" %> +
    + <%= f.submit class: "button success expanded" %> +
    -
    <% end %> diff --git a/app/views/admin/site_customization/pages/edit.html.erb b/app/views/admin/site_customization/pages/edit.html.erb index c92933cbf..9a2fc92ab 100644 --- a/app/views/admin/site_customization/pages/edit.html.erb +++ b/app/views/admin/site_customization/pages/edit.html.erb @@ -2,14 +2,13 @@ Admin - <%= t("admin.menu.site_customization.pages") %> - <%= @page.title %> <% end %> -<%= link_to admin_site_customization_pages_path, class: "back" do %> - - <%= t("admin.site_customization.pages.edit.back") %> -<% end %> +<%= back_link_to admin_site_customization_pages_path %> -<%= button_to t("admin.site_customization.pages.index.delete"), admin_site_customization_page_path(@page), method: :delete, class: "button hollow alert float-right margin-right" %> +<%= button_to t("admin.site_customization.pages.index.delete"), admin_site_customization_page_path(@page), method: :delete, class: "button hollow alert small float-right" %> -
    -

    <%= t("admin.site_customization.pages.edit.title", page_title: @page.title) %>

    - <%= render 'form' %> +
    +
    +

    <%= t("admin.site_customization.pages.edit.title", page_title: @page.title) %>

    + <%= render 'form' %> +
    diff --git a/app/views/admin/site_customization/pages/index.html.erb b/app/views/admin/site_customization/pages/index.html.erb index eef3370e6..f81631640 100644 --- a/app/views/admin/site_customization/pages/index.html.erb +++ b/app/views/admin/site_customization/pages/index.html.erb @@ -2,42 +2,45 @@ Admin - <%= t("admin.menu.site_customization.pages") %> <% end %> -<%= link_to t("admin.site_customization.pages.index.create"), new_admin_site_customization_page_path, class: "button float-right margin-right" %> +<%= link_to t("admin.site_customization.pages.index.create"), new_admin_site_customization_page_path, class: "button float-right" %>

    <%= t("admin.site_customization.pages.index.title") %>

    -

    <%= page_entries_info @pages %>

    +

    <%= page_entries_info @pages %>

    -
    <%= t("admin.site_customization.images.index.image") %><%= t("admin.actions.actions") %>
    - - - - - - - - - - - - <% @pages.each do |page| %> - - - - - - - - - <% end %> - -
    <%= t("admin.site_customization.pages.page.title") %><%= t("admin.site_customization.pages.page.created_at") %><%= t("admin.site_customization.pages.page.updated_at") %><%= t("admin.site_customization.pages.page.status") %>
    - <%= link_to page.title, edit_admin_site_customization_page_path(page) %> - <%= I18n.l page.created_at, format: :short %><%= I18n.l page.created_at, format: :short %><%= t("admin.site_customization.pages.page.status_#{page.status}") %> - - <%= link_to t("admin.site_customization.pages.index.see_page"), page.url %> - - - <%= link_to t("admin.site_customization.pages.index.delete"), admin_site_customization_page_path(page), method: :delete %> -
    + + + + + + + + + + + + <% @pages.each do |page| %> + + + + + + + + + <% end %> + +
    <%= t("admin.site_customization.pages.page.title") %><%= t("admin.site_customization.pages.page.created_at") %><%= t("admin.site_customization.pages.page.updated_at") %><%= t("admin.site_customization.pages.page.status") %><%= t("admin.actions.actions") %>
    + <%= link_to page.title, edit_admin_site_customization_page_path(page) %> + <%= I18n.l page.created_at, format: :short %><%= I18n.l page.created_at, format: :short %><%= t("admin.site_customization.pages.page.status_#{page.status}") %> + + <% if page.status == "published" %> + <%= link_to t("admin.site_customization.pages.index.see_page"), page.url, target: "_blank" %> + <% else %> + <%= t("admin.site_customization.pages.index.see_page") %> + <% end %> + + + <%= link_to t("admin.site_customization.pages.index.delete"), admin_site_customization_page_path(page), method: :delete, class: "delete" %> +
    - <%= paginate @pages %> +<%= paginate @pages %> diff --git a/app/views/admin/site_customization/pages/new.html.erb b/app/views/admin/site_customization/pages/new.html.erb index 43a86d108..2d555c5b8 100644 --- a/app/views/admin/site_customization/pages/new.html.erb +++ b/app/views/admin/site_customization/pages/new.html.erb @@ -2,13 +2,11 @@ Admin - <%= t("admin.menu.site_customization.pages") %> - <%= t("admin.site_customization.pages.new.title") %> <% end %> -<%= link_to admin_site_customization_pages_path, class: "back" do %> - - <%= t("admin.site_customization.pages.new.back") %> -<% end %> +<%= back_link_to admin_site_customization_pages_path %> -
    -

    <%= t("admin.site_customization.pages.new.title") %>

    - <%= render 'form' %> +
    +
    +

    <%= t("admin.site_customization.pages.new.title") %>

    + <%= render 'form' %> +
    - diff --git a/config/locales/admin.en.yml b/config/locales/admin.en.yml index 5ae6d6f49..7b937db8b 100755 --- a/config/locales/admin.en.yml +++ b/config/locales/admin.en.yml @@ -679,16 +679,14 @@ en: notice: Content block deleted successfully edit: title: Editing content block - back: Back errors: form: error: Error index: create: Create new content block - delete: Delete + delete: Delete block title: Content blocks new: - back: Back title: Create new content block content_block: body: Body @@ -698,6 +696,7 @@ en: title: Custom images update: Update delete: Delete + image: Image update: notice: Image updated successfully error: Image couldn't be updated @@ -715,7 +714,6 @@ en: notice: Page deleted successfully edit: title: Editing %{page_title} - back: Back errors: form: error: Error @@ -723,11 +721,10 @@ en: options: Options index: create: Create new page - delete: Delete + delete: Delete page title: Custom Pages see_page: See page new: - back: Back title: Create new custom page page: created_at: Created at diff --git a/config/locales/admin.es.yml b/config/locales/admin.es.yml index f5db64135..160b187ea 100644 --- a/config/locales/admin.es.yml +++ b/config/locales/admin.es.yml @@ -679,16 +679,14 @@ es: notice: Bloque borrado correctamente edit: title: Editar bloque - back: Volver errors: form: error: Error index: create: Crear nuevo bloque - delete: Borrar + delete: Borrar bloque title: Bloques new: - back: Volver title: Crear nuevo bloque content_block: body: Contenido @@ -698,6 +696,7 @@ es: title: Personalizar imágenes update: Actualizar delete: Borrar + image: Imagen update: notice: Imagen actualizada correctamente error: No se ha podido actualizar la imagen @@ -715,7 +714,6 @@ es: notice: Página eliminada correctamente edit: title: Editar %{page_title} - back: Volver errors: form: error: Error @@ -723,11 +721,10 @@ es: options: Opciones index: create: Crear nueva página - delete: Borrar + delete: Borrar página title: Páginas see_page: Ver página new: - back: Back title: Página nueva page: created_at: Creada diff --git a/spec/features/admin/site_customization/content_blocks_spec.rb b/spec/features/admin/site_customization/content_blocks_spec.rb index cc5dca34c..de2155737 100644 --- a/spec/features/admin/site_customization/content_blocks_spec.rb +++ b/spec/features/admin/site_customization/content_blocks_spec.rb @@ -88,7 +88,7 @@ feature "Admin custom content blocks" do expect(page).to have_content("#{block.name} (#{block.locale})") expect(page).to have_content(block.body) - click_button "Delete" + click_button "Delete block" expect(page).to_not have_content("#{block.name} (#{block.locale})") expect(page).to_not have_content(block.body) @@ -98,7 +98,7 @@ feature "Admin custom content blocks" do block = create(:site_customization_content_block) visit edit_admin_site_customization_content_block_path(block) - click_button "Delete" + click_button "Delete block" expect(page).to_not have_content("#{block.name} (#{block.locale})") expect(page).to_not have_content(block.body) diff --git a/spec/features/admin/site_customization/pages_spec.rb b/spec/features/admin/site_customization/pages_spec.rb index 5d03638d8..3e940ea69 100644 --- a/spec/features/admin/site_customization/pages_spec.rb +++ b/spec/features/admin/site_customization/pages_spec.rb @@ -62,7 +62,7 @@ feature "Admin custom pages" do custom_page = create(:site_customization_page, title: "An example custom page") visit edit_admin_site_customization_page_path(custom_page) - click_button "Delete" + click_button "Delete page" expect(page).to_not have_content("An example custom page") end