Merge branch 'master' into catalog

Conflicts:
	src/General/Index.coffee
	src/General/lib/thread.class

CHANGELOG.md

@@ -1,3 +1,25 @@
+### v1.9.3.0 
+*2014-09-14*
+
+Based on v1.9.2.9.
+
+**MayhemYDG**, **ccd0**
+- Various small bugfixes.
+
+**MayhemYDG**
+- Keep title in 404'd tabs.
+
+**ccd0**
+- Change name of Greasemonkey beta version to `4chan X beta` so the beta and stable versions can be installed side by side.
+- Index sorting/searching/etc. features (`JSON Navigation` option) and including WebM when expanding all images (`Expand videos` option) are now on by default.
+
+### v1.9.2.9 
+*2014-09-13*
+
+**ccd0**
+- Fix minimum Chrome version (32).
+- Fix Linkify bug.
+
 ### v1.9.2.8 
 *2014-09-11*
 

LICENSE

@@ -1,5 +1,5 @@
 /*
-* 4chan X - Version 1.9.2.8
+* 4chan X - Version 1.9.3.0
 *
 * Licensed under the MIT license.
 * https://github.com/ccd0/4chan-x/blob/master/LICENSE

README.md

@@ -1,6 +1,7 @@
-Fork of [Spittie's 4chan X](https://github.com/Spittie/4chan-x) (itself a fork of [Seaweed's](https://github.com/seaweedchan/4chan-x)).
+Adds various features to 4chan.
+A continuation of the fork formerly maintained by [Seaweed](https://github.com/seaweedchan/4chan-x) and subsequently [Spittie](https://github.com/Spittie/4chan-x).
 
-If you're looking for OneeChan, try
+If you're looking for a maintained fork of OneeChan, try
 https://github.com/Nebukazar/OneeChan
 
 ## [Install](https://ccd0.github.io/4chan-x/builds/4chan-X.user.js) (Firefox)
@@ -25,7 +26,7 @@ This should also work for non-Windows/dev/canary Chrome and Chromium-based versi
 Some recent versions of Chromium/Chrome (38.0.2085 - 38.0.2103) (and versions of Opera based on them) suffer from a (now fixed) [bug](https://crbug.com/393686) that prevents extensions from making HTTP requests if more than one extension is enabled. This breaks, among other things, thread updating, quick reply, and, when `JSON Navigation` is enabled, the thread index. If you are experiencing this, try using another version of Chromium/Chrome/Opera, disabling your other extensions, or using a different browser.
 
 ## Other browsers
-This fork of 4chan X is not guaranteed to work correctly in other browsers, but you are welcome to try your luck. Pull requests to fix the bugs you will likely find are always welcome.
+This fork of 4chan X is not guaranteed to work correctly in other browsers, but you are welcome to try your luck. Pull requests to fix the bugs you will likely find are always welcome. You may have better luck with [loadletter's v2 fork](https://github.com/loadletter/4chan-x), which has fewer features but less dependence on browser-specific APIs.
 
 ### dwb
 1. Install dwb with your package manager

builds/4chan-X-beta.meta.js

@@ -1,6 +1,6 @@
 // ==UserScript==
-// @name         4chan X
-// @version      1.9.2.8
+// @name         4chan X beta
+// @version      1.9.3.0
 // @minGMVer     1.14
 // @minFFVer     26
 // @namespace    4chan-X

builds/4chan-X.meta.js

@@ -1,6 +1,6 @@
 // ==UserScript==
 // @name         4chan X
-// @version      1.9.2.8
+// @version      1.9.3.0
 // @minGMVer     1.14
 // @minFFVer     26
 // @namespace    4chan-X

builds/updates-beta.xml

@@ -1,7 +1,7 @@
 <?xml version='1.0' encoding='UTF-8'?>
 <gupdate xmlns='http://www.google.com/update2/response' protocol='2.0'>
   <app appid='lacclbnghgdicfifcamcmcnilckjamag'>
-    <updatecheck codebase='https://ccd0.github.io/4chan-x/builds/4chan-X-beta.crx' version='1.9.2.8' />
+    <updatecheck codebase='https://ccd0.github.io/4chan-x/builds/4chan-X-beta.crx' version='1.9.3.0' />
   </app>
 </gupdate>
 

builds/updates.xml

@@ -1,7 +1,7 @@
 <?xml version='1.0' encoding='UTF-8'?>
 <gupdate xmlns='http://www.google.com/update2/response' protocol='2.0'>
   <app appid='lacclbnghgdicfifcamcmcnilckjamag'>
-    <updatecheck codebase='https://ccd0.github.io/4chan-x/builds/4chan-X.crx' version='1.9.2.8' />
+    <updatecheck codebase='https://ccd0.github.io/4chan-x/builds/4chan-X.crx' version='1.9.3.0' />
   </app>
 </gupdate>
 

package.json

@@ -3,7 +3,7 @@
   "description": "Cross-browser userscript for maximum lurking on 4chan.",
   "meta": {
     "name": "4chan X",
-    "version": "1.9.2.8",
+    "version": "1.9.3.0",
     "repo": "https://github.com/ccd0/4chan-x/",
     "page": "https://github.com/ccd0/4chan-x",
     "downloads": "https://ccd0.github.io/4chan-x/builds/",
@@ -38,7 +38,7 @@
   "devDependencies": {
     "font-awesome": "4.2.0",
     "grunt": "0.4.5",
-    "grunt-concurrent": "0.5.0",
+    "grunt-concurrent": "1.0.0",
     "grunt-contrib-clean": "0.6.0",
     "grunt-contrib-coffee": "0.11.1",
     "grunt-contrib-compress": "0.11.0",
@@ -46,7 +46,7 @@
     "grunt-contrib-copy": "0.5.0",
     "grunt-contrib-watch": "0.6.1",
     "grunt-crx": "0.3.3",
-    "grunt-shell": "1.0.1",
+    "grunt-shell": "1.1.1",
     "load-grunt-tasks": "0.6.0"
   },
   "repository": {
@@ -64,4 +64,4 @@
   "engines": {
     "node": ">=0.10"
   }
-}
+}

src/General/Build.coffee

@@ -153,12 +153,6 @@ Build =
     else
       "/#{boardID}/thread/#{threadID}\#q#{postID}"
 
-    pageIcon = if isOP and g.VIEW is 'index' and Conf['JSON Navigation']
-      pageNum   = Math.floor(Index.liveThreadIDs.indexOf(postID) / Index.threadsNumPerPage) + 1
-      <%= html(' <span class="page-num" title="This thread is on page ${pageNum} in the original index.">[${pageNum}]</span>') %>
-    else
-      <%= html('') %>
-
     sticky = if isSticky
       <%= html(' <img src="//s.4cdn.org/image/sticky${retina}.gif" alt="Sticky" title="Sticky" class="stickyIcon retina">') %>
     else
@@ -183,7 +177,7 @@ Build =
         '<span class="postNum${desktop2}">' +
           '<a href="${postLink}" title="Link to this post">No.</a>' +
           '<a href="${quoteLink}" title="Reply to this post">${postID}</a>' +
-          '&{pageIcon}&{sticky}&{closed}&{replyLink}' +
+          '&{sticky}&{closed}&{replyLink}' +
         '</span>' +
       '</div>'
     ) %>

src/General/Config.coffee

@@ -2,8 +2,8 @@ Config =
   main:
     'Miscellaneous':
       'JSON Navigation' : [
-        false
-        'Use JSON for loading the Board Index. Also allows searching and sorting the board index and infinite scolling.'
+        true
+        'Replace the board index with a dynamically generated one supporting searching, sorting, and infinite scrolling.'
       ]
       'Catalog Links': [
         true
@@ -432,8 +432,8 @@ Config =
       'Expand all images along with spoilers.'
     ]
     'Expand videos': [
-      false
-      'Expand all images also expands videos (no autoplay).'
+      true
+      'Expand all images also expands videos.'
     ]
     'Expand from here': [
       false

src/General/Index.coffee

@@ -432,7 +432,6 @@ Index =
       try
         threadRoot = Build.thread g.BOARD, threadData
         if thread = g.BOARD.threads[threadData.no]
-          thread.setPage Math.floor i / Index.threadsNumPerPage
           thread.setCount 'post', threadData.replies + 1,                threadData.bumplimit
           thread.setCount 'file', threadData.images  + !!threadData.ext, threadData.imagelimit
           thread.setStatus 'Sticky', !!threadData.sticky
@@ -441,8 +440,9 @@ Index =
           thread = new Thread threadData.no, g.BOARD
           threads.push thread
         Index.nodes.push threadRoot
-        continue if thread.ID of thread.posts
-        posts.push new Post $('.opContainer', threadRoot), thread, g.BOARD
+        unless thread.ID of thread.posts
+          posts.push new Post $('.opContainer', threadRoot), thread, g.BOARD
+        thread.setPage i // Index.threadsNumPerPage + 1
       catch err
         # Skip posts that we failed to parse.
         errors = [] unless errors

src/General/UI.coffee

@@ -345,7 +345,7 @@ UI = do ->
     else
       top
 
-    [left, right] = if clientX <= @clientWidth - 400
+    [left, right] = if clientX <= @clientWidth / 2
       [clientX + 45 + 'px', null]
     else
       [null, @clientWidth - clientX + 45 + 'px']

src/General/html/Settings/Advanced.html

@@ -56,7 +56,7 @@
 
 <fieldset>
   <legend>File Info Formatting <span class=warning data-feature='File Info Formatting'>is disabled.</span></legend>
-  <div><input name=fileInfo class=field spellcheck=false>: <span class='fileText file-info-preview'></span></div>
+  <div><input name=fileInfo class=field spellcheck=false>: <span class='file-info file-info-preview'></span></div>
   <div>Link: <code>%l</code> (truncated), <code>%L</code> (untruncated), <code>%T</code> (Unix timestamp)</div>
   <div>Original file name: <code>%n</code> (truncated), <code>%N</code> (untruncated), <code>%t</code> (Unix timestamp)</div>
   <div>Spoiler indicator: <code>%p</code></div>

src/General/lib/databoard.class

@@ -17,9 +17,11 @@ class DataBoard
 
   delete: ({boardID, threadID, postID}) ->
     if postID
+      return unless @data.boards[boardID]?[threadID]
       delete @data.boards[boardID][threadID][postID]
       @deleteIfEmpty {boardID, threadID}
     else if threadID
+      return unless @data.boards[boardID]
       delete @data.boards[boardID][threadID]
       @deleteIfEmpty {boardID}
     else

src/General/lib/thread.class

@@ -16,9 +16,12 @@ class Thread
     g.threads.push @fullID, board.threads.push @, @
 
   setPage: (pageNum) ->
-    icon = $ '.page-num', @OP.nodes.post
-    for key in ['title', 'textContent']
-      icon[key] = icon[key].replace /\d+/, pageNum
+    {info} = @OP.nodes
+    unless icon = $ '.page-num', info
+      icon = $.el 'span', className: 'page-num'
+      $.after $('a[title="Reply to this post"]', info), [$.tn(' '), icon]
+    icon.title       = "This thread is on page #{pageNum} in the original index."
+    icon.textContent = "[#{pageNum}]"
     @catalogView.nodes.pageCount.textContent = pageNum if @catalogView
   setCount: (type, count, reachedLimit) ->
     return unless @catalogView
@@ -41,13 +44,11 @@ class Thread
       src: "#{Build.staticPath}#{typeLC}#{Build.gifIcon}"
       alt:   type
       title: type
-      className: "#{typeLC}Icon"
+      className: "#{typeLC}Icon retina"
     root = if type is 'Closed' and @isSticky
       $ '.stickyIcon', @OP.nodes.info
-    else if g.VIEW is 'index'
-      $ '.page-num', @OP.nodes.info
     else
-      $ '[title="Reply to this post"]', @OP.nodes.info
+      $('.page-num', @OP.nodes.info) or $('[title="Reply to this post"]', @OP.nodes.info)
     $.after root, [$.tn(' '), icon]
 
     return unless @catalogView

src/Images/ImageHover.coffee

@@ -9,6 +9,7 @@ ImageHover =
     return unless @file?.isImage or @file?.isVideo
     $.on @file.thumb, 'mouseover', ImageHover.mouseover
   mouseover: (e) ->
+    return unless doc.contains @
     post = Get.postFromNode @
     {file} = post
     {isVideo} = file

src/Linkification/Linkify.coffee

@@ -153,6 +153,7 @@ Linkify =
     [key, uid, options, link, post] = data
     embed = $.el 'a',
       className:   'embedder'
+      rel:         'nofollow noreferrer'
       href:        link.href
       textContent: '(embed)'
 
@@ -225,7 +226,7 @@ Linkify =
 
   ordered_types: [
       key: 'audio'
-      regExp: /\.(?:mp3|ogg|wav)$/
+      regExp: /\.(?:mp3|ogg|wav)$/i
       style: ''
       el: (a) ->
         $.el 'audio',
@@ -247,7 +248,7 @@ Linkify =
           return file for file of files when files.hasOwnProperty file
     ,
       key: 'image'
-      regExp: /\.(?:gif|png|jpg|jpeg|bmp)$/
+      regExp: /\.(?:gif|png|jpg|jpeg|bmp)$/i
       style: 'border: 0; width: auto; height: auto;'
       el: (a) ->
         $.el 'div', <%= html('<a target="_blank" href="${a.href}"><img src="${a.href}"></a>') %>
@@ -277,7 +278,7 @@ Linkify =
         el
     ,
       key: 'MediaCrush'
-      regExp: /^\w+:\/\/(?:www\.)?mediacru\.sh\/([\w\-]+)/i
+      regExp: /^\w+:\/\/(?:www\.)?mediacru\.sh\/([\w\-]+)/
       style: 'border: 0;'
       el: (a) ->
         el = $.el 'div'
@@ -445,7 +446,7 @@ Linkify =
       dummy: true
     ,
       key: 'video'
-      regExp: /\.(?:ogv|webm|mp4)$/
+      regExp: /\.(?:ogv|webm|mp4)$/i
       style: 'border: 0; width: auto; height: auto;'
       el: (a) ->
         $.el 'video',

src/Monitoring/Unread.coffee

@@ -180,7 +180,7 @@ Unread =
     countQuotingYou = Object.keys(Unread.postsQuotingYou).length
 
     if Conf['Unread Count']
-      d.title = "#{if Conf['Quoted Title'] and countQuotingYou then '(!) ' else ''}#{if count or !Conf['Hide Unread Count at (0)'] then "(#{count}) " else ''}#{if g.DEAD then "/#{g.BOARD}/ - 404" else "#{Unread.title}"}"
+      d.title = "#{if Conf['Quoted Title'] and countQuotingYou then '(!) ' else ''}#{if count or !Conf['Hide Unread Count at (0)'] then "(#{count}) " else ''}#{if g.DEAD then Unread.title.replace '-', '- 404 -' else Unread.title}"
 
     return unless Conf['Unread Favicon']