TE
科技回声
首页24小时热榜最新最佳问答展示工作
GitHubTwitter
首页

科技回声

基于 Next.js 构建的科技新闻平台,提供全球科技新闻和讨论内容。

GitHubTwitter

首页

首页最新最佳问答展示工作

资源链接

HackerNews API原版 HackerNewsNext.js

© 2025 科技回声. 版权所有。

Show HN: Improved My News App, Would Love Feedback

21 点作者 salmaanp超过 7 年前

6 条评论

salmaanp超过 7 年前
Over the past year, I&#x27;ve become a bit wiser with some professional experience under my belt and thought of implementing the same idea with different technologies and using best practices, example: the previous version hosted on AWS had security policies set to everything open from everywhere and no SSL certificates. Yes, I&#x27;ve wised up now.<p>Would love any feedback, especially if you could do a quick code review on my Go and React code! React: <a href="https:&#x2F;&#x2F;github.com&#x2F;SalmaanP&#x2F;FastNews-Frontend-React" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;SalmaanP&#x2F;FastNews-Frontend-React</a> Go: <a href="https:&#x2F;&#x2F;github.com&#x2F;SalmaanP&#x2F;FastNews-Backend-Go" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;SalmaanP&#x2F;FastNews-Backend-Go</a><p>For Comparison the previous layout: <a href="http:&#x2F;&#x2F;52.34.237.199&#x2F;" rel="nofollow">http:&#x2F;&#x2F;52.34.237.199&#x2F;</a>
评论 #16137841 未加载
评论 #16137006 未加载
评论 #16138061 未加载
TipVFL超过 7 年前
Looks pretty good, there were a few design things that bugged me.<p>1) Setting your text to evenly space the words across each line makes for a terrible reading experience. Reading on mobile the amount of space between some words was painful.<p>2) I think this would look better if you aligned your margins on everything. Basically have an overall margin for the page and have everything run up right against it.<p>3) Get rid of the margin around the grey header on your content boxes and have everything in your content boxes an equal distance from the edge.<p>4) In two-column mode I&#x27;m seeing the left column is much wider than the right column and the content boxes are of differing heights. I think this will look better if you make all the content boxes equally sized.<p>Actually, I just tested again and now (on my Galaxy S6&#x27;s landscape mode) it is showing three equal columns that break the page. The header no longer reaches all the way across the screen and the third column abuts the edge of the screen.<p>6) Align your menu button to the right and your title to the left.<p>5) In your menu I think your selection outline and your search bar should be the same width with aligned edges.
评论 #16158470 未加载
gitgud超过 7 年前
Nice, really fast and simple!<p>One thing which bothers me though is the whitespace in the rows. How about using css to collapse the white space like bootstrap4&#x27;s card-columns:<p><a href="https:&#x2F;&#x2F;v4-alpha.getbootstrap.com&#x2F;components&#x2F;card&#x2F;#card-columns" rel="nofollow">https:&#x2F;&#x2F;v4-alpha.getbootstrap.com&#x2F;components&#x2F;card&#x2F;#card-colu...</a>
评论 #16158483 未加载
sova超过 7 年前
It&#x27;s cool, your folds are not tall enough to show the full headline or numbers sometimes. I think the content could be shortened or font size decreased until user asks for zoom. Overall very nice organization, style, boundaries, layout. Would like to see more focus on &quot;broad view of all topics&quot; and then &quot;zoomed in detail of one.&quot;<p>Top bar needs some sort of color to draw the eye to the presence of fresh page content.
评论 #16158499 未加载
superasn超过 7 年前
You can try experimenting with jQuery masonry plugin. It might make your layout even better. Just a hunch.
评论 #16158449 未加载
macqm超过 7 年前
Where to report bugs? E.g. some articles miss headlines.
评论 #16166071 未加载