Skip to content

[4.0] TinyMCE builder pluigin#27466

Merged
Quy merged 1 commit intojoomla:4.0-devfrom
brianteeman:buildermargin
Jan 10, 2020
Merged

[4.0] TinyMCE builder pluigin#27466
Quy merged 1 commit intojoomla:4.0-devfrom
brianteeman:buildermargin

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

(there may be a better way to do this but I couldnt find it)

Before

image

After

image

testing

don't forget npm i

(there may be a better way to do this but I couldnt find it)

### Before

### After
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jan 10, 2020
@jwaisner
Copy link
Copy Markdown
Member

I have tested this item 🔴 unsuccessfully on 1f57e61


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

@jwaisner
Copy link
Copy Markdown
Member

After patching and completing npm i, the alignment did not change as the expected result. I tested this in Firefox and Chrome.
27466

@brianteeman
Copy link
Copy Markdown
Contributor Author

Did you run npm i to rebuild the scripts?

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 10, 2020

I have tested this item ✅ successfully on 1f57e61


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 10, 2020

@jwaisner Try clearing browser's cache.

@jwaisner
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 1f57e61

@brianteeman yes

@Quy just tried that and it didnt work

End result, I did a complete reset of my environment and it works as expected.


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

@Quy Quy removed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jan 10, 2020
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 10, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 10, 2020
@Fedik

This comment has been minimized.

@Fedik
Copy link
Copy Markdown
Member

Fedik commented Jan 10, 2020

no, sorry, ignore my previous comment

@Quy Quy merged commit e52d662 into joomla:4.0-dev Jan 10, 2020
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Jan 10, 2020
@Quy Quy added this to the Joomla 4.0 milestone Jan 10, 2020
@brianteeman
Copy link
Copy Markdown
Contributor Author

Thanks

@brianteeman brianteeman deleted the buildermargin branch January 10, 2020 20:20
brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request Feb 4, 2020
(there may be a better way to do this but I couldnt find it)

### Before

### After
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants