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
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
Update method for using Icons
Re: Update method for using Icons
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.
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 (5.82 KiB) Viewed 26909 times
Re: Update method for using Icons
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.
unfortunately you can not include hot topic as its animated
unfortunately you can not include hot topic as its animated
Re: Update method for using Icons
Using free tools pngcrush and PNGZopfli, you can compress this figure to 11.3kb without losing quality.hanakin wrote:12-13kb request as follows
Sorry for my English.
-
- Registered User
- Posts: 523
- Joined: Sat Apr 22, 2006 10:29 pm
- Contact:
Re: Update method for using Icons
Nice work on the new icons!
Re: Update method for using Icons
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.
Might be able to use it for the composites forum icons.
- callumacrae
- Former Team Member
- Posts: 1046
- Joined: Tue Apr 27, 2010 9:37 am
- Location: England
- Contact:
Re: Update method for using Icons
-1 to sprites. They won't make it into phpBB before HTTP 2.0 is supported in most major browsers.
- FloFlo91
- Registered User
- Posts: 11
- Joined: Sun Jun 08, 2014 4:06 pm
- Location: Paris, France
- Contact:
Re: Update method for using Icons
+1 Hanakin, We have to reduce the amount of icons by using sprites and font icons.
I'm sorry but what means FA please ?nickvergessen wrote:Let's just switch to FA in 3.2 instead of changing it twice in a row
- 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
It's Font Awesome: http://fortawesome.github.io/Font-Awesome/
Re: Update method for using Icons
ok slight update it just about working any inputs on icon fixes & changes are welcome?
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
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