[feat] pfelement | Update composed default to true in the emitEvent method

[wesruv]

Description of the issue

Working with some analytics folks they mentioned they couldn't get the show/hide events from the tabs on redhat.com's contact page. The tabs are inside of a pfe-content-set and it looks like the events from the child components don't make it outside of pfe-content-set.

Impacted component(s)

• pfe-content-set

Steps to reproduce

1. Go to https://www.redhat.com/en/contact
2. In the console, execute this code to show that the document isn't picking up the event: document.addEventListener('pfe-tabs:shown-tab', function(event) {console.log('lightDomListener', event)});
3. Then execute this code to show that the shadowRoot does pick up the event: document.querySelector('pfe-content-set').shadowRoot.addEventListener('pfe-tabs:shown-tab', function(event) {console.log('shadowDomListener', event)});
4. Click on tabs (e.g. Sales, Customer Service, Product & learning support, etc)

Expected behavior

I'm thinking pfe-content-set could boost the child events, and it might be nice if it pushed out a generic event (so a site dev wouldn't have to know which state the content set was in).

[wesruv]

I don't see how that PR addressed the issue, which is still occuring. Reopening.

[bennypowers]

I'd like to suggest a different approach.

Composed events might not be the way we want to go here. Other design systems use non composed events. For example these events on <vaadin-upload>

https://github.com/vaadin/web-components/blob/9da367cb581a5f2a5636e9355003ec28f72196c6/packages/upload/src/vaadin-upload.js#L662-L711

or these events on <mwc-snackbar>

https://github.com/material-components/material-web/blob/6278ee5df98ca7eae0d694e58b052fdbf4b0519d/packages/snackbar/mwc-snackbar-base.ts#L95-L113

By counter-example, shoelace does tend to compose events:
https://github.com/shoelace-style/shoelace/blob/9c31d148feff0c40b6d3407d85b4c4872e0832f3/src/internal/event.ts#L11

as does FAST:

https://github.com/microsoft/fast/blob/bdd28c8861b3523a9984ca8feda2af9019861a8a/packages/web-components/fast-element/src/components/controller.ts#L12-L16

So for analytics events, I propose that we in fact remove analytics events from PFE, subclass elements in <rh-*> and implement composed analytics events there.

Proposal doc: https://docs.google.com/document/d/1tSAV0aysOrMl3XupUuY7oV50Cxlu7AcxZdV-gtowDzg/edit#heading=h.mx5h1h2oj4l2

[bennypowers]

A nice counter-point by the author of the linked article

https://twitter.com/WestbrookJ/status/1471830306617450498?s=20