Problem/Motivation

The config options to enable the uswds table are not saved. This looks like a bug with the configuration form not being saved. Is there anything missing in the configuration, that could be causing this. This issue exists with the vanilla drupal installation as well.

Steps to reproduce

  1. Enable the module uswds ckeditor integration
  2. got to the ckeditor filter format page http:///admin/config/content/formats/manage/full_html
  3. Click on the tab CKEditor USWDS table under CKEditor plugin settings
  4. Click on the checkbox which reads "Override table plugin with USWDS table." and click on save configuration button at bottom
  5. Go back to the filter format config page and the checkbox is not checked

Issue fork uswds_ckeditor_integration-3322510

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ramarajuk created an issue. See original summary.

smustgrave made their first commit to this issue’s fork.

smustgrave opened merge request !45

smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented
Priority: Critical » Major
Status: Active » Needs review

Lowering as it doesn't break the site.

but try this MR.

ramarajuk’s picture

ramarajuk CreditAttribution: ramarajuk as a volunteer commented

Thanks for the fix. I created the same patch and applied on my local composer and it works.

BTW, Great work on the module. I work with mark merchant now and he is excited that you had created this module.

  • smustgrave committed 33c4ccc on 1.0.x 
    Issue #3322510: Unable to save the config for ckeditor uswds_table...
smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented
Status: Needs review » Fixed

Awesome! Glad it worked for you guys. Tell him I said hello!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.