Adding media controller, 1 webm/compose form allowed, previews generated

diff --git a/app/assets/javascripts/components/components/status.jsx b/app/assets/javascripts/components/components/status.jsx
index 3882fc97fd2e4c852ba9de91ec566cf53dda33e9..8cdbd5625d88a352645c26669adf90be58fc2f62 100644 (file)
--- a/app/assets/javascripts/components/components/status.jsx
+++ b/app/assets/javascripts/components/components/status.jsx
@@ -40,8 +40,9 @@ const Status = React.createClass({
     if (e.button === 0) {
       e.preventDefault();
       hashHistory.push(`/accounts/${id}`);
-      e.stopPropagation();
     }
+
+    e.stopPropagation();
   },
 
   render () {

diff --git a/app/assets/javascripts/components/components/upload_form.jsx b/app/assets/javascripts/components/components/upload_form.jsx
index 429e672c08344b82e699f5e430021ee6d161e311..f0b6f7992e5317c94760f64b88494186c1f720b7 100644 (file)
--- a/app/assets/javascripts/components/components/upload_form.jsx
+++ b/app/assets/javascripts/components/components/upload_form.jsx
@@ -25,9 +25,11 @@ const UploadForm = React.createClass({
       );
     }.bind(this));
 
+    const noMoreAllowed = (this.props.media.some(m => m.get('type') === 'video')) || (this.props.media.size > 3);
+
     return (
       <div style={{ marginBottom: '20px', padding: '10px', paddingTop: '0' }}>
-        <UploadButton onSelectFile={this.props.onSelectFile} disabled={this.props.is_uploading || this.props.media.size > 3} />
+        <UploadButton onSelectFile={this.props.onSelectFile} disabled={this.props.is_uploading || noMoreAllowed } />
 
         <div style={{ marginTop: '10px', overflow: 'hidden' }}>
           {uploads}

diff --git a/app/assets/javascripts/components/features/status/index.jsx b/app/assets/javascripts/components/features/status/index.jsx
index 122db8906f6a3d3ed3a631a6b798e5dd9613c283..39bc2bec5919a86f0232aa8ae9acc4b846ffd779 100644 (file)
--- a/app/assets/javascripts/components/features/status/index.jsx
+++ b/app/assets/javascripts/components/features/status/index.jsx
@@ -4,6 +4,8 @@ import ImmutablePropTypes       from 'react-immutable-proptypes';
 import { fetchStatus }          from '../../actions/statuses';
 import Immutable                from 'immutable';
 import EmbeddedStatus           from '../../components/status';
+import { favourite, reblog }    from '../../actions/interactions';
+import { replyCompose }         from '../../actions/compose';
 
 function selectStatus(state, id) {
   let status = state.getIn(['timelines', 'statuses', id]);
@@ -49,8 +51,20 @@ const Status = React.createClass({
     }
   },
 
+  handleFavouriteClick (status) {
+    this.props.dispatch(favourite(status));
+  },
+
+  handleReplyClick (status) {
+    this.props.dispatch(replyCompose(status));
+  },
+
+  handleReblogClick (status) {
+    this.props.dispatch(reblog(status));
+  },
+
   renderChildren (list) {
-    return list.map(s => <EmbeddedStatus status={s} key={s.get('id')} />);
+    return list.map(s => <EmbeddedStatus status={s} key={s.get('id')} onReply={this.handleReplyClick} onFavourite={this.handleFavouriteClick} onReblog={this.handleReblogClick} />);
   },
 
   render () {
@@ -63,7 +77,7 @@ const Status = React.createClass({
     return (
       <div>
         {this.renderChildren(ancestors)}
-        <EmbeddedStatus status={status} />
+        <EmbeddedStatus status={status} onReply={this.handleReplyClick} onFavourite={this.handleFavouriteClick} onReblog={this.handleReblogClick} />
         {this.renderChildren(descendants)}
       </div>
     );

diff --git a/app/controllers/media_controller.rb b/app/controllers/media_controller.rb
new file mode 100644 (file)
index 0000000..5a7b0bc
--- /dev/null
+++ b/app/controllers/media_controller.rb
@@ -0,0 +1,13 @@
+class MediaController < ApplicationController
+  before_action :set_media_attachment
+
+  def show
+    redirect TagManager.instance.url_for(@media_attachment.status)
+  end
+
+  private
+
+  def set_media_attachment
+    @media_attachment = MediaAttachment.where.not(status_id: nil).find(params[:id])
+  end
+end

diff --git a/app/models/media_attachment.rb b/app/models/media_attachment.rb
index 7dddfd61064c588c82611bcf769b54be069182f3..73fb967c474a157b07f912f3853bffb710c532e4 100644 (file)
--- a/app/models/media_attachment.rb
+++ b/app/models/media_attachment.rb
@@ -5,7 +5,7 @@ class MediaAttachment < ApplicationRecord
   belongs_to :account, inverse_of: :media_attachments
   belongs_to :status,  inverse_of: :media_attachments
 
-  has_attached_file :file, styles: lambda { |f| f.instance.image? ? { small: '510x680>' } : { small: { format: 'webm' } } }, processors: lambda { |f| f.video? ? [:transcoder] : [:thumbnail] }
+  has_attached_file :file, styles: lambda { |f| f.instance.image? ? { small: '510x680>' } : { small: { convert_options: { output: { vf: 'scale="min(510\, iw):min(680\, ih)":force_original_aspect_ratio=decrease' } }, format: 'png', time: 1 } } }, processors: lambda { |f| f.video? ? [:transcoder] : [:thumbnail] }
   validates_attachment_content_type :file, content_type: IMAGE_MIME_TYPES + VIDEO_MIME_TYPES
   validates_attachment_size :file, less_than: 4.megabytes
 
@@ -26,4 +26,8 @@ class MediaAttachment < ApplicationRecord
   def video?
     VIDEO_MIME_TYPES.include? file_content_type
   end
+
+  def type
+    image? ? 'image' : 'video'
+  end
 end

diff --git a/app/views/api/media/create.rabl b/app/views/api/media/create.rabl
index 0239aa585dd5d9a2226d8f72d481f33a3eee1da7..803a9309443f74ae3f55f0f0c31fb41d32e22e4f 100644 (file)
--- a/app/views/api/media/create.rabl
+++ b/app/views/api/media/create.rabl
@@ -1,4 +1,5 @@
 object @media
-attribute :id
+attribute :id, :type
 node(:url) { |media| full_asset_url(media.file.url) }
 node(:preview_url) { |media| full_asset_url(media.file.url(:small)) }
+node(:text_url) { |media| medium_url(media) }

diff --git a/config/routes.rb b/config/routes.rb
index 4a863a2c1a01b7ae069d2e5fe2ed3d3b6dd2d3fb..1232f8a4425d66803179668db199436dc7e474c1 100644 (file)
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -30,6 +30,7 @@ Rails.application.routes.draw do
   end
 
   resource  :settings, only: [:show, :update]
+  resources :media, only: [:show]
 
   namespace :api do
     # PubSubHubbub