ApollosProject / apollos-embeds

Apollos React Embeds that can be used in Webflow and other third-party websites.
https://www.npmjs.com/package/@apollosproject/apollos-embeds
0 stars 0 forks source link

Fixed issue in Search bar placeholder overlap with icon #165

Closed rajitharussel closed 9 months ago

rajitharussel commented 9 months ago

🐛 Issue

when using an small screen size, Search bar place holder text overlaps with user avatar

✏️ Solution

un necessary padding removed which causes this text overlap

🔬 To Test

  1. Open feed contains search and check the Search bar in mobile screens.
  2. http://localhost:3000/?id=facing-your-saul-TWVkaWFDb250ZW50SXRlbS0wNzgwMWQ0Ni1iYWE4LTQ0ZGUtODEwNi1jMThlODA5YmNlZWI%3D

📸 Screenshots

Before

Screenshot 2024-02-19 at 16 11 49

After

Simulator Screenshot - iPhone 15 Pro Max - 2024-02-15 at 07 19 06

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
apollos-micro-service ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2024 10:37am
apollos-web-embeds ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 19, 2024 10:37am
redreceipt commented 9 months ago

@rajitharussel can you show what it looks like when there's more text than will fit in the box

rajitharussel commented 9 months ago

@rajitharussel can you show what it looks like when there's more text than will fit in the box

https://github.com/ApollosProject/apollos-embeds/assets/138706838/e9da9316-286c-4a3c-8d45-954d8ec8f6b4

redreceipt commented 9 months ago

Can we change the placeholder copy to "What are you looking for?". The current is too long

Screenshot 2024-02-12 at 10 33 21 AM
rajitharussel commented 9 months ago

Can we change the placeholder copy to "What are you looking for?". The current is too long

Screenshot 2024-02-12 at 10 33 21 AM

@redreceipt only for the mobile or always?

redreceipt commented 9 months ago

I think always. @nlewis84 can overrule me though

nlewis84 commented 9 months ago

@rajitharussel Sorry I didn't see this earlier. We can do "What are you looking for?" always.

nlewis84 commented 9 months ago

Can you attach some before and after photos to this? It would be super helpful if you dropped a link to a feed into the test instructions. Will need a rebase, also.

rajitharussel commented 9 months ago

@rajitharussel Sorry I didn't see this earlier. We can do "What are you looking for?" always.

Can we change the placeholder copy to "What are you looking for?". The current is too long

Screenshot 2024-02-12 at 10 33 21 AM

@redreceipt only for the mobile or always?

added

rajitharussel commented 9 months ago

Can you attach some before and after photos to this? It would be super helpful if you dropped a link to a feed into the test instructions. Will need a rebase, also.

updated the description