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

feat: the button bar covers the last filter options for Datagrid filter panel #4337 #4350

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

SeonyuK
Copy link
Contributor

@SeonyuK SeonyuK commented Feb 16, 2024

Contributes to #4337

{{short description}}
the button bar covers the last filter options for Datagrid filter panel

What did you change?

Add the bottom padding

How did you test and verify your work?

Check on the storybook
Screenshot 2024-02-16 at 3 13 20 PM

@SeonyuK SeonyuK requested a review from a team as a code owner February 16, 2024 19:18
Copy link

netlify bot commented Feb 16, 2024

Deploy Preview for carbon-for-ibm-products ready!

Name Link
🔨 Latest commit cb897a3
🔍 Latest deploy log https://app.netlify.com/sites/carbon-for-ibm-products/deploys/65cfb4eb88304d000881e906
😎 Deploy Preview https://deploy-preview-4350--carbon-for-ibm-products.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@matthewgallo matthewgallo added this pull request to the merge queue Feb 27, 2024
Merged via the queue into carbon-design-system:main with commit c8392dc Feb 27, 2024
14 checks passed
github-merge-queue bot pushed a commit that referenced this pull request May 23, 2024
#4437)

* Boilerplate of a migration guide added. WIP

* Wordsmithed a bit of the opening paragraph

* WIP

* WIP

* Updated a few of the migrated components.

* fix

* refactor: convert ToolbarButton to typescript (#4440)

* refactor: convert ToolbarButton to typescript

* docs: update url

* chore: format

* fix: add repository config to community package json (#4444)

* fix: add missing support for shouldDisableMenuItem (#4443)

* fix(DataSpreadsheet): Reduce redundancy in handleKeyPress #4187 (#4217)

* fix(DataSpreadsheet):refractor handleKeyPress method

* fix(DataSpreadsheet):refractor handleKeyPress method update1

* fix(DataSpreadsheet):refractor handleKeyPress method lint

* fix(DataSpreadsheet):refractor handleKeyPress method lint err

* fix(DataSpreadsheet):refractor handleKeyPress method lint cahnges

* feat: the button bar covers the last filter options for Datagrid filter panel #4337 (#4350)

* Test commit, disregard

* Fixing my trivial change

* docs: continued progress on the migrated components section

* docs: simple changes

* docs: added carbon components that were forwarded

* docs: added LOE column to roughly aid in estimating effort to migrate

* docs: added reference urls

* docs: updated comments

* docs: working on loe column

* docs: updated from paul's feedback

* fix: updated searchbar components props based on pull 4923

* fix: updates from review

---------

Co-authored-by: Paul Balchin <[email protected]>
Co-authored-by: matassp <[email protected]>
Co-authored-by: David Menendez <[email protected]>
Co-authored-by: Matt Gallo <[email protected]>
Co-authored-by: Amal K Joy <[email protected]>
Co-authored-by: SeonyuK <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants