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

Icon for the Toolbar in 3.x #23

Merged
merged 1 commit into from
Apr 27, 2014
Merged

Icon for the Toolbar in 3.x #23

merged 1 commit into from
Apr 27, 2014

Conversation

zero-24
Copy link

@zero-24 zero-24 commented Apr 27, 2014

Currently we have no icon at the toolbar. This PR will change (or more fallback) to a circle with a point in it. As the others that are in the images folder don't look so good at this place i think.

But if we want to have e.g. the red beetle (https://github.com/joomla-extensions/patchtester/tree/master/administrator/components/com_patchtester/assets/images)

I can change this PR. Let me know :)
Thanks :)

Currently we have no icon at the toolbar. This PR will change (or more fallback) to a circle with a point in it. As the others that are in the images folder don't look so good.

But if we want to have e.g. the red beetle (https://github.com/joomla-extensions/patchtester/tree/master/administrator/components/com_patchtester/assets/images)

I can change this PR

Let me know :)
@mbabker
Copy link

mbabker commented Apr 27, 2014

This works for now, if we find something in the icon set (http://www.nonr.nl/jui/index.php/icons) that's more appropriate then we can always change it later.

mbabker added a commit that referenced this pull request Apr 27, 2014
Icon for the Toolbar in 3.x
@mbabker mbabker merged commit f18da36 into joomla-extensions:master Apr 27, 2014
@zero-24 zero-24 deleted the patch-1 branch April 27, 2014 15:50
@zero-24
Copy link
Author

zero-24 commented Apr 27, 2014

👍

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

2 participants