hikaya-io / hakawati

A collection of UI components
GNU General Public License v3.0
2 stars 1 forks source link

Drawer styling: Adjust height, shadow, speed #68

Closed ninetteadhikari closed 4 years ago

ninetteadhikari commented 4 years ago

What is the Purpose?

Adjust styling for Drawer component:

What was the approach?

Scoped CSS change

Are there any concerns to addressed further before or after merging this PR?

Still trying to figure out how to lighten the mask shadow layer behind the drawer. It is still too dark

Mentions?

@andrewtpham @michaelbukachi

Issue(s) affected?

None

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hikaya/vue-ui-components/6lyq564xa ✅ Preview: https://vue-ui-components-git-drawer-styling.hikaya.now.sh

Tanvir-rahman commented 4 years ago

I was looking at the changes. Might be we can reduce some lines in style section using scss format. Well that's obviously in improvement part... 😄

ninetteadhikari commented 4 years ago

Hey. I was thinking along the lines of a dynamic prop for controlling the height.

hmm there is no Attribute for this in Element-UI so not sure how to expose the height 🤔 @Tanvir-rahman do you know how to do this? and if you recommendation to clean up css please let me know. Always happy to learn 🙂

lupyana commented 4 years ago

@ninetteadhikari

use the rgba values to dial in the shade you would like;

Tanvir-rahman commented 4 years ago

Hi @ninetteadhikari. Yeah we always are learning from each another. 😃 I think @lupyana introduced some. Also whenever we will write scss we can follow format like...

.a {
  prop: value;
  &:.c {
    prop: value
  }
 .b {
    prop: value;
 }
}

instead of .a { prop: value} .a.c { prop:value } .a .b { prop: value }

These are just some formatting. Nothing else..... 👍