show firstPaint by default on summary when IE is used closes #340

This commit is contained in:
soulgalore 2014-01-03 14:23:51 +01:00
parent 156d60a24b
commit bdb01d2cc7
2 changed files with 3 additions and 3 deletions

View File

@ -2,7 +2,7 @@ CHANGELOG sitespeed.io
version 2.4-wip
------------------------
* If Chrome is used, display firstPaintTime in the summary as default #307
* If Chrome or IE is used, display firstPaintTime in the summary as default #307
* Added more default fields in the summary: requestsMissingGzip, jsWeightPerPage, cssWeightPerPage #325
* Changed order of the summary fields so that logical fields are grouped together
* Made it clearer that CSS & JS weight are per file in the summary (meaning inline CSS/JS are not included)

View File

@ -288,8 +288,8 @@ if [ "$SUMMARY_BOXES" != "" ]
if $COLLECT_BROWSER_TIMINGS
then
SUMMARY_BOXES="$SUMMARY_BOXES",serverResponseTime,backEndTime,pageDownloadTime,frontEndTime,domContentLoadedTime,pageLoadTime
## Extra: If we use chrome, always add the firstpaint
if [[ "$BROWSER_TIME_PARAMS " == *chrome* ]]
## Extra: If we use chrome or IE , always add the firstpaint
if [[ "$BROWSER_TIME_PARAMS " == *chrome* ]] || [[ "$BROWSER_TIME_PARAMS " == *ie* ]]
then
SUMMARY_BOXES="$SUMMARY_BOXES",firstPaintTime
fi