2. it made 30 requests to load 2 comments, including fetching the same url 5 times.
3. you call the APIs in serial, but you can (should) do it in parallel.
4. you're using async=false on `xmlhttp.open("GET", filePath, false);`. this is a bad practice. also, consider using `fetch` which is a more modern API.
5. you can omit `?print=pretty` and the fetch will be smaller, probably parse faster, etc.
- More legible than HN, a combo of size and color
- Too much space below short entries ?
- Context menu (rclick) on links hijacked, no 'open in new tab'
OuterVale|1 year ago
GaryNumanVevo|1 year ago
gabrielsroka|1 year ago
1. it's really slow.
2. it made 30 requests to load 2 comments, including fetching the same url 5 times.
3. you call the APIs in serial, but you can (should) do it in parallel.
4. you're using async=false on `xmlhttp.open("GET", filePath, false);`. this is a bad practice. also, consider using `fetch` which is a more modern API.
5. you can omit `?print=pretty` and the fetch will be smaller, probably parse faster, etc.
6. the back button doesn't work
compare my version (which does much less, but does it faster) https://github.com/gabrielsroka/gabrielsroka.github.io/blob/... and https://gabrielsroka.github.io/hn.html?small
jasontheone111|1 year ago
claxo|1 year ago
jasontheone111|1 year ago
pndy|1 year ago
smt88|1 year ago
jasontheone111|1 year ago
unknown|1 year ago
[deleted]