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

fix(search): search box on small screens #754

Merged
merged 18 commits into from
Sep 13, 2023

Conversation

Sinucid
Copy link
Contributor

@Sinucid Sinucid commented Sep 10, 2023

Fix small screens experience of site search box component

closes: HRZ-89724

@Sinucid Sinucid requested a review from a team as a code owner September 10, 2023 10:18
@netlify
Copy link

netlify bot commented Sep 10, 2023

Deploy Preview for dev-composable-storefront ready!

Name Link
🔨 Latest commit 134c8d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-composable-storefront/deploys/65006bfd1bbd52000839a3ab
😎 Deploy Preview https://deploy-preview-754--dev-composable-storefront.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
3 paths audited
Performance: 36 (no change from production)
Accessibility: 95 (no change from production)
Best Practices: 97 (no change from production)
SEO: 87 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@netlify
Copy link

netlify bot commented Sep 10, 2023

Deploy Preview for dev-oryx-components ready!

Name Link
🔨 Latest commit 134c8d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-oryx-components/deploys/65006bfd8aeeda000840f4b8
😎 Deploy Preview https://deploy-preview-754--dev-oryx-components.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.

@netlify
Copy link

netlify bot commented Sep 10, 2023

Deploy Preview for dev-fulfillment ready!

Name Link
🔨 Latest commit 134c8d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-fulfillment/deploys/65006bfde9517d000812b72d
😎 Deploy Preview https://deploy-preview-754--dev-fulfillment.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.

@cypress
Copy link

cypress bot commented Sep 10, 2023

1 flaky test on run #9898 ↗︎

0 13 39 0 Flakiness 1

Details:

docs
Project: Composable Storefront Commit: 134c8d953d
Status: Passed Duration: 03:59 💡
Started: Sep 12, 2023 1:53 PM Ended: Sep 12, 2023 1:57 PM
Flakiness  src/integration/product.cy.ts • 1 flaky test • sf-smoke

View Output Video

Test Artifacts
Product details page suite > should show product details Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@cypress
Copy link

cypress bot commented Sep 10, 2023

1 flaky test on run #2304 ↗︎

0 27 0 0 Flakiness 1

Details:

docs
Project: Fulfillment App Commit: 134c8d953d
Status: Passed Duration: 05:21 💡
Started: Sep 12, 2023 1:53 PM Ended: Sep 12, 2023 1:58 PM
Flakiness  src/integration/partialy-picking-picklist.cy.ts • 1 flaky test • fa-regression

View Output Video

Test Artifacts
Partial picking a picklist > should check partial picking Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Copy link
Contributor

@tobi-or-not-tobi tobi-or-not-tobi left a comment

Choose a reason for hiding this comment

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

left 2 nitpicks

libs/base/ui/search/searchbox/src/searchbox.styles.ts Outdated Show resolved Hide resolved
libs/base/ui/search/searchbox/src/searchbox.model.ts Outdated Show resolved Hide resolved
This is debatable, agreed. We do this as it's very early in the project and this will at then help the community rather then block them.
@Sinucid Sinucid merged commit e18028d into development Sep 13, 2023
29 checks passed
@Sinucid Sinucid deleted the fix/responsive-search-box branch September 13, 2023 07:10
tolerants pushed a commit that referenced this pull request Sep 13, 2023
Fix small screens experience of site search box component

closes: HRZ-89724

(cherry picked from commit e18028d)
@tolerants tolerants mentioned this pull request Sep 13, 2023
tolerants added a commit that referenced this pull request Sep 15, 2023
Release 1.0.2, backport of:
- #754
- #760

---------

Co-authored-by: Sinucid <sinucid@users.noreply.github.com>
tolerants pushed a commit that referenced this pull request Sep 15, 2023
Release 1.0.2, backport of:
- #754
- #760

---------

Co-authored-by: Sinucid <sinucid@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants