Repository navigation
Feature whatweoffer component - #320
shayla-develops-webs wants to merge 13 commits into
Conversation
✅ Deploy Preview for services-dev-webdevpath ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Thanks for the feedback! Can you further explain what you mean by this ^? |
|
Hey @cherylli! Looked into both things. The pill colors are all good, they match up across all three rows, nothing off there. For the decoration sizing I checked Chrome and Edge side by side at the same zoom (50%, 67%, 100%) and the brackets scale the exact same way in both, so it's not a browser thing, it's just doing what it's supposed to (same as our other decorations). One catch though, we can't have the brackets stay full size at every screen width and keep the pill box centered/full width. There's just not enough room outside the box for that at most normal screen sizes but I could shrink the pill box to make some more room. |
thanks. Not sure if we are looking at different sources? @Web-Dev-Path/development-team can someone else have a look at this? Is the deployment preview working for anyone else, it's not scrolling for me |
Yea some extra eyes would be helpful. The deployment preview is working fine for me, no issues there. |
|
I found out the deployment preview (What we Offer scrolling) doesn't work for me on chrome and edge but works on firefox. I'm on Linux (fedora), not sure if anyone can verify if this is the case. |
|
Hi, sorry been sick lately. I think there might be some confusion about "alternate color". I mean alternate in rows, not column. In fact they both alternate. This is the original design
For example, you can see in the second image, "Startup & Nonprofit Tech Support" are blue in the first line, and third line/row, while it's yellow in the middle line/row I'm also aware that a couple are some color in all rows but I feel that it might be a figma animation issue. I think the best approach would be asking the desginer @briangesteban @liamgingell for clarification and look at the implementation |








Have you updated the CHANGELOG.md file? If not, please do it.
Yes
What is this change?
Added the what we offer section to the services page




Were there any complications while making this change?
no
How to replicate the issue?
n/a
If necessary, please describe how to test the new feature or fix.
you can test the new feature by going to localhost/services
When should this be merged?
once approvals met
Explicitly excluded
I left out the pill highlight since the dialog box covers the pill and the highlight would disappear on close. A user would only see the highlight for a sec.