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

[5.0] tinyMCE styles #41945

Merged
merged 2 commits into from
Sep 27, 2023
Merged

[5.0] tinyMCE styles #41945

merged 2 commits into from
Sep 27, 2023

Conversation

dgrammatiko
Copy link
Contributor

@dgrammatiko dgrammatiko commented Sep 27, 2023

Pull Request for Issue #40622 .

Summary of Changes

  • Add the missing separator
  • Remove the outer border radius

Testing Instructions

Run npm ci and create an article, check the tinyMCE editor

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Screenshot 2023-09-27 at 14 31 43
Screenshot 2023-09-27 at 14 31 32

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@HLeithner

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev labels Sep 27, 2023
@brianteeman brianteeman mentioned this pull request Sep 27, 2023
@richard67 richard67 added the bug label Sep 27, 2023
@Quy
Copy link
Contributor

Quy commented Sep 27, 2023

I have tested this item ✅ successfully on ce9c9f6


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41945.

@HLeithner HLeithner merged commit 6766fb8 into joomla:5.0-dev Sep 27, 2023
3 of 4 checks passed
@HLeithner
Copy link
Member

thx

@dgrammatiko dgrammatiko deleted the 5.0-dev-tinyMCE branch September 27, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants