Open Bug 1344452 Opened 7 years ago Updated 7 years ago

Applying svg emoticons

Categories

(Other Applications :: ChatZilla, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: hunboy, Assigned: hunboy)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(1 file)

After landing Bug 1342944 we should apply the svg version of emoticons
Attached patch the css patchSplinter Review
For secure reasons I will patch the new emoticons in a separated bugs. These are the currntly exists.
Assignee: rginda → utasir
Status: NEW → ASSIGNED
Attachment #8843551 - Flags: review?(bugzilla-mozilla-20000923)
Depends on: 1342944
Comment on attachment 8843551 [details] [diff] [review]
the css patch

Review of attachment 8843551 [details] [diff] [review]:
-----------------------------------------------------------------

Do the emoticons keep their current 16px by 16px size with this patch? It would be cool to follow-up with making them scale.

r=silver
Attachment #8843551 - Flags: review?(bugzilla-mozilla-20000923) → review+
If we want fontSize dependent emoticons, we need use as background-image and background-size: cover properties with an edited SVGs where we remove the fixed size attributes from the SVG root element.
Unfortunately the content attribute ignores the css defined width, height.

We can file a separated bug for it. I filed Bug 1362768 for it.
Blocks: 1362768
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: