Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Feature/fixes #50

Merged
merged 10 commits into from
Nov 12, 2015
Merged

Feature/fixes #50

merged 10 commits into from
Nov 12, 2015

Conversation

FinalAngel
Copy link
Member

adds nice icons/preview for all text enabled plugins

image

…ature/fixes

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@@ -71,6 +70,8 @@ def render(self, context, instance, placeholder):
context.update({'instance': instance})
return context

def icon_src(self, instance):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are passing instance but not using it. is that to satisfy a pre-defined interface?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

icon_src requires the instance argument otherwise:

TypeError at /en/admin/cms/page/edit-plugin/82/ icon_src() takes exactly 1 argument (2 given)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

k

@intellectronica
Copy link
Contributor

👍

FinalAngel added a commit that referenced this pull request Nov 12, 2015
@FinalAngel FinalAngel merged commit db82d0e into develop Nov 12, 2015
@FinalAngel FinalAngel deleted the feature/fixes branch November 12, 2015 14:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants