[CardHeader] Removed the unnecessary right padding

[aahan96]

• PR has tests / docs demo, and is linted.
• Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
• Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Removed the unnecessary right padding from the title style. Doesn't affect desktop, but makes the title look a lot better on smaller screens. Closes #4695

[oliviertassinari]

@aahan96 I agree, this paddingRight is weird. It would be much better to rely on the browser layout engine.

What about long Card title?

[aahan96]

@oliviertassinari How about this? I think a Card Title shouldn't be that long. And if that is, it'll just be cut. Do you think we should continue with this approach?

[oliviertassinari]

@aahan96 Have you tried on a smaller screen?

I think a Card Title shouldn't be that long

We have no idea how it's going to be used. We can't take that risk.

[oliviertassinari]

@aahan96 Any update on this PR? I see two ways to fix it:

1. We take advantage of the flex-layout engin so we don't need to use the absolute positioning. However, that doesn't play well with old version of IE. We would have to drop the officially drop the support of those old versions. Hence, that something that I think would be better on the next branch.
2. We inspect the children property and only apply this paddingRight: '90px' when one of the children is a <CardExpandable /> component.

[oliviertassinari]

I'm closing this PR. Feel free to reopen it with a proper solution.