Account created on 9 November 2007, over 16 years ago
  • Senior Manager, Web Systems at WETAΒ 
#

Merge Requests

Recent comments

πŸ‡ΊπŸ‡ΈUnited States jesss
  • Moved the point about how the issue should describe how the change will affect stakeholders from the review section to the list of items the issue should include when created.
  • Tightened up the line about stakeholders with a vested interest. Their vested interest is in the change, not in the discussion about the change.
πŸ‡ΊπŸ‡ΈUnited States jesss

Commenting to note that bsnodgrass had asked me to review the Google Doc, which I did. I made a number of suggestions to add clarity and improve the document flow and organization.

πŸ‡ΊπŸ‡ΈUnited States jesss

Also:

joarferme
jdearie
bendygirl
alex-ua
jennaharrismosley
klanahan
ptkoroma

πŸ‡ΊπŸ‡ΈUnited States jesss

I am here with:
karen11
fogpuddle
nneka

πŸ‡ΊπŸ‡ΈUnited States jesss

I'm on 3.0.0-beta4. Did you clear cache after applying the patch?

πŸ‡ΊπŸ‡ΈUnited States jesss

Patch #86 is working for me!

πŸ‡ΊπŸ‡ΈUnited States jesss

Added #nonprofits to the list of popular channels.

πŸ‡ΊπŸ‡ΈUnited States jesss

@fjgarlin, thanks for working on this! I'd be happy to review, but I don't know the password for the testing page.

πŸ‡ΊπŸ‡ΈUnited States jesss

I am still seeing this behavior in alpha14 on 10.2.3. Any chance of a fix?

πŸ‡ΊπŸ‡ΈUnited States jesss

From what I'm seeing on my site, there is no actual data loss. It's simply that when rendering the field, the text filter is stripping out tags that should be allowed. If you go to edit the content, all those "missing" tags are still there.

πŸ‡ΊπŸ‡ΈUnited States jesss

Coming here from πŸ› Several elements are missing from ckeditor5 Basic HTML Active .

My site's content is missing basic HTML tags after updating to 10.2.0 -- most noticeably bold and links. Turning off the Limit HTML Tags filter restored the missing tags but as others have noted, that is not a good long-term solution.

When I turned the Limit HTML Tags filter back on again, I was presented with the following error message:

The current CKEditor 5 build requires the following elements and attributes:
<br> <p> <h2> <h3> <h4> <h5> <h6> <* dir="ltr rtl" lang> <cite> <dl> <dt> <dd> <span> <figure> <figcaption> <iframe src height width allowfullscreen frameborder> <strong> <em> <code> <blockquote> <a href data-entity-type data-entity-uuid data-entity-substitution> <ul> <ol reversed start> <li> <img src alt height width data-entity-uuid data-entity-type data-caption data-align> <drupal-media data-entity-type data-entity-uuid alt data-caption data-align>
The following elements are missing:
<strong> <em> <code> <blockquote> <a href data-entity-type data-entity-uuid data-entity-substitution> <ul> <ol reversed start> <li> <img src alt height width data-entity-uuid data-entity-type data-caption data-align> <drupal-media data-entity-type data-entity-uuid alt data-caption data-align>

Of course, the field where I would add those elements back in is read-only...

With CKEditor 5 this is a read-only field. The allowed HTML tags and attributes are determined by the CKEditor 5 configuration. Manually removing tags would break enabled functionality, and any manually added tags would be removed by CKEditor 5 on render.

I do have source editing enabled (as well as the CodeMirror plugin), and everything worked without issue prior to updating core.

πŸ‡ΊπŸ‡ΈUnited States jesss

#6 (MR !23) resolves the warnings I've been seeing. Thank you!

πŸ‡ΊπŸ‡ΈUnited States jesss

No joy. This patch doesn't solve the problem -- it just moves it down a few lines.

There is a $configuration['id'] set with inline blocks. The issue is that it only has two components, not four. So the explode snippet itself needs to be reworked to account for use cases with fewer than four components.

πŸ‡ΊπŸ‡ΈUnited States jesss

jesss β†’ created an issue.

πŸ‡ΊπŸ‡ΈUnited States jesss

My apologies for the confusion -- we're using the core Media Library widget, not Entity Browser.

Steps to reproduce:

  1. Add an autocomplete entity reference field to a media type using the Media Library widget on the form display.
  2. Create a paragraph type for that media type.
  3. Within Mercury Editor, add a paragraph of that type.
  4. After uploading your media, attempt to add an entity reference to that media item. See how the autocomplete dropdown is hidden by the modal.
πŸ‡ΊπŸ‡ΈUnited States jesss

I will note that Queue Order does do what it says on the tin, and allow you to give certain queue's more "weight" than others, and those annotations remain in the codebase for this module. However, there are other methods to control queue operations (such as Ultimate Cron) and Queue Order can cause conflicts. If Queue Order is the preferred method for a website, it can be installed separately, without the dependency.

πŸ‡ΊπŸ‡ΈUnited States jesss

I just tried to apply patch #202 and while the patch applied cleanly, it didn't seem to do anything -- I still don't have the option to display the selected entities as a table. I believe that's because the updated schema that has the new widget settings isn't imported if the module has already been enabled. I saw there was some discussion upthread about removing an update hook -- that probably needs to be revisited.

Production build 0.69.0 2024