Show HN: Improved My News App, Would Love Feedback(fastnews.me) |
Show HN: Improved My News App, Would Love Feedback(fastnews.me) |
Would love any feedback, especially if you could do a quick code review on my Go and React code! React: https://github.com/SalmaanP/FastNews-Frontend-React Go: https://github.com/SalmaanP/FastNews-Backend-Go
For Comparison the previous layout: http://52.34.237.199/
The bullet points summarization stuff is great. That's what I always look for on Reddit posts.
Keep it up!
Whicb api is that and how do you even check that? I did have cors enabled while developing since it was running off two different ports, could that be it?
I'd prefer pictures, though.
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.
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.
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.
4) In two-column mode I'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.
Actually, I just tested again and now (on my Galaxy S6'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.
6) Align your menu button to the right and your title to the left.
5) In your menu I think your selection outline and your search bar should be the same width with aligned edges.
I actually forgot to write the two column media query, so I guess thats why it looks bad on landscape mode in phones and tablets.
The menu seems wonky I agree.
One thing which bothers me though is the whitespace in the rows. How about using css to collapse the white space like bootstrap4's card-columns:
https://v4-alpha.getbootstrap.com/components/card/#card-colu...
Top bar needs some sort of color to draw the eye to the presence of fresh page content.
Can you give me an example of what kind of color and when?
Thanks for the feedback!