Skip to content

Fix modal forms for editor#12077

Merged
andreslucena merged 3 commits intodevelopfrom
fix/editor-modals
Nov 28, 2023
Merged

Fix modal forms for editor#12077
andreslucena merged 3 commits intodevelopfrom
fix/editor-modals

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Nov 25, 2023

🎩 What? Why?

Link and Video layout in editor are broken. This PR fixes it.

📌 Related Issues

Link your PR to an issue

Testing

  1. Login as admin
  2. Enter organization edit page
  3. In the Terms of Service editor
  4. Any piece of text and press "link" icon
  5. See the layout of pop-up
  6. Close the pop-up
  7. Click on Insert video button
  8. See the layout of pop-up
  9. Apply patch
  10. Repeat 4,5,6,7,8

📷 Screenshots

Please add screenshots of the changes you are proposing

Before

Link
image

Video
image

After

LInk
image

Video
image

♥️ Thank you!

@alecslupu alecslupu added type: fix PRs that implement a fix for a bug project: redesign Barcelona City Council contract labels Nov 25, 2023
github-actions[bot]
github-actions bot previously approved these changes Nov 25, 2023
github-actions[bot]
github-actions bot previously approved these changes Nov 25, 2023
@alecslupu alecslupu requested a review from a team November 25, 2023 12:29
@alecslupu alecslupu marked this pull request as ready for review November 25, 2023 12:29
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed for video modal.

Can you fix the image modal 🙏🏽?

Screenshot of the image modal in the WYSIWYG editor

@alecslupu
Copy link
Copy Markdown
Contributor Author

Fixed in 492bd0f

image

Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@andreslucena andreslucena added the no-backport Pull Requests that should not be backported label Nov 28, 2023
@andreslucena
Copy link
Copy Markdown
Member

Only failing check is from codecov

@andreslucena andreslucena merged commit 2a32858 into develop Nov 28, 2023
@andreslucena andreslucena deleted the fix/editor-modals branch November 28, 2023 15:15
entantoencuanto added a commit that referenced this pull request Nov 29, 2023
…/redesign-staging

* fix/control-comments-replies-polymorphism:
  Add a polymorphic option to restrict acts_as_tree lookup
  Add tests to check the 'Hide replies' message error
  Fix pipeline after translations were updated (#12084)
  Remove the duplicate reference of icons (#12083)
  Fix modal forms for editor (#12077)
  New Crowdin updates (#12003)
  Display only once resource type in search (#12041)
  Redesign: fix responsiveness on nav-bar admin (#12045)
  Apply flash styles to Announcements (part 2) (#12039)
  Remove legacy design support from WYSIWYG editor (TipTap) (#12015)
  Extract methods from meetings seeds (#12053)
  Show message when there are no content in the page (#12056)
  Show message when there are no sortitions (#12057)
  Fix icons on flash alerts (#12069)
  Update numbering in RELEASE_NOTES.md (#12066)
  Add prerequirements to the manual installation tutorial (#12068)
  Differentiate non-clickable elements in admin menu (#12000)
  Remove official logo header image (#12064)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-backport Pull Requests that should not be backported project: redesign Barcelona City Council contract type: fix PRs that implement a fix for a bug

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Layout broken for editor pop-ups

2 participants