web-illinois / illinois_framework_theme

An Illinois branded Drupal theme based off of the bootstrap4 theme.
5 stars 1 forks source link

Drupal 10 Support #896

Closed trubach closed 4 months ago

trubach commented 1 year ago

Drupal 9 support ends in November 2023. This is a parent issue to track what needs to be done for the ILFW to support Drupal 10:

Here's an article that outlines the process of upgrading to Drupal 10

trubach commented 1 year ago

An article about including custom styles in CKEditor5: https://www.drupal.org/docs/core-modules-and-themes/core-modules/ckeditor-5-module/how-to-style-custom-content-in-ckeditor-5

added stylesheets for ckeditor5, which pulls in the framework type styles, this is different than what people are used to

if you want to make changes to ckeditor5 styles, you need to recompile css sass --watch scss/ck5style.scss:css/ck5style.css

nhjohnstone commented 1 year ago

Info on starterkit theme and creating a custom sub theme: https://www.drupal.org/docs/core-modules-and-themes/core-themes/starterkit-theme

trubach commented 1 year ago

RE: SimpleSAMLPHP - here's an alternative authentication method we might use using AzureAD

https://www.drupal.org/docs/contributed-modules/openid-connect-microsoft-azure-ad

nhjohnstone commented 11 months ago

No D10 version for SCSS compiler module - patched https://www.drupal.org/project/scss_compiler/issues/3298001 had to patch the version too

installed for scss_compiler patch to work composer require mglaman/composer-drupal-lenient

add to composer.json scss patches and drupal-lenient

core modules that are deprecated, uninstall: Quick Edit RDF

TB megamenu has a version for 9 and a different version for 10, need to include both in composer.json https://www.drupal.org/project/tb_megamenu - uninstall

theme updates needed: "drupal/bootstrap4": "^3.0", - uninstall "drupal/bootstrap5": "^3.0", "drupal/tb_megamenu" : "^2.0@alpha||^3.0@alpha" - uninstall

core updates needed: 'drupal/advanced_text_formatter:^3.0@RC' 'drupal/bamboo_twig:^6.0' 'drupal/block_exclude_pages:^2.1@beta' 'drupal/config_sync:^3.0@alpha' 'drupal/pathologic:^2.0@alpha' 'drupal/twig_tweak:^3.2'

"drupal/devel": "^5.1"

test uninstalling these modules before upgrade: Blazy, TB megamenu, Bootstrap4, Slick, Slick Views, Video Embed Field, CKeditor(4) Google Analytics module is deprecated with GA4 - uninstall?

if you are running into requirements issues composer prohibits drupal/core 10

had to remove scss_compiler module to run update to d10 and was able to add it back

update 10/11 scss_compiler now has a d10 compatible version, so that has been updated, patches removed

nhjohnstone commented 11 months ago

updating to ckeditor5 for basic html text format gives error: CKEditor 5 only works with HTML-based text formats. The "Video Embed WYSIWYG" (video_embed_wysiwyg) filter implies this text format is not HTML anymore. https://www.drupal.org/project/video_embed_field/issues/3311063 --10/11 update there is now a patch here we can try to apply had to disable this, can embed video with media plugin --11/13 patch applied, the video embed button has been added back to ck editor and seems to be working

other warnings on upgrade to ckeditor5 The CKEditor 4 button Anchor does not have a known upgrade path. If it allowed editing markup, then you can do so now through the Source Editing functionality. The CKEditor 4 button video_embed does not have a known upgrade path. If it allowed editing markup, then you can do so now through the Source Editing functionality. The video_embed plugin settings do not have a known upgrade path. Updating to CKEditor 5 added support for some previously unsupported tags/attributes. A plugin introduced support for the following: These attributes: rowspan (for , ), colspan (for , ); Additional details are available in your logs.

the paste text and paste from word buttons are not in ckeditor 5 paste with shift added and the word code is stripped out https://ckeditor.com/docs/ckeditor5/latest/features/pasting/paste-plain-text.html

in the link tool dialog box, the media library button is no longer there there is a patch here we are using on extension site: https://www.drupal.org/project/linkit_media_library/issues/3329877

Screenshot 2023-11-13 at 9 26 23 AM

For Extension web site, we might use this module: https://www.drupal.org/project/imce, which opens a file browser instead of the media library. We need to link to PDFs.

Screenshot 2023-11-10 at 12 26 29 PM
nhjohnstone commented 11 months ago

Here is the upgrade status report before updating anything, gives info on what needs to change for ILFW modules upgrade-status-export-2023-09-07T20 22 51+0000.txt

nhjohnstone commented 11 months ago

Twig tweak updated to 3, there are some changes

trubach commented 4 months ago

Closing this issue since we've released the ILFW 4.0 with Drupal 10 support. We still need to implement CK Editor 5, but that is in a separate issue.