Skip to content

Tinymce code icon in simple view#11657

Merged
wilsonge merged 6 commits intojoomla:stagingfrom
brianteeman:tinymce
Aug 18, 2016
Merged

Tinymce code icon in simple view#11657
wilsonge merged 6 commits intojoomla:stagingfrom
brianteeman:tinymce

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

@brianteeman brianteeman commented Aug 18, 2016

Summary of Changes

Testing Instructions

Switch tinymce into simple functionality
you will see the code editor AFTER the xtd-plugins

Switch tinymce into advanced functionality
you will see the code editor BEFORE the xtd-plugins with the link unlink icons

After the PR the icon will be before the xtd-plugins - see screenshots

Before

cmqv

After

09z 2

@ggppdk
Copy link
Copy Markdown
Contributor

ggppdk commented Aug 18, 2016

I have tested this item ✅ successfully on bfde487

The source code <> button does not have a label (text) next to its icon
so it should probably be before the XTD-buttons (more consistent ?)

also it does look better with this change


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

@dgrammatiko
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on bfde487


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

@dgrammatiko
Copy link
Copy Markdown
Contributor

RTC Thanks @brianteeman

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 18, 2016
@MATsxm
Copy link
Copy Markdown

MATsxm commented Aug 18, 2016

I have tested this item ✅ successfully on bfde487

👍


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

@wilsonge wilsonge merged commit b2a4b48 into joomla:staging Aug 18, 2016
@wilsonge wilsonge added this to the Joomla 3.6.3 milestone Aug 18, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 18, 2016
@brianteeman brianteeman deleted the tinymce branch August 18, 2016 22:14
ggppdk pushed a commit to ggppdk/joomla-cms that referenced this pull request Aug 19, 2016
* toolbar

* Revert "toolbar"

This reverts commit 8f995f1.

* tinymce code icon on toolbar
roland-d pushed a commit to roland-d/joomla-cms that referenced this pull request Sep 11, 2016
* toolbar

* Revert "toolbar"

This reverts commit 8f995f1.

* tinymce code icon on toolbar
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.

6 participants