Update method for using Icons

These RFCs were either rejected or have been replaced by an alternative proposal. They will not be included in phpBB.
User avatar
hanakin
Infrastructure Team
Infrastructure Team
Posts: 789
Joined: Sat Dec 25, 2010 9:02 pm
Contact:

Re: Update method for using Icons

Post by hanakin » Sun Jun 08, 2014 5:01 pm

ok so currently with all the planned changes and RFCs their are 43 unique server requests to pull 180kb for all the icons. This is not counting any of the icons for the contact menu in the profile.

by changing the method now w can reduce these requests down to 1, 12-13kb request as follows
navigation-icons.png
navigation-icons.png (12.67 KiB) Viewed 5168 times
I took some liberties with some of the icons as they really made no sense and its still a WIP so any recommendations of icon changes are welcome

aleha
Registered User
Posts: 143
Joined: Tue Mar 26, 2013 2:19 am

Re: Update method for using Icons

Post by aleha » Sun Jun 08, 2014 6:52 pm

Icons that also need to be updated are those big circle icons at the left of the Topic title. They seem to be kinda blurry.

These icons are clickable if you have unread posts on the topic, and take you to the last unread post.
But also the small icons appear and are clickable if you have unread posts on the topic.
Maybe this should change because we have two icons next to each other for the same usage.
Attachments
circle2.png
circle2.png (5.82 KiB) Viewed 5155 times

User avatar
hanakin
Infrastructure Team
Infrastructure Team
Posts: 789
Joined: Sat Dec 25, 2010 9:02 pm
Contact:

Re: Update method for using Icons

Post by hanakin » Sun Jun 08, 2014 7:02 pm

yes they need to be redone utilizing sprites as well. I pushed for this 4 yrs ago with the initial change to using css, but at the time some were reluctant to use sprites. if you wish to tackle this have at it.
imageset.png
imageset.png (15.06 KiB) Viewed 5151 times
unfortunately you can not include hot topic as its animated

User avatar
Sumanai
Registered User
Posts: 95
Joined: Sat Aug 31, 2013 11:12 am

Re: Update method for using Icons

Post by Sumanai » Sun Jun 08, 2014 7:15 pm

hanakin wrote:12-13kb request as follows
Using free tools pngcrush and PNGZopfli, you can compress this figure to 11.3kb without losing quality.
Sorry for my English.

User avatar
keith10456
Registered User
Posts: 523
Joined: Sat Apr 22, 2006 10:29 pm
Contact:

Re: Update method for using Icons

Post by keith10456 » Sun Jun 08, 2014 9:05 pm

Nice work on the new icons!

User avatar
PayBas
Registered User
Posts: 305
Joined: Tue Jul 29, 2008 6:08 pm
Contact:

Re: Update method for using Icons

Post by PayBas » Sun Jun 08, 2014 10:24 pm

Lets keep an eye on this: https://github.com/FortAwesome/Font-Awesome/issues/3640

Might be able to use it for the composites forum icons.

User avatar
callumacrae
Infrastructure Team
Infrastructure Team
Posts: 1046
Joined: Tue Apr 27, 2010 9:37 am
Location: England
Contact:

Re: Update method for using Icons

Post by callumacrae » Mon Jun 09, 2014 6:18 pm

-1 to sprites. They won't make it into phpBB before HTTP 2.0 is supported in most major browsers.
Made by developers, for developers!
My blog

User avatar
FloFlo91
Registered User
Posts: 11
Joined: Sun Jun 08, 2014 4:06 pm
Location: Paris, France
Contact:

Re: Update method for using Icons

Post by FloFlo91 » Sat Jun 14, 2014 7:38 pm

+1 Hanakin, We have to reduce the amount of icons by using sprites and font icons.
nickvergessen wrote:Let's just switch to FA in 3.2 instead of changing it twice in a row
I'm sorry but what means FA please ?

User avatar
M.Gaetan89
Registered User
Posts: 64
Joined: Tue Jan 28, 2014 7:17 pm
Location: Divonne-les-Bains, France
Contact:

Re: Update method for using Icons

Post by M.Gaetan89 » Sat Jun 14, 2014 7:40 pm


User avatar
hanakin
Infrastructure Team
Infrastructure Team
Posts: 789
Joined: Sat Dec 25, 2010 9:02 pm
Contact:

Re: Update method for using Icons

Post by hanakin » Wed Jul 23, 2014 7:39 pm

ok slight update it just about working any inputs on icon fixes & changes are welcome?
icons.png
icons.png (13 KiB) Viewed 4832 times
PSD file: https://www.dropbox.com/s/6awigiemq4kv1 ... -icons.psd

Ticket: https://tracker.phpbb.com/browse/PHPBB3-12888
PR: https://github.com/phpbb/phpbb/pull/2774

Locked