In the content type configuration for D7 a summary is shown on the vertical tab that contains some information whether or not scheduled publishing or unpublishing is enabled. This information is missing in D8.

To replicate this, install the standard profile and navigate to /admin/structure/types/manage/article.

CommentFileSizeAuthor
#4 summary_not_visible_on-2467399-4.patch1.39 KBjoekers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented

See patch by @joekers in comment #4 of #2462133: Link in log message should not be escaped.

  • pfrenssen committed 1af76dc on 8.x-1.x authored by joekers 
    Issue #2467399 by joekers: Summary not visible on content type vertical...
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented

I tried the patch from #2462133: Link in log message should not be escaped but it didn't work yet.

joekers’s picture

joekers
Primary language English
CreditAttribution: joekers at Zoocha commented
Assigned: Unassigned » joekers
Status: Active » Needs review
FileSize
summary_not_visible_on-2467399-4.patch1.39 KB

Fixed this so that the summary information shows when enabling/disabling scheduler settings. The problem was that drupal.ajax (change record) wasn't added as a dependancy, and Drupal.settings is now drupalSettings which also needs to be added as a dependancy. Also updated the js selector now that settings form element isn't a fieldset.

  • pfrenssen committed 7057d9b on 8.x-1.x 
    Revert "Issue #2467399 by joekers: Summary not visible on content type...
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented

I accidentally used the commit message for this issue when committing the fix for #2467395: Display content type configuration in a vertical tab in commit 1af76dc. I have reverted that commit and have recommitted it using the correct message.

  • pfrenssen committed 3a96397 on 8.x-1.x authored by joekers 
    Issue #2467399 by joekers: Summary not visible on content type vertical...
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented
Status: Needs review » Fixed

Thanks a lot for figuring this out! It works perfectly!

In addition to the fix for this issue there is also a fix for the theming workaround that was introduced in #1172040: Contrib solution for non-collapsible fieldsets and missing titles. I will make two separate commits for these, but will use this issue number for both.

  • pfrenssen committed 459df39 on 8.x-1.x authored by joekers 
    Issue #2467399 by joekers: Theme name is no longer set on vertical tabs...
joekers’s picture

joekers
Primary language English
CreditAttribution: joekers at Zoocha commented

Cool! Thanks for your help - I'll try and tackle some of the other issues :)

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented

You're welcome, thanks for all the great fixes!

I will be going to Drupal Dev Days next week so I will be mainly focused on Drupal core issues, but I'll be back on Scheduler from April 20.

Status: Fixed » Closed (fixed)

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