[ Cassondra ]

Results 169 comments of [ Cassondra ]

Validated that this still is not working in 1.6.0: https://codesandbox.io/s/pfe-number-with-angular-forked-r2z1y?file=/src/app/app.component.html

I made a codepen showing how you can do this on the layout level. I'm not sure that the component itself needs to be updated for this use-case but I...

Alternatively, we could add an attribute `pfe-full-width` to be added to a call-to-action to unset the max-width on the component but it seems like a lot more work to have...

@kylebuch8 Is this still an issue? It looks like tests are passing for React and Vue. I suspect Angular is still a problem though.

Here's a great article about custom, native, and hybrid. I propose we take the hybrid path: https://css-tricks.com/striking-a-balance-between-native-and-custom-select-elements/

> Needing to load another stylesheet is a different requirement than most PFE components. The following components also require external stylesheets: - pfe-band - pfe-card - pfe-tab-panel - pfe-accordion-panel -...

+1 and let's document this approach because most components will need this

Hi @grandemayta, I'll help follow up on this to see where we're at with debugging. In future, would you please avoid using "guys" as a greeting? We're a diverse group...

@kylebuch8 Was this issue resolved or do we still need a PR to get this patched?

I upgraded to 1.8.0 for pfe-navigation and am now seeing `Cannot read property 'querySelector' of null` on this line in the constructor: `this._menuItem.shadowRoot.querySelector(".pfe-navigation-item__trigger").addEventListener("click", this._menuItemClickHandler);` I think we need a promise...