search

jackabald / NuancedNews

NuancedNews is a web appilcation that aggregates news stories from multiple sources to provide a well-rounded perspective on current events.
MIT License
5 stars 13 forks source link

Styling news cards on Home page. #37

Closed jackabald closed 2 months ago

jackabald commented 2 months ago

  1. News cards should be spaced evenly. Right now, CNN and NYT are bordering each other. Either create some padding there or create a border for these cards (whatever you think looks better, play around and see what you think looks the nicest)

  2. Style the "pre" hover div to include either a picture of the logo or a different picture that represents the company; the pre-hover style should be very simple and easy on the eyes but also professional and sleak looking.

Drawing-1 sketchpad

Most if not all of these fixes will be in the NewCards.jsx component and NewsCards.css

jackabald commented 2 months ago

@ShubhamNakashe

Would you like me to assign this issue to you?

ShubhamNakashe commented 2 months ago

Yeah Sure, I would love to solve this issue. Task is just to add styling to pre hovered new cards and space them evenly right ??

jackabald commented 2 months ago

@ShubhamNakashe Correct, that's it.

ShubhamNakashe commented 2 months ago

image

Is it ok ?? Shall I send PR request ??

jackabald commented 2 months ago

Looks great send the PR!!!