skin/icons/trash.svg - Issue 29596678: Issue 5973 - broken icons on Firefox v50-53

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: skin/icons/trash.svg

Issue 29596678: Issue 5973 - broken icons on Firefox v50-53 (Closed)

Patch Set: Created Nov. 3, 2017, 5:54 p.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
1 <svg id="e4fee3cd-bcdb-4c90-95b7-79c640275902" data-name="artwork" xmlns="http:/ /www.w3.org/2000/svg" viewBox="0 0 48 48"><title>trash</title><rect x="22.96" y= "16" width="2" height="22"/><rect x="29.96" y="16" width="2" height="22"/><rect x="15.96" y="16" width="2" height="22"/><path d="M44,7H33V5a5,5,0,0,0-5-5H20a5,5 ,0,0,0-5,5V7H4V9H6.68V39a9,9,0,0,0,9,9H32.23a9,9,0,0,0,9-9V9H44ZM17,5a3,3,0,0,1, 3-3h8a3,3,0,0,1,3,3V7H17ZM39.23,39a7,7,0,0,1-7,7H15.68a7,7,0,0,1-7-7V9H39.23Z"/> </svg>	1 <svg xmlns="http://www.w3.org/2000/svg">
	saroyanm 2017/11/03 18:02:26 This Icon is being used in mobile version as well. This Icon is being used in mobile version as well. saroyanm 2017/11/03 18:28:29 Updated mobile-options.html to use "ID". Show quoted text On 2017/11/03 18:02:26, saroyanm wrote: > This Icon is being used in mobile version as well. Updated mobile-options.html to use "ID".
	2 <view id="default" viewBox="0 0 48 48"/>

	3 <rect x="22.96" y="16" width="2" height="22" fill="#099DD1"/>

	4 <rect x="29.96" y="16" width="2" height="22" fill="#099DD1"/>

	5 <rect x="15.96" y="16" width="2" height="22" fill="#099DD1"/>

	6 <path d="M44,7H33V5a5,5,0,0,0-5-5H20a5,5,0,0,0-5,5V7H4V9H6.68V39a9,9,0,0,0,9,9 H32.23a9,9,0,0,0,9-9V9H44ZM17,5a3,3,0,0,1,3-3h8a3,3,0,0,1,3,3V7H17ZM39.23,39a7,7 ,0,0,1-7,7H15.68a7,7,0,0,1-7-7V9H39.23Z" fill="#099DD1"/>

	7

	8 <view id="hover" viewBox="0 48 48 48"/>

	9 <rect x="22.96" y="16" width="2" height="22" transform="translate(0, 48)" fill ="#5CBCE1"/>

	10 <rect x="29.96" y="16" width="2" height="22" transform="translate(0, 48)" fill ="#5CBCE1"/>

	11 <rect x="15.96" y="16" width="2" height="22" transform="translate(0, 48)" fill ="#5CBCE1"/>

	12 <path d="M44,7H33V5a5,5,0,0,0-5-5H20a5,5,0,0,0-5,5V7H4V9H6.68V39a9,9,0,0,0,9,9 H32.23a9,9,0,0,0,9-9V9H44ZM17,5a3,3,0,0,1,3-3h8a3,3,0,0,1,3,3V7H17ZM39.23,39a7,7 ,0,0,1-7,7H15.68a7,7,0,0,1-7-7V9H39.23Z" transform="translate(0, 48)" fill="#5CB CE1"/>

	13 </svg>

OLD	NEW

« skin/icons/checkmark.svg ('K') | « skin/icons/tooltip.svg ('k') | skin/social/facebook.svg » ('j') | no next file with comments »