Status

Other Applications
ChatZilla
--
enhancement
ASSIGNED
a year ago
a year ago

People

(Reporter: hunboy, Assigned: hunboy)

Tracking

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

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
After landing Bug 1342944 we should apply the svg version of emoticons
(Assignee)

Comment 1

a year ago
Created attachment 8843551 [details] [diff] [review]
the css patch

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)
(Assignee)

Updated

a year ago
Depends on: 1342944

Comment 2

a year ago
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+
(Assignee)

Comment 3

a year ago
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.
(Assignee)

Updated

a year ago
Blocks: 1362768
You need to log in before you can comment on or make changes to this bug.