Need modification in experience cards.

[ashutosh1919]

I like below cards more than the existing design. It is @dineshnadimpalli's implementation. Good work.

@dineshnadimpalli, can you raise PR to add this to masterPortfolio.

[ashutosh1919]

@dineshnadimpalli, we need to change this thing that we need to make width 100% in this

[dineshnadimpalli]

Thanks @ashutosh1919 😁. Sure, will try something with 100% width and will also try adding the description part to this one.

[saiteja13427]

Now this task is open to anyone.

Please comment here if you want to take it up and I will assign it to you!

[Vyomrana02]

I can modify the cards of the site as per the pic given above, Assign me the task.

[saiteja13427]

Sure @Vyomrana02, assigning it to you. Reach out in case of any queries!

[Vyomrana02]

Is this Okay??

[saiteja13427]

@Vyomrana02 Looks good.

How about adding skill icons as well and if skill icons are not there, then maybe that empty space shouldn't be there?

[Vyomrana02]

Is this Okay!! I have removed blank space cause in the given cards we have an key field named languages so here in portfolio.js we dont have this field. The output after removing space is like this

[saiteja13427]

This also looks fine to me, what do you think @ashutosh1919, Should we add a description or skills to the cards?

Also @Vyomrana02, in the latest image I can see the vertical lines behind the blue circles, that shouldn't happen.

[ashutosh1919]

@saiteja13427 @Vyomrana02 I think we should include description for the job role.

[saiteja13427]

Yeah, let us add description! @Vyomrana02 maybe you can add description and raise a PR

[ashutosh1919]

@Vyomrana02 have you raised a PR or are you still working in your fork? Coz I don't see PR here. Please attach it here if you have already raised.

[Vyomrana02]

@ashutosh1919 I have created PR and as per @saiteja13427 i have edited it. Output:-

[Vyomrana02]

Merged in #296 .