6 ms·
I redesigned Hacker News (But I used the API this time)
- deleted 2y ago[deleted]
- claxo 2y ago- 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'
- jasontheone111 2y agoIt opens in new tab by default when you click on the source
- pndy 2y agoYesterday: https://news.ycombinator.com/item?id=40912583 https://news.ycombinator.com/item?id=40912583
- smt88 2y agoFar, far worse on iPhone 12 Mini. Genuinely unreadable.
- jasontheone111 2y agoI forgot to add adaptive CSS.
- OuterVale 2y agoI loaded it up, and it hung fetching things for ~5 seconds. Part of what makes Hacker News great is its speed and simplicity.
- GaryNumanVevo 2y agoSame here, incredibly slow to load
- gabrielsroka 2y agoI like some of it, but: 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/master/hn.html https://github.com/gabrielsroka/gabrielsroka.github.io/blob/... and https://gabrielsroka.github.io/hn.html?small https://gabrielsroka.github.io/hn.html?small
- jasontheone111 2y agoThank you for the info, I will work on speeding up my website :)