Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pretty article #13

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Pretty article #13

wants to merge 3 commits into from

Conversation

abbydrury
Copy link
Contributor

@abbydrury abbydrury commented Jul 30, 2024

Description

Theme components for a new "pretty article" content type (eventually possibly just the regular article), including paragraph types for banner and resource panel.

Future Work

If you copy/paste HTML code that inserts a FontAwesome icon into CKEditor 5 text areas and save the page, you'll be unable to edit it again without disabling JavaScript – for some reason, the combination of CKEditor 5 + edit + FontAwesome tags in content boxes (not FontAwesome-specific fields) + JavaScript takes over the CPU (I have seen as high as 300% in the task manager in Chrome) and the browser locks up.

Disabling JavaScript, removing the tag, and saving the node restores the ability to edit with JavaScript enabled. 🤷‍♀️
:thanksIHateIt:

Testing

Describe how the tester could verify that this branch works. Did you add any automated tests?

To be completed by the person testing

  • Visual confirmation of new functionality
  • Tests pass
  • Related documentation has been updated

…ad for the accessibility of the site at 200% zoom on small viewports and doesn't really seem to gain us anything
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant