sukiwp / suki

A flexible and lightweight WordPress theme for blogs, WooCommerce, and other purposes
https://sukiwp.com/
GNU General Public License v2.0
74 stars 12 forks source link

fix cart menu dropdown #6

Closed luizbills closed 6 years ago

luizbills commented 6 years ago

Current behavior: Cart dropdown below site content image

Expected behavior: Cart dropdown above site content

daviedR commented 6 years ago

Hi @luizbills,

I don't see the issues on my site. Do you know how to replicate this issue?

luizbills commented 6 years ago

Let's try.

daviedR commented 6 years ago

I can't replicate the issue. Is there any live URL where I can look into?

luizbills commented 6 years ago

No. But I'll provide one

luizbills commented 6 years ago

~Can you send your e-mail for me in luizpbills[at]gmail.com ?~

luizbills commented 6 years ago

i found something: the error only occurs when the cart is at the bottom of the header. image

because of these z-index:

.suki-header-bottom-bar-inner {
    z-index: 1;
    border-width: 0 0 1px 0;
}

.site-content {
    position: relative;
    z-index: 1;
}

better solution: we need increment in z-index in .suki-header-bottom-bar-inner or o remove from .site-content.

demo: http://luizpb-com.umbler.net/produto/product-demo/

daviedR commented 6 years ago

I see..

I think we should remove the z-index from .site-content.

luizbills commented 6 years ago

done

luizbills commented 6 years ago

@daviedR do you fixed this in v0.5.1?

daviedR commented 6 years ago

Yes, I fixed it on 0.5.1 along with other adjustments

luizbills commented 6 years ago

Ok! thanks.