I would also recommend adding composer "require-dev" to support development requirements.
Had to do a lot messing around to get things to show up, so I'm going to close this issue for now.
Another option is changing the min-height to 60px, from layout paragraphs builder.css => .js-lpb-region class. However, this still takes away how close it resembles the final rendering.
@timurtripp I can verify this works in my environment with safari.
Nice work!
I'm on Drupal 10.2 w/ CKE5.
Agreed that it seems to be a CKEditor5 issue with the widget and using arrows to navigate off them.
I did test the selection of text + icon for selection of link creation and discovered it is not an issue in chrome but is in Safari. For some reason Safari does not allow the selection of text and a widget for link creation.
@ timurtripp I can verify that icons can be linked with the patch.
However, I have to do it through source view as it is seems the popup menu gets in the way when selecting for linking. Even when I try and put my cursor in the middle of a link, it will break the link into two parts that bookend the icon. Not sure how to get around the popup selection either. If I want the cursor to be after the icon I have to use the arrow keys. If the icon is the last thing in the editor, clicking anywhere in the editor after the icon will select the icon. Thus forcing me to use the arrow keys to move the cursor past it in order to continue entering content. It seems like the popup menu is a bit to aggressive
esch → created an issue.
Pushed some updates that improve site creation with site name, admin password being set. Also, updated `druid uli`to generated first login url.
Recommend changing the install steps to avoid continuous DDEV config changes. I had a more current DDEV version, which updated the config file.
Here are the steps I recommend:
git clone git@git.drupal.org:issue/operations-3389217.git
cd operations
composer install
ddev start
ddev composer operations:install-ddev
Did not test with Lando either and agree about the duplicate commands. However, not sure how to get around it with the alias difference.
@johan-den-hollander thanks for the quick reply.
Unfortunately, we are on Pantheon. They don't allow additional DBs(https://docs.pantheon.io/guides/platform-considerations/platform-site-in...) or support third party hosted DBs. Thus our stuff has to go into the Drupal DB.
cheers,
matt
I have just tested the following, on Pantheon with Ping Federated:
Drupal: 10.1.3
PHP: 8.2.10
MariaDB: 10.4.25
NGINX: 1.25.2
Composer: 2.5.5
drupal/simplesamlphp_auth: dev-4.x
simplesamlphp/simplesamlphp: dev-simplesamlphp-2.1
NO PATCHES APPLIED
I was able to successfully authenticate without issue. However, I'm still getting the Drupal Status Report error: Transaction isolation level.
READ-COMMITTED
For this to work correctly, all tables must have a primary key. The following table(s) do not have a primary key: SimpleSAMLphp_saml_LogoutStore, SimpleSAMLphp_tableVersion. See the setting MySQL transaction isolation level page for more information.
#3375702-2 is working for me.
No worries @aangel. I was actually thinking of submitting a Merge Request since it was a simple item, but will defer to you.
cheers,
matt
esch → created an issue.
reordered the arrays to line up
adding myself to Ping list
I'm seeing this show up again in the 6.2.x-dev branch again.
I'm seeing this error message show up again in 6.2.x-dev branch again.
Having the same issue.