#1180 - App root page - add new images #1199
Conversation
Aaryanpal
force-pushed
the
1180_app_root_image
branch
from
47cac4f to
027886f
Compare
updated screenshot in root page
Aaryanpal
force-pushed
the
1180_app_root_image
branch
from
027886f to
cc0ebcd
Compare
|
@Aaryanpal Thanks for working on this! I noticed the grey border looked quite a bit difference for the sections you updated vs. the untouched one and it got me thinking. I decided to take a look at the figma file and it appears we drifted away from the design prior to this commit. The figma design shows just the image with a shadow applied. @kasugaijin should we correct this back to match the figma design or did I miss some decisions to alter this to include the grey border? (my vote is original figma design) [figma](https://www.figma.com/design/0jVgYASUJy0KiX3BVc3dFM/Tasks?node-id=1150-254&node-type=frame)
|
|
@jmilljr24 @Aaryanpal yeah I think we should stick with the shadow on the image instead of the grey background. It is much easier to implement with CSS. @Aaryanpal how do you feel about taking that on to make all of the images use a shadow instead of grey background? Should be a pretty quick fix? |
|
@Aaryanpal bump - just checking if you're still able to refactor images per above comments? Thanks! |
|
I can take over this PR from here. |
updated screenshot in root page
🔗 Issue
#1180
✍️ Description
Added New Screenshot for the application
Browser Resolution=> 1205x900
Screenshot edit by Screenzy
Compress by => tinypng
📷 Screenshots/Demos
Desktop
Screencast.from.2024-11-30.16-24-53.webm
Mobile
Screencast.from.2024-11-30.16-26-31.webm