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

Double icon for images in the editor toolbar #9266

Merged
merged 1 commit into from Mar 1, 2016

Conversation

AlexRed
Copy link
Contributor

@AlexRed AlexRed commented Mar 1, 2016

Pull Request for Issue # .

Summary of Changes

Testing Instructions

After the plugin buttons that were at the bottom of the editor as been moved inside the editor toolbar, there are two double icons for images: one of TinyMCE and one Joomla.

This patch changes the icon of Joomla Image button, to distinguish it from the other one.

After the plugin buttons that were at the bottom of the editor as been moved inside the editor toolbar, there are two double icons for images: one of TinyMCE and one Joomla.

This patch changes the icon of Joomla Image button, to distinguish it from the other one.
@infograf768
Copy link
Member

Makes sense to differentiate them indeed.
before:
screen shot 2016-03-01 at 10 20 56
after
screen shot 2016-03-01 at 10 22 13

@infograf768
Copy link
Member

I have tested this item ✅ successfully on a6781fa


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

@stellainformatica
Copy link
Contributor

@test: patch tested, working fine, the two icons are differentiated


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

@infograf768
Copy link
Member

@stellainformatica
Please fill out the "Test this" to successful on https://issues.joomla.org/tracker/joomla-cms/9266

@stellainformatica
Copy link
Contributor

I have tested this item ✅ successfully on a6781fa

@test: patch tested, working fine, the two icons are differentiated


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

@infograf768
Copy link
Member

I guesss this can go in 3.5.0

@wilsonge @roland-d


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 1, 2016
wilsonge added a commit that referenced this pull request Mar 1, 2016
Double icon for images in the editor toolbar
@wilsonge wilsonge merged commit c99f0af into joomla:staging Mar 1, 2016
@wilsonge wilsonge added this to the Joomla! 3.5.0 milestone Mar 1, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 1, 2016
@wilsonge
Copy link
Contributor

wilsonge commented Mar 1, 2016

Merged - thanks!

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.

None yet

5 participants