Skip to content

Conversation

@eyevana
Copy link
Contributor

@eyevana eyevana commented Jan 26, 2023

pf-popover

Implemented Patternfly's Popover as web component.

Related issues

Preview

Link(s) to demo page(s) where this element can be viewed:

Browser requirements

Your component should work in all of the following environments:

  • Latest 2 versions of Edge
  • Internet Explorer 11 (should be useable, not pixel perfect)
  • Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
  • Firefox 78 (or latest version for Red Hat Enterprise Linux distribution)
  • Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
  • Latest 2 versions of Safari
  • Android mobile device (such as the Galaxy S9)
  • Apple mobile device (such as the iPhone X)
  • Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist

  • Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
  • Tests have been updated to cover these changes.
  • Browser testing passed.
  • Changelog updated.
  • Documentation (README.md, WHY.md, etc.) updated or added.
  • Link to the demo recording:
  • Approved by designer.

Merging

Please squash when merging and ensure your commit message uses conventional commit formatting.

Be sure to share your updates with the [email protected] mailing list!

@changeset-bot
Copy link

changeset-bot bot commented Jan 26, 2023

🦋 Changeset detected

Latest commit: 3c7161c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@patternfly/pfe-core Minor
@patternfly/elements Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Member

@bennypowers bennypowers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

time to merge in the main branch. Please address the comments here before merging, to make it easier on you when you merge them in.

Once main is merged and conflicts are resolved, please rename the shadow stylesheets to .css and validate their syntax (watch for comments and nesting)

@github-actions github-actions bot added the work in progress POC / Not ready for review label Feb 1, 2023
@github-actions github-actions bot added demo Updating demo pages doc functionality Functionality, typically pertaining to the JavaScript. styles An issue or PR pertaining only to CSS/Sass labels Feb 1, 2023
@bennypowers bennypowers changed the title feat: pf-popover feat(popover): add pf-popover Apr 27, 2023
@bennypowers bennypowers enabled auto-merge (squash) April 27, 2023 11:05
@bennypowers bennypowers merged commit 78c8e44 into main Apr 27, 2023
@bennypowers bennypowers deleted the feat/pf-popover branch April 27, 2023 13:29
@eyevana
Copy link
Contributor Author

eyevana commented Apr 27, 2023

@bennypowers the demo page looks soooo much better. Thank you! 🎉

@bennypowers bennypowers mentioned this pull request Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AT passed Automated testing has passed demo Updating demo pages feature New feature or request functionality Functionality, typically pertaining to the JavaScript. styles An issue or PR pertaining only to CSS/Sass tests Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants