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

Show Print and Email Icons #11261

Merged
merged 2 commits into from
Aug 13, 2016
Merged

Show Print and Email Icons #11261

merged 2 commits into from
Aug 13, 2016

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Jul 23, 2016

There is some confusion on these options as the first option refers to an icon or icon + text
and the individual options are nothing to do with the icons at all.

This PR corrects that and standardises the tooltips.

You can find this in the article options

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Jul 23, 2016
@MATsxm
Copy link

MATsxm commented Jul 23, 2016

I have tested this item ✅ successfully on 7039b57

On review - Thanks


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

@ghost
Copy link

ghost commented Jul 23, 2016

I have tested this item ✅ successfully on 7039b57


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

@brianteeman
Copy link
Contributor Author

Rtc thanks


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 23, 2016
@brianteeman brianteeman added this to the Joomla 3.6.2 milestone Jul 25, 2016
@rdeutz rdeutz merged commit e7d59de into joomla:staging Aug 13, 2016
@brianteeman brianteeman deleted the icons branch August 13, 2016 17:11
@brianteeman brianteeman removed the RTC This Pull Request is Ready To Commit label Aug 13, 2016
ggppdk pushed a commit to ggppdk/joomla-cms that referenced this pull request Aug 19, 2016
* Change menu to link

* Update print email options
roland-d pushed a commit to roland-d/joomla-cms that referenced this pull request Sep 11, 2016
* Change menu to link

* Update print email options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants