aboutsummaryrefslogtreecommitdiffstats
path: root/actiontext
Commit message (Collapse)AuthorAgeFilesLines
* Update sanitizer in ActionView::Helpers::SanitizeHelperJuanito Fatas2019-08-051-1/+1
| | | | | - The sanitizer has been changed to safe_list_sanitizer. - deprecate white_list_sanitizer
* It may be better to explicitly require 'object/try' where we call `try`Akira Matsuda2019-08-013-0/+5
| | | | | | In most cases it works now without explicit require because it's accidentally required through active_support/core_ext/date_and_time/calculations.rb where we still call `try`, but that would stop working if we changed the Calculations implementation and remove the require call there.
* Bump Trix to ^1.2.0Javan Makhmali2019-07-301-1/+1
| | | | | | | | | Adds an attachment button to the Trix toolbar that improves overall file upload usability, especially on mobile devices where files can't be dragged / dropped. References: - https://github.com/basecamp/trix/releases/tag/1.2.0 - https://github.com/basecamp/trix/pull/619 - https://github.com/basecamp/trix/issues/582
* Use match? where we don't need MatchDataAkira Matsuda2019-07-292-2/+2
|
* Let the generated initializers/backtrace_silencers.rb code use Regexp#match?Akira Matsuda2019-07-291-1/+1
|
* Implement ActiveStorage::Blob#attachable_plain_text_representationGeorge Claghorn2019-07-072-0/+9
| | | | | | Fixes that file attachments without captions would not be represented in plain text generated from rich-text content, causing ActionText::RichText#present? to return false. Closes #36607.
* Unify to use 4 spaces indentation in CHANGELOGs [ci skip]Ryuta Kamizono2019-06-051-13/+13
| | | | | Especially, somehow `CHANGELOG.md` in actiontext and activestorage in master branch had used 3 spaces indentation.
* Correct test nameGeorge Claghorn2019-05-201-1/+1
|
* Allow filling in the only rich-text area without a locatorGeorge Claghorn2019-05-192-7/+17
|
* Prefer Capybara::Node::Element#execute_scriptGeorge Claghorn2019-05-191-4/+1
|
* Deduplicate ActionText::RichText embedsGeorge Claghorn2019-05-172-1/+10
| | | | Fix that an ActiveRecord::RecordNotUnique error would be raised when saving rich-text content with the same file attached multiple times.
* Add ActionDispatch::SystemTestCase#fill_in_rich_text_areaGeorge Claghorn2019-05-1311-1160/+975
|
* Update Webpacker in Action Text's test dummy appGeorge Claghorn2019-05-1113-564/+443
|
* Start Rails 6.1 developmentRafael Mendonça França2019-04-243-16/+4
|
* Make Action Text's rendering helpers more configurableJavan Makhmali2019-04-221-9/+14
| | | | | - Allow configuring the sanitizer and its options - Split attachment rendering and sanitizing helpers so each can be overridden by applications
* Adds a warning message for action text installer if application pack is missing.Abhay Nikam2019-04-191-0/+13
|
* Output junit format test reportFumiaki MATSUSHIMA2019-04-041-0/+2
|
* Avoid creating ActionText::RichText records unnecessarilyGeorge Claghorn2019-03-232-1/+9
| | | | Assigning a has_one association for a persisted record saves the change immediately, so attempting to read a rich-text attribute on a persisted record without a corresponding ActionText::RichText would eagerly create one. Avoid assigning the rich text association to fix.
* Fix updating rich text via nested attributesGeorge Claghorn2019-03-176-7/+40
| | | Closes #35159.
* Merge tag 'v6.0.0.beta3'eileencodes2019-03-133-2/+7
|\ | | | | | | v6.0.0.beta3 release
| * Prep releaseeileencodes2019-03-113-2/+7
| | | | | | | | | | | | | | * Update RAILS_VERSION * Bundle * rake update_versions * rake changelog:header
* | Merge pull request #35485 from excid3/default-attachable-trix-partialJavan Makhmali2019-03-115-2/+31
|\ \ | | | | | | Adds default trix content attachmment partial path
| * | Adds default trix partialChris Oliver2019-03-055-2/+31
| |/
* | Merge pull request #35529 from ↵Ryuta Kamizono2019-03-092-2/+2
|\ \ | | | | | | | | | | | | abhaynikam/35492-follow-up-to-updates-links-to-https Updated links from http to https in guides, docs, etc
| * | Updated links from http to https in guides, docs, etcAbhay Nikam2019-03-092-2/+2
| | |
* | | Upgrade webpack-dev-server version in test appsSharang Dashputre2019-03-092-435/+229
| | |
* | | Use the latest stable release of webpackerSharang Dashputre2019-03-092-2000/+3727
|/ /
* / Explicitly require rails-html-sanitizer gem in ActionText helpersChris LaRose2019-03-061-0/+2
|/ | | | | | | | | | | If the [`action_text.helper` initializer][0] runs after `ActionController::Base` has been loaded, but before the `rails-html-sanitizer` gem has been `require`d, then the reference to the constant `Rails::Html` in the body of the `ActionText::ContentHelper` module raises an `uninitialized constant` exception. [0]: https://github.com/rails/rails/blob/21703382393c87212c27c988420ee5c133c1aa9f/actiontext/lib/action_text/engine.rb#L31-L35
* Preparing for 6.0.0.beta2 releaseRafael Mendonça França2019-02-253-2/+7
|
* Disable available locale checks in Action Test testyuuji.yaginuma2019-02-241-0/+3
| | | | | | | | | Without this change, `store_translations` silently fails when available locales already initialized. Ref: https://travis-ci.org/rails/rails/jobs/497615616#L6846 https://travis-ci.org/rails/rails/jobs/497605027#L6856
* Allows rich_text_area_tag to add I18n translated placeholder text if ↵Abhay Nikam2019-02-222-2/+52
| | | | placeholder option set to true
* SQLite3: Implement `add_foreign_key` and `remove_foreign_key`Ryuta Kamizono2019-02-111-0/+1
| | | | | | | | | | | | I implemented Foreign key create in `create_table` for SQLite3 at #24743. This follows #24743 to implement `add_foreign_key` and `remove_foreign_key`. Unfortunately SQLite3 has one limitation that `PRAGMA foreign_key_list(table-name)` doesn't have constraint name. So we couldn't implement find/remove foreign key by name for now. Fixes #35207. Closes #31343.
* Merge pull request #35071 from kamipo/text_without_limitRyuta Kamizono2019-01-293-5/+4
|\ | | | | MySQL: Support `:size` option to change text and blob size
| * Allow changing text and blob size without giving the `limit` optionRyuta Kamizono2019-01-293-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In MySQL, the text column size is 65,535 bytes by default (1 GiB in PostgreSQL). It is sometimes too short when people want to use a text column, so they sometimes change the text size to mediumtext (16 MiB) or longtext (4 GiB) by giving the `limit` option. Unlike MySQL, PostgreSQL doesn't allow the `limit` option for a text column (raises ERROR: type modifier is not allowed for type "text"). So `limit: 4294967295` (longtext) couldn't be used in Action Text. I've allowed changing text and blob size without giving the `limit` option, it prevents that migration failure on PostgreSQL.
| * Revert "Apply `t.timestamps` changes in Action Text and Action Mailbox"Ryuta Kamizono2019-01-292-1/+2
| | | | | | | | This reverts commit 30f666f87ab873258b797b39f29cf852f7621bea.
| * Apply `t.timestamps` changes in Action Text and Action MailboxRyuta Kamizono2019-01-292-2/+1
| | | | | | | | Follow up #34956.
* | Add line break to Action Text installation outputsVinicius Brasil2019-01-271-1/+1
|/ | | | | | | | | | | | | | The Action Text installations appends `require("trix")` to the application.js file. The problem is that there isn't a line break in the beginning of the installation output, leading to syntax errors, e.g.: ``` import './application.scss'require("trix") ``` This commit moves the line break from the end to the beginning of the output, fixing it to: ``` import './application.scss' require("trix") ```
* Merge pull request #34970 from kamipo/timestamps_with_precision_by_defaultRyuta Kamizono2019-01-262-8/+7
|\ | | | | Make `t.timestamps` with precision by default.
| * Make `t.timestamps` with precision by defaultRyuta Kamizono2019-01-262-8/+7
| |
* | Fix error saving Action Text content containing non-blob attachablesJavan Makhmali2019-01-252-1/+10
|/ | | | | | | | | | Failing test before the ActionText::RichText change: ``` Error: ActionText::ModelTest#test_embed_extraction_only_extracts_file_attachments: ArgumentError: Could not find or build blob: expected attachable, got #<ActionText::Attachables::RemoteImage:0x00007fb0259fef70 @url="http://example.com/cat.jpg", @content_type="image", @width=nil, @height=nil> ```
* Prefer ImageProcessing's resize_to_limit macro over resize_to_fitGeorge Claghorn2019-01-241-1/+1
| | | | Don't upsize images smaller than the specified dimensions.
* Fix document formatting on Action Text docs [ci skip]colorbox2019-01-242-4/+4
| | | | Use `+` instead of backquote.
* Changed webserver to web server.alkesh262019-01-221-1/+1
|
* Preparing for 6.0.0.beta1 releaseRafael Mendonça França2019-01-183-2/+4
|
* Merge pull request #34956 from kamipo/actionmailbox_datetime_precisionRyuta Kamizono2019-01-181-5/+5
|\ | | | | Allow using Action Mailbox on MySQL 5.5
| * Allow using Action Mailbox on MySQL 5.5Ryuta Kamizono2019-01-181-5/+5
| | | | | | | | | | | | | | | | | | | | Active Record still support MySQL 5.5 which doesn't support datetime with precision. https://github.com/rails/rails/blob/9e34df00039d63b5672315419e76f06f80ef3dc4/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb#L99-L101 So we should check `supports_datetime_with_precision?` on the connection.
* | Action Text: bundle package.json in built gem.Kasper Timm Hansen2019-01-181-1/+1
|/ | | | | | | | | | | | | | | | | | After 866da19fd9fc12d001ab99ac87890ea1cebb2cd9 we now use package.json to install Action Text's JS dependencies (see JS_PACKAGE_PATH). But when the gem pacakge for Action Text is built, package.json was not included so running `rails action_text:install` in a Rails app would fail with: ``` rails action_text:install rails aborted! Errno::ENOENT: No such file or directory @ rb_sysopen - xxx/gems/actiontext-6.0.0.alpha/lib/templates/../../package.json ./bin/rails:4:in `<main>' Tasks: TOP => app:template (See full trace by running task with --trace) ```
* Tidy up action_text:install taskJavan Makhmali2019-01-171-9/+20
| | | | Automate installing the appropriate packages with yarn and appending them to the default application.js pack.
* Make trix a peer dependency of actiontext since it’s not used directlyJavan Makhmali2019-01-172-2/+3
|
* Move all npm packages to @rails scopeJavan Makhmali2019-01-105-20/+8
| | | | Fixes #33083