@ashwin366Submitted almost 4 years ago
Any feedback would be appreciated. Thank you in advance.
Chris Karagiannis
@cakaragiannisAll comments
- @cakaragiannisPosted almost 4 years ago
You've done a great job true to the original design. The animations are a nice touch. If I had to nit-pick, I'd remove the borders from the list items on the white cards, or pad them to match the padding on the darker card.
1 - @klaudiaholystSubmitted about 4 years ago
Hi, I would be grateful for any advice! What to do better, what is wrong, what is good ;)
Thanks a lot!!!
@cakaragiannisPosted about 4 years agoWell done on this. I agree with the previous comment. I would suggest adjusting font sizes to create a better visual hierarchy between the page and card headings. You could also soften the shadows.
0 - @narendranandaSubmitted about 4 years ago
- I want to know how to give spacing in table ?
- I how can I make this program better ?
@cakaragiannisPosted about 4 years agoHi Narendra,
With just a few small changes, it could be looking perfect!
One small tip: Wrap all elements within the body tags inside div tags. Add a class name such as "container", give it a width, then apply margin: 0 auto to centre your layout in the viewport.
Hope this helps!
0 - @cakaragiannisSubmitted about 4 years ago
If you can offer a simpler way to manage the pop-up share icons, I would love to hear it. I spent way too much time deciding how to approach it's responsive nature.
@cakaragiannisPosted about 4 years agoThank you. This was the most efficient way I came up with a solution, but as a beginner I'm certainly open to improvement suggestions.
0
