From 0fe5ecbb24ca44cf4af831fbb9ab927bf4ed3777 Mon Sep 17 00:00:00 2001
From: Neil Lalonde <neillalonde@gmail.com>
Date: Thu, 5 Dec 2013 15:50:15 -0500
Subject: [PATCH] When there are 2000 or more posts, only show the post number
 on the progress bar

---
 .../javascripts/discourse/controllers/topic_controller.js | 8 ++++++++
 .../javascripts/discourse/templates/topic.js.handlebars   | 4 ++--
 app/assets/stylesheets/desktop/topic.scss                 | 2 +-
 config/locales/client.en.yml                              | 1 +
 4 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/discourse/controllers/topic_controller.js b/app/assets/javascripts/discourse/controllers/topic_controller.js
index 56b0a663fc..5fffb498a4 100644
--- a/app/assets/javascripts/discourse/controllers/topic_controller.js
+++ b/app/assets/javascripts/discourse/controllers/topic_controller.js
@@ -293,6 +293,14 @@ Discourse.TopicController = Discourse.ObjectController.extend(Discourse.Selected
     return false;
   }.property('postStream.loaded', 'currentPost', 'postStream.filteredPostsCount'),
 
+  hugeNumberOfPosts: function() {
+    return (this.get('postStream.filteredPostsCount') >= 1000);
+  }.property('postStream.filteredPostsCount'),
+
+  progressPositionTitle: function() {
+    return I18n.t("topic.progress.position", {current: this.get('progressPosition'), total: this.get('highest_post_number')});
+  }.property('progressPosition', 'highest_post_number'),
+
   deselectPost: function(post) {
     this.get('selectedPosts').removeObject(post);
 
diff --git a/app/assets/javascripts/discourse/templates/topic.js.handlebars b/app/assets/javascripts/discourse/templates/topic.js.handlebars
index b785637b65..8f4627b625 100644
--- a/app/assets/javascripts/discourse/templates/topic.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/topic.js.handlebars
@@ -51,8 +51,8 @@
           <div id='topic-progress-wrapper' {{bindAttr class="dockedCounter:docked"}}>
             <nav id='topic-progress' title="{{i18n topic.progress.title}}" {{bindAttr class="hideProgress:hidden"}}>
               <button id='jump-top' title="{{i18n topic.progress.jump_top}}" {{bindAttr disabled="jumpTopDisabled"}} {{action jumpTop}}><i class="icon-circle-arrow-up"></i></button>
-              <div class='nums'>
-                <h4 title="{{i18n topic.progress.current}}">{{progressPosition}}</h4> <span>{{i18n of_value}}</span> <h4>{{highest_post_number}}</h4>
+              <div class='nums' {{bindAttr title="progressPositionTitle"}}>
+                <h4>{{progressPosition}}</h4><span {{bindAttr class="hugeNumberOfPosts:hidden"}}> <span>{{i18n of_value}}</span> <h4>{{highest_post_number}}</h4></span>
               </div>
               <button id='jump-bottom' title="{{i18n topic.progress.jump_bottom}}" {{bindAttr disabled="jumpBottomDisabled"}} {{action jumpBottom}}><i class="icon-circle-arrow-down"></i></button>
               <div class='bg'>&nbsp;</div>
diff --git a/app/assets/stylesheets/desktop/topic.scss b/app/assets/stylesheets/desktop/topic.scss
index 438ab3798f..04dda3634d 100644
--- a/app/assets/stylesheets/desktop/topic.scss
+++ b/app/assets/stylesheets/desktop/topic.scss
@@ -132,8 +132,8 @@ a:hover.reply-new {
     border: 0;
     background: none;
     color: #666;
-    width: 50%;
     text-align: left;
+    width: 30px;
     height: 30px;
     margin: 0;
     i {
diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml
index b1abf9f576..2912cdd153 100644
--- a/config/locales/client.en.yml
+++ b/config/locales/client.en.yml
@@ -674,6 +674,7 @@ en:
         jump_bottom: jump to last post
         total: total posts
         current: current post
+        position: "post %{current} of %{total}"
 
       notifications:
         title: ''