Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Widen blob size column | George Claghorn | 2017-08-24 | 1 | -1/+1 | |
| | ||||||
* | Merge pull request #30213 from ckundo/ccundiff-alt-text-default | Rafael Mendonça França | 2017-08-23 | 1 | -3/+3 | |
|\ | | | | | | | Do not generate default alt text for images | |||||
| * | Do not generate default alt text in image tags | Cameron Cundiff | 2017-08-17 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Auto-generating content from the filename of an image is not suitable alternative text; alt text that isn't fully considered can be distracting and fatiguing for screen readers users (blind, low vision, dyslexic people). - Setting a filename fallback short circuits screen reader default behavior and configuration for blank descriptions. - Setting poor defaults also creates false negatives for accessibility linting and testing software, that makes it harder to improve application accessibility. *** - After this change, if authors leave images without alt text, screen readers will fallback to default behavior for missing alt text. - Also with this change, Automated linting and testing tools will correctly generate warnings. [Fixes #30096] | |||||
* | | Use 5.2 version of ActiveRecord::Migration for Active Storage | bogdanvlviv | 2017-08-22 | 2 | -2/+2 | |
| | | ||||||
* | | Merge pull request #30356 from ↵ | Kasper Timm Hansen | 2017-08-22 | 1 | -2/+0 | |
|\ \ | | | | | | | | | | | | | bogdanvlviv/remove-frozen_string_literal-from-blob-migration Remove frozen_string_literal comment from activestorage's migration | |||||
| * | | Remove frozen_string_literal comment from activestorage's migration | bogdanvlviv | 2017-08-22 | 1 | -2/+0 | |
| | | | | | | | | | | | | | | | The activestorage's migration is used as template for apps Related to #30348 | |||||
* | | | Add frozen string literal pragma | George Claghorn | 2017-08-21 | 1 | -0/+2 | |
| | | | ||||||
* | | | Encode Content-Disposition filenames according to RFC 2231 | George Claghorn | 2017-08-21 | 6 | -4/+74 | |
|/ / | | | | | | | Closes #30134. | |||||
* | | Update regex | George Claghorn | 2017-08-20 | 1 | -1/+1 | |
| | | ||||||
* | | Fix tests | George Claghorn | 2017-08-20 | 4 | -4/+4 | |
| | | ||||||
* | | Fix syntax error | George Claghorn | 2017-08-20 | 1 | -1/+1 | |
| | | ||||||
* | | DRY | George Claghorn | 2017-08-20 | 8 | -16/+20 | |
| | | ||||||
* | | Rename ActiveStorage::Filename#extname to extension_with_delimiter | George Claghorn | 2017-08-20 | 2 | -9/+29 | |
| | | ||||||
* | | Add executable file `activestorage/bin/test` | bogdanvlviv | 2017-08-20 | 1 | -0/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | ``` rails$ cd activestorage/ rails/activestorage$ bin/test ........(compressed).... rails/activestorage$ bin/test test/controllers/disk_controller_test.rb .(compressed).. rails/activestorage$ bin/test test/controllers/disk_controller_test.rb:42 . ``` | |||||
* | | Remove Active Storage code climate config file | Jon Moss | 2017-08-18 | 1 | -7/+0 | |
| | | | | | | | | There's already a .codeclimate.yml file at the root of the project. | |||||
* | | Merge pull request #30292 from maclover7/jm-ast-purge | George Claghorn | 2017-08-17 | 1 | -0/+7 | |
|\ \ | | | | | | | Add missing test for Blob#purge | |||||
| * | | Add missing test for Blob#purge | Jon Moss | 2017-08-17 | 1 | -0/+7 | |
| |/ | ||||||
* / | Use nil Logger for tests | Jon Moss | 2017-08-17 | 1 | -1/+1 | |
|/ | | | | | | | | | | | | | | | | Makes tests much more quietly, as opposed to the enormous amount of logging that appears right now. This setting is used in AJ, as well as other frameworks. Output from test run: ``` ......................................................... Finished in 3.003355s, 18.9788 runs/s, 45.2827 assertions/s. 57 runs, 136 assertions, 0 failures, 0 errors, 0 skips ``` | |||||
* | Merge pull request #30280 from koic/change_namespace_of_active_storage_rake_task | Ryuta Kamizono | 2017-08-16 | 1 | -1/+1 | |
|\ | | | | | Change namespace of Active Storage rake task | |||||
| * | Change namespace of Active Storage rake task | Koichi ITO | 2017-08-16 | 1 | -1/+1 | |
| | | ||||||
* | | Fix RuboCop offenses | Koichi ITO | 2017-08-16 | 1 | -2/+2 | |
|/ | | | | And enable `context_dependent` of Style/BracesAroundHashParameters cop. | |||||
* | restores some double newlines deleted in ae87217 | Xavier Noria | 2017-08-15 | 3 | -0/+6 | |
| | | | | | | These were intentional, see https://github.com/rails/rails/pull/30061#issuecomment-320068368 | |||||
* | minor tweaks in Active Storage after a walkthrough | Xavier Noria | 2017-08-15 | 15 | -48/+59 | |
| | ||||||
* | Move test to its proper place | George Claghorn | 2017-08-14 | 1 | -0/+0 | |
| | | | | ActiveStorage::Filename is a model, so its test case belongs in test/models. | |||||
* | Run Active Storage service tests in CI | George Claghorn | 2017-08-14 | 1 | -0/+0 | |
| | ||||||
* | Should escape regexp wildcard character `.` | Koichi ITO | 2017-08-14 | 3 | -4/+4 | |
| | ||||||
* | Merge pull request #30211 from koic/frozen_activestorage | Matthew Draper | 2017-08-13 | 74 | -4/+148 | |
|\ | | | | | Use frozen string literal in Active Storage | |||||
| * | Use frozen string literal in Active Storage | Koichi ITO | 2017-08-12 | 74 | -4/+148 | |
| | | ||||||
* | | Use `references` to respect primary key type in active storage tables | Ryuta Kamizono | 2017-08-13 | 1 | -5/+3 | |
|/ | | | | | | If created active storage tables by mysql2 or postgresql adapters, a primary key is defined as a bigint. It should be used `references` to the reference columns to respect primary key type. | |||||
* | Merge pull request #30198 from betesh/activestorage-rack-test-uploaded-file | Rafael França | 2017-08-11 | 2 | -2/+8 | |
|\ | | | | | Activestorage rack test uploaded file | |||||
| * | Make activestorage treat Rack::Test::UploadedFile just like ↵ | Isaac Betesh | 2017-08-11 | 2 | -2/+8 | |
| | | | | | | | | ActionDispatch::Http::UploadedFile | |||||
* | | Merge pull request #30147 from yhirano55/fix_trailing_blank_lines | Rafael França | 2017-08-11 | 2 | -2/+0 | |
|\ \ | | | | | | | [Active Storage] `rubocop -a --only Layout/TrailingBlankLines` | |||||
| * | | [Active Storage] `rubocop -a --only Layout/TrailingBlankLines` | Yoshiyuki Hirano | 2017-08-09 | 3 | -3/+1 | |
| | | | ||||||
* | | | Merge pull request #30113 from dixpac/remove_unecessary_require_in_test_helper | Rafael França | 2017-08-11 | 1 | -2/+0 | |
|\ \ \ | | | | | | | | | ActiveStorage: Remove unnecessary require in test_helper | |||||
| * | | | Remove unecessary require in test_helper | dixpac | 2017-08-09 | 1 | -2/+0 | |
| |/ / | ||||||
* | | | Remove unnedded Active Storage installation steps | Rafael Mendonça França | 2017-08-11 | 1 | -8/+0 | |
| | | | | | | | | | | | | | | | Active Storage was now merged to master and it is automatically installed in a new application. | |||||
* | | | Merge pull request #30201 from ↵ | Rafael França | 2017-08-11 | 1 | -12/+8 | |
|\ \ \ | | | | | | | | | | | | | | | | | kamipo/eval_active_storage_attached_name_only_once Evaluate `@active_storage_attached_#{name}` only once | |||||
| * | | | Evaluate `@active_storage_attached_#{name}` only once | Ryuta Kamizono | 2017-08-12 | 1 | -12/+8 | |
| | | | | | | | | | | | | | | | | | | | | Currently `"@active_storage_attached_#{name}"` in `define_method` is evaluated every call. It is better to evaluate it only once. | |||||
* | | | | Merge pull request #30202 from kamipo/add_null_false_to_active_storage_tables | Rafael França | 2017-08-11 | 1 | -10/+10 | |
|\ \ \ \ | | | | | | | | | | | Add `null: false` to Active Storage tables | |||||
| * | | | | Add `null: false` to Active Storage tables | Ryuta Kamizono | 2017-08-12 | 1 | -10/+10 | |
| |/ / / | | | | | | | | | | | | | These columns aren't intended nullable. | |||||
* / / / | Enable browser caching for blobs and variants (#30196) | Anton Khamets | 2017-08-11 | 3 | -0/+17 | |
|/ / / | ||||||
* | | | Microsoft Azure --> Microsoft Azure Storage [ci skip] | Ryuta Kamizono | 2017-08-12 | 1 | -1/+1 | |
| | | | ||||||
* | | | Fix formatting of Active Storage docs [ci skip] | Koichi ITO | 2017-08-11 | 7 | -33/+33 | |
| | | | | | | | | | | | | Follow up of #30188. | |||||
* | | | Fix formatting of Active Storage docs [ci skip] | yuuji.yaginuma | 2017-08-11 | 9 | -36/+36 | |
| |/ |/| | | | | | | | * Use `+` instead of backquote. * Remove escape from class to be link | |||||
* | | Run Active Storage tests in CI (#30144) | George Claghorn | 2017-08-10 | 4 | -20/+18 | |
| | | ||||||
* | | Isolate ActiveStorage namespaces (#30095) | Dino Maric | 2017-08-10 | 2 | -1/+3 | |
| | | | | | | | | | | | | | | * Isolate ActiveStorage namespaces * Rename migrations task [Rafael Mendonça França + Dino Maric] | |||||
* | | Fix a typo | qsona | 2017-08-10 | 1 | -1/+1 | |
|/ | ||||||
* | Merge pull request #30112 from dixpac/filter_minitest_backtrace | Rafael França | 2017-08-08 | 1 | -0/+4 | |
|\ | | | | | ActiveStorage:Filter out Minitest backtrace. | |||||
| * | Filter out Minitest backtrace. | dixpac | 2017-08-07 | 1 | -0/+4 | |
| | | | | | | | | | | Filter out Minitest backtrace while allowing backtrace from other libraries to be shown. | |||||
* | | Merge pull request #30118 from georgeclaghorn/active-storage-load-hooks | Rafael França | 2017-08-08 | 1 | -16/+15 | |
|\ \ | | | | | | | Set ActiveStorage::Blob.service when ActiveStorage::Blob is loaded |