Problem/Motivation

When first adding the tab widget the default tabs are populated with a sample paragraph. You can expand the available tabs by clicking the button again and increasing the tab count. However, this does not populate the tabs with a sample paragraph. This makes editing them difficult or impossible.

Steps to reproduce

  1. Ensure you have a text format configured with this plugin
  2. Open a node add/edit page
  3. Click the button to add a tab set
  4. Select your tab set and click the button again, increasing the number of tabs
  5. You should see that new tabs are added to reach the new count you set
  6. Select one of these new tabs - you will see it has no content.
  7. Try clicking inside the tab, you will see you cannot easily select the content to edit it

Proposed resolution

New tabs should be populated with sample content.

Remaining tasks

I will attach an MR and patch shortly.

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:

Comments

Dylan Donkersgoed created an issue. See original summary.

dylan donkersgoed’s picture

Status: Active » Needs review
StatusFileSize
new249.37 KB

Changes are in the MR.

I'm also attaching a patch which includes both this change and the change from https://www.drupal.org/project/ckeditor_bootstrap_tabs/issues/3397894.

dylan donkersgoed’s picture

StatusFileSize
new125.03 KB

That last patch does not work (at least against the 2.0.0 version) better one attached.

klidifia’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed behavior. The (combined) patch applied fixes it so newly added tabs have the default content: (note now says "Tab x Content" and before has Tab x Contents"

ozin’s picture

Status: Reviewed & tested by the community » Needs work

Hi folks!
Thanks for your work on this issue.
When you update the MR, I will merge it and do a new release with the latest fixes.

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

randalv’s picture

@ozin MR updated according to the last patch provided by @Dylan Donkersgoed.

ozin’s picture

Status: Needs work » Fixed

Thanks @RandalV

Status: Fixed » Closed (fixed)

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