Closed (fixed)
Project:
Smiley
Version:
7.x-1.0-beta2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
1 Apr 2012 at 16:56 UTC
Updated:
12 Jun 2012 at 23:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
fizk commentedDo you want to add a class by overriding a theme function or by typing it in on the administration settings pages?
Comment #2
samgreco commentedSeems like having a choice in the admin settings would be a good idea.
Comment #3
samgreco commentedAny further thoughts on this?
Comment #4
fizk commentedFixed. Try using the latest 7.x dev release in 24 hours and let me know if it works for you.
Comment #5
samgreco commentedDoesn't seem to be working. I get the selection in the config form, but not getting the class in the rendered output. Nothing is different.
Comment #6
fizk commentedCan you attach the smiley.admin.inc file, I want to double check that you have the right version.
Comment #7
samgreco commentedSure.
Comment #8
fizk commentedThanks, can you also attach smiley.module.
What did you put into the "CSS class to add to smiley images" field?
Comment #9
samgreco commentedI just added the word smiley. Did I misunderstand how it works?
Comment #10
fizk commentedCan you try disabling the module, then re-enabling the module? Also try clearing the cache.
Comment #11
samgreco commentedSorry tried both and no change.
Comment #12
fizk commentedAssuming that you tested this by creating a new node and adding a smiley to it and then looking for the class attached to the smiley img tag, I don't know why it's not working for you and I can't reproduce this.
Comment #13
samgreco commentedActually I didn't create a new node. I will try that and reply.