New passthrough(pt) property implementation for Button Components

[ulasturann]

• Button
• SpeedDial
• SplitButton

[habubey]

Button commit: 5b41227

[melloware]

It looks like this change introduced a bug. If you look at Card.spec.js it now is expecting this.

<Button label="Cancel" icon="pi pi-times" className="p-button-secondary ml-2" />

See how the className is duplicated in the output?

<button
            aria-label="Cancel"
            class="p-button p-component p-button-secondary ml-2 p-button-secondary ml-2"
          >

[habubey]

What do you think about changing or editing the test file (spec).js? It seems more preferable to me.

[melloware]

we can update the test output but what I thought was strange was the classes are duplicated class="p-button p-component p-button-secondary ml-2 p-button-secondary ml-2" which does this happen in the real browser with Button now?

if this happens then it feels like the Button code is doing something new that is causing this.

[habubey]

You're right, we should avoid duplicates. Let me check.