Open
Bug 1344452
Opened 7 years ago
Updated 7 years ago
Applying svg emoticons
Categories
(Other Applications :: ChatZilla, enhancement)
Other Applications
ChatZilla
Tracking
(Not tracked)
ASSIGNED
People
(Reporter: hunboy, Assigned: hunboy)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(1 file)
4.55 KB,
patch
|
bugzilla-mozilla-20000923
:
review+
|
Details | Diff | Splinter Review |
After landing Bug 1342944 we should apply the svg version of emoticons
Assignee | ||
Comment 1•7 years ago
|
||
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)
Comment 2•7 years 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•7 years 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.
You need to log in
before you can comment on or make changes to this bug.
Description
•