Added events card

[cloudbehl]

ref: kubevirt/web-ui#275

[coveralls]

Pull Request Test Coverage Report for Build 1336

• 10 of 10 (100.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.04%) to 87.32%

---

Totals
Change from base Build 1328:	0.04%
Covered Lines:	3216
Relevant Lines:	3530

---

💛 - Coveralls

[rawagner]

please move body to Dashboard folder (it would contain div with id and className of DashboardCardBody) so it can be reused here and in cluster overview.

[cloudbehl]

@rawagner Ahh missed your comment and did it differently. Directly used Cardbody and passed just the ID. please check if that's ok otherwise we do it your way.

[rawagner]

You should also move events.scss from ClusterOverview to Dashboard folder

[rawagner]

CardBody is already used in DashboardCardBody so I would use simple div here

[cloudbehl]

@rawagner I tried using div and it's not working. It expects class cardBody's class to be in same div.

If it's coming like this[1] then its working. but when using div the card-pf-body is a parent at that point it's not working.

1. <div id="events-body" class="card-pf-body kubevirt-events__body" style="position: relative;">

<div class="card-pf-body>
    <div id="events-body" class="card-pf-body kubevirt-events__body" style="position: relative;"> 
    </div>
</div>

[rawagner]

@cloudbehl I tried it and found the issue

1. div in EventsBody

<div id="events-body" className="kubevirt-events__body">
    {children}
  </div>

2. .kubevirt-events__body css style needs to have fixed height height: 400px instead of currently used max-height. Also it would be good to change margin to margin: -15px -15px;

[cloudbehl]

@rawagner Ack, will make the change and I will update the PR.

[cloudbehl]

@rawagner I provided the min-height and the max-height. Instead of just passing the height.
If there is only one Event in the box then we will show to the our min-height. Does it sounds good?

Also due to the margin change the processing/loaded icon is going out of context.

[rawagner]

Sounds good. I will fix the loading state in followup PR