πŸ‡§πŸ‡ͺBelgium @Thomasdbcklr

Account created on 9 July 2014, over 10 years ago
#

Merge Requests

Recent comments

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ changed the visibility of the branch 3425357 to active.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ changed the visibility of the branch 3425357 to hidden.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Pushed a fix, things we could do better:

* I disabled the following component file // import '../../../../core/themes/classy/css/components/file.css'; // for file upload field
** Could just reference the contrib theme instead?
* Instead of disabling the ESlint in the webpack config make sure it works?

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ changed the visibility of the branch 3415469-fix-storybook-broken to active.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ changed the visibility of the branch 3415469-fix-storybook-broken to hidden.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

We are using the following which are deprecated functions in D10 and will not work

$theme_path = drupal_get_path('theme', 'rocketship_theme_starter');
$variables['title'] = render($variables['page']['#title']);

Would have liked to create a start point already but it seems the ['page']['#title'] has been changed as well and needs further looking into.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Had problems with this one while working on #3368324 πŸ“Œ Replace Classy core theme by contrib theme Fixed

Our gulp tasks require certain .css files to exists mainly

  'css:fonts',
  'css:mail',
  'css:editor'

When compiling you'll get the following error:
Error: File not found with singular glob: /rocketship_theme_starter/css/style.editor.css (if this was purposeful, use `allowEmpty` option)

We can either keep empty .css files or figure out how to change these tasks that they don't require the files to be present at first ( haven't looked into how we could do this )

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Scope has changed to require the contrib theme instead, so we can potentially convert our generator to use the core Starterkit generator instead

  • Bumped the generated theme .info file's core version to 9 || 10
  • Added classy, components & unified_twig_ext in composer.json because we require their latest versions for D10
πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Added the needed ckeditor5-stylesheets reference
Kept the old ckeditor_stylesheets as well for some backwards compatibility
Updated the ckeditor styles we have so they work for both 4 & 5 for the moment.
Added some basic font-family support for ckeditor 5 as well

πŸ‡§πŸ‡ͺBelgium Thomasdbcklr

Thomasdbcklr β†’ made their first commit to this issue’s fork.

Production build 0.71.5 2024