d33pspace / Renewal-Design-Html

0 stars 0 forks source link

Behavior overview - padding #11

Closed d33pspace closed 3 years ago

d33pspace commented 3 years ago

Screenshot 2021-09-07 092858

d33pspace commented 3 years ago

@MariiaBubley Comment from the front-end designer is: "you fixed it, it was 8 in the design. Now you want us to do 16, but the developer sees 24 in the design. What is the size right?" Can you please comment?

MariiaBubley commented 3 years ago

@academws here we use component (from page 🦄Organisms) - this component has inner padding - 8px. But page margin is 16px, so 8px+16px=24px. Please look at the mockups more attentively. image

d33pspace commented 3 years ago

@MariiaBubley This page has been updated, please confirm the edits are correct.

Behavior overview: https://d33pspace.github.io/RenewalMgtApp-Html/app/reports_behavior_overview.html

MariiaBubley commented 3 years ago

👌🏼

MariiaBubley commented 3 years ago

But there still is a problem https://d33pspace.github.io/RenewalMgtApp-Html/app/guest_profile_behavior.html

d33pspace commented 3 years ago

@academws Please fix margin spacing on guest_profile_behavoir.html

d33pspace commented 3 years ago

@MariiaBubley This is fixed. Please confirm.

MariiaBubley commented 3 years ago

👌🏼