elevatorsaga icon indicating copy to clipboard operation
elevatorsaga copied to clipboard

Published 20 hours ago • verified publisher icon magwo

Show user times

Open mhelsley opened this issue 10 years ago • 3 comments

This series adds some summary statistics, reformats the statistics box to more compactly show the average and max stats, and enables displaying of time values on individual elevator users so that players can better understand the flaws in their code.

This should solve the following issues reported on github: Issue #77 Issue #52

mhelsley avatar Feb 27 '15 00:02 mhelsley

Comments? Questions?

mhelsley avatar Mar 21 '15 23:03 mhelsley

Several good changes but I'm worried about performance.

magwo avatar Mar 22 '15 10:03 magwo

I haven't looked at rendering performance. If it does make it worse my guess is the table layout code is what would do it. Since the table's number of rows and columns never change the most likely thing that could cause the layout code to do work is a change in the value of the table entries reflecting the stats. I used jQuery's text() rather than HTML to try to avoid any layout reflows but that may not be sufficient.

If I can't use a table there then the stats list gets rather long. I could probably use div + span with hardcoded widths -- after all, the size of font and total available width are already fixed.

mhelsley avatar Mar 26 '15 19:03 mhelsley