X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/3666b674330ec8d14a224932a191d6121c5974e2..77d495ccdc4c6d07cb7795177c73ba541afc44e0:/app/views/browse/changeset.html.erb?ds=sidebyside diff --git a/app/views/browse/changeset.html.erb b/app/views/browse/changeset.html.erb index 3c41dd83f..9acbd05ae 100644 --- a/app/views/browse/changeset.html.erb +++ b/app/views/browse/changeset.html.erb @@ -1,31 +1,34 @@ <% set_title(t(".title", :id => @changeset.id)) %> -

- - <%= t(".title", :id => @changeset.id) %> -

+<%= render "sidebar_header", :title => t(".title", :id => @changeset.id) %>
-
<%= linkify(@changeset.tags["comment"].to_s.presence || t("browse.no_comment")) %>
-
<%= changeset_details(@changeset) %>
+

+ <%= linkify(@changeset.tags["comment"].to_s.presence || t("browse.no_comment")) %> +

+

<%= changeset_details(@changeset) %>

<%= render :partial => "tag_details", :object => @changeset.tags.except("comment") %> -

<%= t(".discussion") %>

- - <% if current_user %> -
-
- <% if @changeset.subscribers.exists?(current_user.id) %> - " data-method="POST" data-url="<%= changeset_unsubscribe_url(@changeset) %>" /> - <% else %> - " data-method="POST" data-url="<%= changeset_subscribe_url(@changeset) %>" /> - <% end %> -
+
+
+

<%= t(".discussion") %>

- <% end %> -
+ <% if current_user %> +
+ +
+ <% end %> +
<% if @comments.length > 0 %>
@@ -34,27 +37,31 @@ <% @comments.each do |comment| %> <% if comment.visible %>
  • - - <%= t(".commented_by", - :when => friendly_date_ago(comment.created_at), - :exact_time => l(comment.created_at), - :user => link_to(comment.author.display_name, user_path(comment.author))).html_safe %> + + <%= t(".comment_by_html", + :time_ago => tag.abbr(friendly_date_ago(comment.created_at), + :title => l(comment.created_at)), + :user => link_to(comment.author.display_name, user_path(comment.author))) %> <% if current_user and current_user.moderator? %> - — <%= t("javascripts.changesets.show.hide_comment") %> + — <%= t("javascripts.changesets.show.hide_comment") %> <% end %> - <%= comment.body.to_html %> +
    + <%= comment.body.to_html %> +
  • <% elsif current_user and current_user.moderator? %>
  • - - <%= t(".hidden_commented_by", - :when => friendly_date_ago(comment.created_at), - :exact_time => l(comment.created_at), - :user => link_to(comment.author.display_name, user_path(comment.author))).html_safe %> - — <%= t("javascripts.changesets.show.unhide_comment") %> + + <%= t(".hidden_comment_by_html", + :time_ago => tag.abbr(friendly_date_ago(comment.created_at), + :title => l(comment.created_at)), + :user => link_to(comment.author.display_name, user_path(comment.author))) %> + — <%= t("javascripts.changesets.show.unhide_comment") %> - <%= comment.body.to_html %> +
    + <%= comment.body.to_html %> +
  • <% end %> <% end %> @@ -64,58 +71,53 @@ <% end %> <% unless current_user %> -
    - <%= link_to(t(".join_discussion"), :controller => "users", :action => "login", :referer => request.fullpath) %> -
    +

    + <%= link_to(t(".join_discussion"), login_path(:referer => request.fullpath)) %> +

    <% end %> <% if current_user %> - <% unless @changeset.is_open? %> -
    - -
    - " data-changeset-id="<%= @changeset.id %>" data-method="POST" data-url="<%= changeset_comment_url(@changeset) %>" disabled="1" /> + <% unless @changeset.open? %> + +
    + +
    + +
    + " data-changeset-id="<%= @changeset.id %>" data-method="POST" data-url="<%= changeset_comment_url(@changeset) %>" disabled="1" class="btn btn-sm btn-primary" />
    <% else %> -
    - <%= t(".still_open") %> -
    +

    + <%= t(".still_open") %> +

    <% end %> <% end %> <% unless @ways.empty? %> -

    - <%= type_and_paginated_count("way", @way_pages) %> - <%= render :partial => "paging_nav", :locals => { :pages => @way_pages, :page_param => "way_page" } %> -

    + <%= render :partial => "paging_nav", :locals => { :heading => type_and_paginated_count("way", @way_pages), :pages => @way_pages, :page_param => "way_page" } %>
      <% @ways.each do |way| %> -
    • <%= link_to printable_name(way, true), { :action => "way", :id => way.way_id.to_s }, { :class => link_class("way", way), :title => link_title(way) } %>
    • +
    • <%= link_to printable_name(way, :version => true), { :action => "way", :id => way.way_id.to_s }, { :class => link_class("way", way), :title => link_title(way) } %>
    • <% end %>
    <% end %> <% unless @relations.empty? %> -

    - <%= type_and_paginated_count("relation", @relation_pages) %> - <%= render :partial => "paging_nav", :locals => { :pages => @relation_pages, :page_param => "relation_page" } %> -

    + <%= render :partial => "paging_nav", :locals => { :heading => type_and_paginated_count("relation", @relation_pages), :pages => @relation_pages, :page_param => "relation_page" } %>
      <% @relations.each do |relation| %> -
    • <%= link_to printable_name(relation, true), { :action => "relation", :id => relation.relation_id.to_s }, { :class => link_class("relation", relation), :title => link_title(relation) } %>
    • +
    • <%= link_to printable_name(relation, :version => true), { :action => "relation", :id => relation.relation_id.to_s }, { :class => link_class("relation", relation), :title => link_title(relation) } %>
    • <% end %>
    <% end %> <% unless @nodes.empty? %> -

    - <%= type_and_paginated_count("node", @node_pages) %> - <%= render :partial => "paging_nav", :locals => { :pages => @node_pages, :page_param => "node_page" } %> -

    + <%= render :partial => "paging_nav", :locals => { :heading => type_and_paginated_count("node", @node_pages), :pages => @node_pages, :page_param => "node_page" } %>
      <% @nodes.each do |node| %> -
    • <%= link_to printable_name(node, true), { :action => "node", :id => node.node_id.to_s }, { :class => link_class("node", node), :title => link_title(node), :rel => link_follow(node) } %>
    • +
    • <%= link_to printable_name(node, :version => true), { :action => "node", :id => node.node_id.to_s }, { :class => link_class("node", node), :title => link_title(node), :rel => link_follow(node) } %>
    • <% end %>
    <% end %> @@ -128,7 +130,7 @@ · <% end %> <%= user = (@prev_by_user || @next_by_user).user.display_name - link_to content_tag(:bdi, user), :controller => "changesets", :action => "index", :display_name => user %> + link_to tag.bdi(user), :controller => "changesets", :action => "index", :display_name => user %> <% if @next_by_user %> · <%= link_to "#{@next_by_user.id} >>", :id => @next_by_user.id %>