Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sheet: Add more close options and transition timing function #620

Merged
merged 22 commits into from
Oct 3, 2023

Conversation

jialin-he
Copy link
Contributor

This change: (check at least one)

  • Adds a new feature
  • Fixes a bug
  • Improves maintainability
  • Improves documentation
  • Is a release activity

Is this a breaking change? (check one)

  • Yes
  • No

Is the: (complete all)

  • Title of this pull request clear, concise, and indicative of the issue number it addresses, if any?
  • Test suite(s) passing?
  • Code coverage maximal?
  • Changeset added?
  • Component status page up to date?

What does this change address?

  • close the sheet when swipe down from short view
  • close the sheet when click on the overlay

How does this change work?

  • add more listeners
  • add transition timing functions

@jialin-he jialin-he requested a review from a team as a code owner September 25, 2023 21:01
@jialin-he jialin-he requested review from daneah, mtorres3 and satya-achanta-venkata and removed request for a team September 25, 2023 21:01
@changeset-bot
Copy link

changeset-bot bot commented Sep 25, 2023

🦋 Changeset detected

Latest commit: a180461

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

This PR includes changesets to release 1 package
Name Type
@ithaka/pharos Patch

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 25, 2023

size-limit report 📦

Path Size
packages/pharos/lib/index.js 52.56 KB (+0.57% 🔺)

Copy link
Member

@daneah daneah left a comment

Choose a reason for hiding this comment

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

Is there any reasonable test to add here?

@jialin-he jialin-he merged commit 7bf150c into develop Oct 3, 2023
@jialin-he jialin-he deleted the feat/update-sheet branch October 3, 2023 15:53
@github-actions github-actions bot mentioned this pull request Oct 3, 2023
sirrah-tam pushed a commit to sirrah-tam/pharos that referenced this pull request Dec 1, 2023
)

* feat(sheet): fix sheet height when click

* feat(sheet): emit event for expand and collapse

* feat(sheet): add changeset

* feat(sheet): make enable expansion an attribute

* feat(sheet): close sheet when swipe down or click on overlay

* docs(sheet): add change set

* feat(sheet): fix bug and set default height

* feat(sheet): add unit test

* feat(sheet): fix bug

* feat(sheet): add height to content

* feat(sheet): do not increase sheet high if already full

* feat(sheet): expand handle area and fix bug

* feat(sheet): remove margin

* feat(sheet): fix bug

* feat(sheet): only close sheet on overlay with touch screen

* feat(sheet): fix bug

* feat(sheet): fix bug

* feat(sheet): increase handle height for laptop

* feat(sheet): set max height to 95%

* test(sheet): remove test
daneah added a commit that referenced this pull request Dec 19, 2023
* develop:
  A11y revamp: Pharos buttons (non-breaking change) (#628)
  Radio, Checkbox: Fix group label (#652)
  Add elevation tokens and documentation (#643)
  fix(sidenav-link): external link opens in new tab (#645)
  Upgrade to TypeScript 5 (#644)
  feat(cli): add newly created components created using pharos-cli to initComponents files (#630)
  chore: version packages (#640)
  Coach Mark: Fix react component positioning (#638)
  Coach Mark: Documentation fixes (#639)
  chore(deps): bump @babel/traverse from 7.20.0 to 7.23.2 (#637)
  chore: version packages (#636)
  Icon: Add Panorama icon (#631)
  chore: version packages (#629)
  Loading spinner: add small and on background variant (#627)
  chore: version packages (#626)
  Sheet: allow expansion with attribute (#625)
  fix(button): remove fill on subtle disabled button on background (#618)
  chore(deps-dev): bump postcss from 8.4.25 to 8.4.31 (#624)
  chore: version packages (#623)
  Sheet: Add more close options and transition timing function (#620)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants